Bug 966018

Summary:	nfs: performance of untar of llinux kernel tarball is pathetic
Product:	[Community] GlusterFS	Reporter:	Pranith Kumar K <pkarampu>
Component:	replicate	Assignee:	Pranith Kumar K <pkarampu>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	urgent	Docs Contact:
Priority:	high
Version:	mainline	CC:	ansubram, bturner, gluster-bugs, pkarampu, rhs-bugs, saujain, spradhan, vbellur
Target Milestone:	---
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	glusterfs-3.4.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	962510	Environment:
Last Closed:	2013-07-24 17:18:02 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	962510
Bug Blocks:

Comment 1 Anand Avati 2013-05-22 12:15:28 UTC

REVIEW: http://review.gluster.org/5066 (cluster/afr: Perform delayed changelog wakeups for anon fd) posted (#1) for review on master by Pranith Kumar Karampuri (pkarampu)

Comment 2 Anand Avati 2013-05-22 12:19:44 UTC

REVIEW: http://review.gluster.org/5067 (cluster/afr: Perform delayed changelog wakeups for anon fd) posted (#1) for review on release-3.4 by Pranith Kumar Karampuri (pkarampu)

Comment 3 Anand Avati 2013-05-23 07:01:25 UTC

REVIEW: http://review.gluster.org/5066 (cluster/afr: Perform delayed changelog wakeups for anon fd) posted (#2) for review on master by Pranith Kumar Karampuri (pkarampu)

Comment 4 Anand Avati 2013-05-23 07:13:35 UTC

REVIEW: http://review.gluster.org/5067 (cluster/afr: Perform delayed changelog wakeups for anon fd) posted (#2) for review on release-3.4 by Pranith Kumar Karampuri (pkarampu)

Comment 5 Anand Avati 2013-06-05 09:47:50 UTC

REVIEW: http://review.gluster.org/5066 (cluster/afr: Perform delayed changelog wakeups for anon fd) posted (#3) for review on master by Vijay Bellur (vbellur)

Comment 6 Anand Avati 2013-06-10 20:04:56 UTC

COMMIT: http://review.gluster.org/5067 committed in release-3.4 by Anand Avati (avati) 
------
commit 2fda7a9de2edd624a03558d9a9a580e5639fef1f
Author: Pranith Kumar K <pkarampu>
Date:   Wed May 22 15:18:19 2013 +0530

    cluster/afr: Perform delayed changelog wakeups for anon fd
    
    Problem:
    Nfs xlator never does open on a file for performing writes,
    afr does not perform changelog wakeup for this fd so operations
    which do metadata operations as soon as the data operations are
    completed perceive a delay od 'post-op-delay-secs'.
    
    Fix:
    Perform changelog wakeup on anon-fd if the fd with same pid is
    not present in inode-list.
    
    Note:
    This approach is a short-term fix. A proper fix needs a new domain
    for taking metadata locks so that data/metadata locks don't compete
    with each other.
    
    BUG: 966018
    Change-Id: Ia9188a253e7943801b665e1b9205e2f551952d87
    Signed-off-by: Pranith Kumar K <pkarampu>
    Reviewed-on: http://review.gluster.org/5067
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Anand Avati <avati>

Comment 7 Anand Avati 2013-06-20 08:45:12 UTC

COMMIT: http://review.gluster.org/5066 committed in master by Vijay Bellur (vbellur) 
------
commit cc9ca7d2b2f5c91d80265c28ca724cb233c2bd77
Author: Pranith Kumar K <pkarampu>
Date:   Wed May 22 15:18:19 2013 +0530

    cluster/afr: Perform delayed changelog wakeups for anon fd
    
    Problem:
    Nfs xlator never does open on a file for performing writes,
    afr does not perform changelog wakeup for this fd so operations
    which do metadata operations as soon as the data operations are
    completed perceive a delay of 'post-op-delay-secs'.
    
    Fix:
    Perform changelog wakeup on anon-fd if the fd with same pid is
    not present in inode-list.
    
    Note:
    This approach is a short-term fix. A proper fix needs a new domain
    for taking metadata locks so that data/metadata locks don't compete
    with each other.
    
    Change-Id: I253afb289eadf30c7951e56fb2c4840d7132f5e4
    BUG: 966018
    Signed-off-by: Pranith Kumar K <pkarampu>
    Reviewed-on: http://review.gluster.org/5066
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>