At Tue, 28 Aug 2012 16:43:50 +0800, levin li wrote: > > On 08/28/2012 04:38 PM, Liu Yuan wrote: > > On 08/28/2012 04:14 PM, levin li wrote: > >> - hdr.flags = SD_FLAG_CMD_WRITE; > >> hdr.data_length = sizeof(sys->this_node); > > > > Why not? We do write &sys->this_node as data to transfer. > > > > Thanks, > > Yuan > > > > For cluster requests we send data through cluster driver (corosync or > zookeeper), > in which case SD_FLAG_CMD_WRITE is never used.
This change looks really strange to me. If SD_FLAG_CMD_WRITE is set, hdr.data_length means the size of the sent data, and if not, it means the size of receive data buffer. I think we shouldn't do multicast the data when SD_FLAG_CMD_WRITE is not set. Thanks, Kazutaka -- sheepdog mailing list sheepdog@lists.wpkg.org http://lists.wpkg.org/mailman/listinfo/sheepdog