On 02/07/2010 02:15 PM, Pete Zaitcev wrote:
The "traditional" CLD API is too difficult to program. In particular, switching from a rigid "Group" policy to arbitrary paths in existing clients in Chunk and tabled turned out to be next to impossible. The issue is due to the fundamental nature of the API as based on very fine-grained events. So, in the interests of clients, introduce the "new" API. Its basic definition is in<ncld.h>. It presents a filesystem-like interface, extended with event callbacks for coarse events, such as an end of CLD session. The patch converts all in-tree clients from cldc_xxx to ncld_xxx and discards some of code that became unused (test/util.c). Signed-Off-By: Pete Zaitcev<zaitcev@xxxxxxxxxx>
At a quick glance, looks good. A few nits like the "if (log)" stuff and indenting struct field names jump out, but nothing major. I'll give it a thorough review and apply on Monday...
Jeff -- To unsubscribe from this list: send the line "unsubscribe hail-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html