Bug 762055 (GLUSTER-323)

Summary: fd leak with quick-read loaded in translator tree
Product: [Community] GlusterFS Reporter: Raghavendra G <raghavendra>
Component: quick-readAssignee: Raghavendra G <raghavendra>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: mainlineCC: anush
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: RTNR Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Anand Avati 2009-10-15 13:19:23 UTC
PATCH: http://patches.gluster.com/patch/1917 in master (performance/quick-read: frame->local need not be set in qr_open_cbk.)

Comment 1 Anand Avati 2009-10-15 13:19:31 UTC
PATCH: http://patches.gluster.com/patch/1918 in release-2.0 (performance/quick-read: frame->local need not be set in qr_open_cbk.)

Comment 2 Raghavendra G 2009-10-15 14:04:33 UTC
A kernel compile can result in the number of opened files in a system crossing the system limit.

qr_open_cbk checks for frame->local and if it is NULL, it sets op_ret = -1. And with op_ret = -1 the file is not marked as opened. Since many fops does not allocate frame->local, the opens sent by these fops does not mark the file to be opened though open is successful at posix. Hence multiple opens are sent on same fd and since only one release is called per fd, there is fd leak.