From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757533Ab2CPTnG (ORCPT ); Fri, 16 Mar 2012 15:43:06 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:33132 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756977Ab2CPTnD (ORCPT ); Fri, 16 Mar 2012 15:43:03 -0400 X-Sasl-enc: Bar0SwoYyErucy+1HH4LjoQo9caKevY/CpWLJbMAZgkE 1331926981 Date: Fri, 16 Mar 2012 12:43:00 -0700 From: Greg KH To: Paul Taysom Cc: Paul Taysom , Mandeep Baines , Jens Axboe , Theodore Tso , Andrew Morton , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Alexander Viro , linux-fsdevel@vger.kernel.org, stable@kernel.org Subject: Re: [PATCH] fs: Fix mod_timer crash when removing USB sticks Message-ID: <20120316194300.GA19075@kroah.com> References: <1326405431-988-1-git-send-email-taysom@chromium.org> <20120316173610.GA26601@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 16, 2012 at 12:29:15PM -0700, Paul Taysom wrote: > On Fri, Mar 16, 2012 at 10:36 AM, Greg KH wrote: > > > > On Thu, Jan 12, 2012 at 01:57:11PM -0800, Paul Taysom wrote: > > > A USB stick with a ext file system on it, would occasionally crash > > > when the stick was pulled. > > > > > > The problem was a timer was being set on the Backing Device Interface, > > > bdi, after the USB device had been removed and the bdi had been > > > unregistered. The bdi would then be later reinitialized by zeroing > > > the timer without removing from the timer from the timer queue. > > > This would eventually result in a kernel crash (NULL ptr dereference). > > > > > > When the bdi is unregistered, the dev field is set to NULL. This > > > indication is used by bdi_unregister to only unregister the device > > > once. > > > > > > Fix: When the backing device is invalidated, the mapping > > > backing_dev_info > > > should be redirected to the default_backing_dev_info. > > > > > > Created 3 USB sticks with ext2, ext4 and one with both apple and DOS > > > file systems on it. Inserted and removed USB sticks many times in random > > > order. With out the bug fix, the kernel would soon crash. With the fix, > > > it did not. Ran on both stumpy and amd64-generic. > > > > > > Signed-off-by: Paul Taysom > > > Cc: Mandeep Baines > > > Cc: Greg KH > > > Cc: Jens Axboe > > > Cc: Theodore Tso > > > Cc: Andrew Morton > > > Cc: > > > Cc: > > > Cc: Alexander Viro > > > Cc: > > > Cc: > > > --- > > >  fs/block_dev.c |    1 + > > >  1 files changed, 1 insertions(+), 0 deletions(-) > > > > > > diff --git a/fs/block_dev.c b/fs/block_dev.c > > > index afe74dd..322cd05 100644 > > > --- a/fs/block_dev.c > > > +++ b/fs/block_dev.c > > > @@ -110,6 +110,7 @@ void invalidate_bdev(struct block_device *bdev) > > >        * But, for the strange corners, lets be cautious > > >        */ > > >       cleancache_flush_inode(mapping); > > > +     mapping->backing_dev_info = &default_backing_dev_info; > > >  } > > >  EXPORT_SYMBOL(invalidate_bdev); > > > > What ever happened to this patch?  Is it still needed?  Can you still > > reproduce the problem on Linus's tree and older kernels? > > > > > Never heard anything back. Ted supplied a partial fix in 3.2.6 (I > believe) for just the ext4 file system. Who should I follow up with? If the fix went into the 3.2-stable tree, then it's in Linus's tree already, which is good. But, what about all of the other filesystems you hit this on, do we need to make the same change to all of them? If so, that kind of implies your original patch is the correct one :) As for who to poke, Ted, Al, Jens, what should we do here? thanks, greg k-h