From mboxrd@z Thu Jan  1 00:00:00 1970
From: "J. Bruce Fields" <bfields@fieldses.org>
Subject: Re: nfsdv4 leaks file descriptors
Date: Wed, 20 Apr 2011 11:30:53 -0400
Message-ID: <20110420153053.GD15919@fieldses.org>
References: <20110412140148.GA17242@gheift>
 <20110413211211.GA25189@fieldses.org>
 <20110414054844.GF17242@gheift>
 <20110414191005.GB1721@fieldses.org>
 <20110415132025.GA6126@gheift>
 <20110415223621.GB12254@fieldses.org>
 <20110420110834.GB11224@gheift>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: linux-nfs@vger.kernel.org
To: Gerhard Heift <ml-nfs-linux-20110412-ef47-UG2eIutGyHWzQB+pC5nmwQ@public.gmane.org>
Return-path: <linux-nfs-owner@vger.kernel.org>
Received: from fieldses.org ([174.143.236.118]:42018 "EHLO fieldses.org"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1755141Ab1DTPay (ORCPT <rfc822;linux-nfs@vger.kernel.org>);
	Wed, 20 Apr 2011 11:30:54 -0400
In-Reply-To: <20110420110834.GB11224@gheift>
Sender: linux-nfs-owner@vger.kernel.org
List-ID: <linux-nfs.vger.kernel.org>

On Wed, Apr 20, 2011 at 01:08:36PM +0200, Gerhard Heift wrote:
> On Fri, Apr 15, 2011 at 06:36:21PM -0400, J. Bruce Fields wrote:
> > On Fri, Apr 15, 2011 at 03:20:30PM +0200, Gerhard Heift wrote:
> > > On Thu, Apr 14, 2011 at 03:10:06PM -0400, J. Bruce Fields wrote:
> > > > On Thu, Apr 14, 2011 at 07:48:45AM +0200, Gerhard Heift wrote:
> > > > > On Wed, Apr 13, 2011 at 05:12:11PM -0400, J. Bruce Fields wrote:
> > > > > > On Tue, Apr 12, 2011 at 04:01:50PM +0200, Gerhard Heift wrote:
> > > > > > > Hello,
> > > > > > > 
> > > > > > > I have a linux kernel (2.6.38.2, armv5tel) running which exports a
> > > > > > > directory with following options:
> > > > > > 
> > > > > > Should be fixed in more recent stable kernel; let me know if not.
> > > > > 
> > > > > An even more recent stable than 2.6.38.2? I just compiled 2.6.39-rc3 and
> > > > > there the leak happens too.
> > > > 
> > > > Ugh.  OK, investigating.
> > > 
> > > It took me a bit of time, but I located the commit which introduced this
> > > behavior: acfdf5c383b38f7f4dddae41b97c97f1ae058f49
> > 
> > Doh--probably we just need this (lightly tested).
> > 
> > Thanks for tracking this down.
> 
> This patch works for me. Thanks for fixing it.

Thanks for the confirmation.  I'll pass the patch along soon.

--b.