Date: Thu, 15 Sep 2005 03:12:36 +0100
From: Al Viro <viro@ZenIV.linux.org.uk>
To: Sripathi Kodi <sripathik@in.ibm.com>
Cc: Roland McGrath <roland@redhat.com>, Linus Torvalds <torvalds@osdl.org>,
       Andrew Morton <akpm@osdl.org>, linux-kernel@vger.kernel.org,
       patrics@interia.pl, Ingo Molnar <mingo@elte.hu>
Subject: Re: [PATCH 2.6.13.1] Patch for invisible threads
Message-ID: <20050915021236.GA25261@ZenIV.linux.org.uk>
References: <20050915005552.38626180A1A@magilla.sf.frob.com> <4328D0A8.90504@in.ibm.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4328D0A8.90504@in.ibm.com>
User-Agent: Mutt/1.4.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1045
Lines: 26

On Wed, Sep 14, 2005 at 08:38:48PM -0500, Sripathi Kodi wrote:
> Roland McGrath wrote:
> >>If you don't like this idea at all, please let me know if there any other 
> >>way of solving the invisible threads problem, short of taking out 
> >>->permission() altogether from proc_task_inode_operations.
> >
> >
> >Have you investigated my suggestion to move __exit_fs from do_exit to
> >release_task?
> 
> Roland,
> 
> No, I had missed this completely. Sorry.
> 
> I just gave it a quick try and it seems to be working fine. I have only 
> moved __exit_fs to the top of release_task, not moved exit_namespace after 
> it. I will try to run some tests to see if this is working fine. Thanks a 
> lot.

Among other things, it means that zombies keep pinning their root and cwd
down, AFAICS.  Not Good(tm).
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/