From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Slaby Subject: Re: OOPS in configfs when doing d_delete Date: Fri, 27 May 2011 23:12:22 +0200 Message-ID: <4DE013B6.9070801@gmail.com> References: <4D623C62.8030509@suse.cz> <20110221104359.GA18538@noexit> <4D6242AD.2090600@suse.cz> <20110222091415.GD30966@noexit> <4DCBA9A5.9040306@gmail.com> <20110518195842.GB5898@noexit.corp.google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit To: Jiri Slaby , npiggin@kernel.dk, Al Viro , linux-fsdevel@vger.kernel.org, LKML Return-path: In-Reply-To: <20110518195842.GB5898@noexit.corp.google.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On 05/18/2011 09:58 PM, Joel Becker wrote: > On Thu, May 12, 2011 at 11:34:29AM +0200, Jiri Slaby wrote: >> On 02/22/2011 10:14 AM, Joel Becker wrote: >>> On Mon, Feb 21, 2011 at 11:47:09AM +0100, Jiri Slaby wrote: >>>>> I think you're right about the superfluous test, but I need more >>>>> investigation to see what's going on. Thanks for the report. >>>>> What was causing attach_group() to fail? Do you know? >>>> >>>> Dunno, I just modprobe'd the configfs example from Doc dir >>>> (configfs_example_macros). >>> >>> I'm going to revisit the failed example (which shouldn't fail, I >>> would think). Can you try the following patch to safely handle the >>> failure rather than crashing the kernel? >>> >>> Joel >> >> Hi, what's the status of this? (It's verified to work some time ago.) > > The runtime fix is queued up for mainline. I'll be looking at > the example code fix for the next merge window. The fixes for the examples were merged today after being some time in the -mm tree. thanks, -- js