Bug 808078 - Setting xdata on writev doesn't work
Summary: Setting xdata on writev doesn't work
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: protocol
Version: mainline
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: ---
Assignee: Amar Tumballi
QA Contact: Sachidananda Urs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-29 13:58 UTC by Jeff Darcy
Modified: 2013-12-19 00:07 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-24 17:27:35 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Jeff Darcy 2012-03-29 13:58:48 UTC
Xdr_to_generic expects the variable-length xdata to be in the same contiguous memory buffer as the fixed part, but currently only the length is present.  This doesn't affect most calls, because the buffer layout is a side effect of the vector_sizer stuff in the transport layer and most calls don't use that method, but it probably does affect the return side of readv as well.

Comment 1 Jeff Darcy 2012-03-29 13:59:56 UTC
http://review.gluster.com/3038

Comment 2 Amar Tumballi 2012-04-02 04:07:38 UTC
Thanks for the patch Jeff, I will run some tests with/wo your patch and mark it reviewed after that.

Comment 3 Amar Tumballi 2012-05-24 09:59:14 UTC
http://review.gluster.com/3182 is merged in master branch, not planing to backport right away to release-3.3 as there are no consumers yet. Hence moving it to ON_QA for master branch.

Comment 4 Sachidananda Urs 2012-06-01 05:30:13 UTC
Not part of 3.3.0 release will be testing later. Changes are in master.

Comment 5 Sachidananda Urs 2012-12-26 07:33:20 UTC
This isn't fixed in qa5, will retry again on qa6.


Note You need to log in before you can comment on or make changes to this bug.