Re: [PATCH] ceph/osd_client: add support for CEPH_OSD_OP_GETXATTR

2015-10-15 Thread Ilya Dryomov
On Thu, Oct 15, 2015 at 12:51 AM, David Disseldorp wrote: > On Wed, 14 Oct 2015 19:57:46 +0200, Ilya Dryomov wrote: > >> On Wed, Oct 14, 2015 at 7:37 PM, David Disseldorp wrote: > ... >> > Ping, any feedback on the patch? >> >> The patch itself looks OK, except for

Re: [PATCH] ceph/osd_client: add support for CEPH_OSD_OP_GETXATTR

2015-10-14 Thread Ilya Dryomov
On Wed, Oct 14, 2015 at 7:37 PM, David Disseldorp wrote: > On Fri, 9 Oct 2015 16:43:09 +0200, David Disseldorp wrote: > >> Allows for xattr retrieval. Response data buffer allocation is the >> responsibility of the osd_req_op_xattr_init() caller. > > Ping, any feedback on the

Re: [PATCH] ceph/osd_client: add support for CEPH_OSD_OP_GETXATTR

2015-10-14 Thread David Disseldorp
On Fri, 9 Oct 2015 16:43:09 +0200, David Disseldorp wrote: > Allows for xattr retrieval. Response data buffer allocation is the > responsibility of the osd_req_op_xattr_init() caller. Ping, any feedback on the patch? Cheers, David -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH] ceph/osd_client: add support for CEPH_OSD_OP_GETXATTR

2015-10-14 Thread David Disseldorp
On Wed, 14 Oct 2015 19:57:46 +0200, Ilya Dryomov wrote: > On Wed, Oct 14, 2015 at 7:37 PM, David Disseldorp wrote: ... > > Ping, any feedback on the patch? > > The patch itself looks OK, except for the part where you rename a local > variable for no reason, AFACT. Thanks for the