Re: [PATCHv2] cld: use XDR for all messages

2010-02-09 Thread Pete Zaitcev
On Tue, 09 Feb 2010 04:38:21 -0500 Jeff Garzik j...@garzik.org wrote: On 02/09/2010 04:20 AM, Jeff Garzik wrote: Useful per-packet cld [the daemon] verbose output seems to have disappeared? Or did I miss a knob somewhere? cldcli no longers dumps per-packet debug info at maximum verbosity,

Re: [PATCH] Re: [PATCHv2] cld: use XDR for all messages

2010-02-05 Thread Jeff Garzik
On 02/03/2010 04:45 PM, Colin McCabe wrote: On Tue, Feb 2, 2010 at 10:35 PM, Jeff Garzikj...@garzik.org wrote: I will continue whittling down the patch until it just contains the XDR changes themselves. In tools/Makefile.am, I don't think you need $(top_srcdir)/lib any more, since

Re: [PATCH] Re: [PATCHv2] cld: use XDR for all messages

2010-02-03 Thread Colin McCabe
On Tue, Feb 2, 2010 at 10:35 PM, Jeff Garzik j...@garzik.org wrote: On 01/29/2010 08:01 PM, Colin McCabe wrote: That seems reasonable. All of those functions could be looked at as common. The intention was to group together a bunch of functions that were useful for packet formatting.

Re: [PATCHv2] cld: use XDR for all messages

2010-01-26 Thread Colin McCabe
Hi all, I'm currently reworking this patch to address the concerns mentioned here (mostly autotools-related). I'm also going to separate it into multiple patches. A few questions: I would really like cld_msg_rpc.h to show up in include, for consistency with the other headers. Is it possible

Re: [PATCHv2] cld: use XDR for all messages

2010-01-20 Thread Colin McCabe
On Wed, Jan 20, 2010 at 5:38 PM, Jeff Garzik j...@garzik.org wrote: On 01/15/2010 12:49 PM, Colin McCabe wrote: This patch moves CLD from using manual data serialization to using XDR (via rpcgen). Both the packet header and the message body are now serialized and deserialized using XDR.