[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.LRH.2.02.1310301944310.22093@file01.intranet.prod.int.rdu2.redhat.com>
Date: Wed, 30 Oct 2013 20:08:41 -0400 (EDT)
From: Mikulas Patocka <mpatocka@...hat.com>
To: Greg KH <greg@...ah.com>
cc: Alasdair G Kergon <agk@...hat.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Thomas Gleixner <tglx@...utronix.de>,
Mike Snitzer <snitzer@...hat.com>, Neil Brown <neilb@...e.de>,
Fr�d�ric Weisbecker
<fweisbec@...il.com>, Knut Petersen <Knut_Petersen@...nline.de>,
linux-kernel <linux-kernel@...r.kernel.org>,
"dm-devel@...hat.com" <dm-devel@...hat.com>,
Paul McKenney <paulmck@...ux.vnet.ibm.com>,
Ingo Molnar <mingo@...nel.org>
Subject: Re: FW: Re: [dm-devel] [BUG 3.12.rc4] Oops: unable to handle kernel
paging request during shutdown
On Wed, 30 Oct 2013, Greg KH wrote:
> > I suggest that you implement a function kobject_put_free, that decrements
> > the kobject reference count and waits until others stop using the kobject
> > and the reference count drops to zero. Then, you change drivers to use
> > kobject_put_free instead of kobject_put in their unload routine - that
> > will fix this sort of module unload races.
>
> The "module unload" issue is rare, thankfully, but yes, this type of
> function will be showing up in 3.13-rc1 through the btrfs tree as it
> needs that functionality, so feel free to use it to resolve this issue
> if you need it.
>
> thanks,
>
> greg k-h
With CONFIG_DEBUG_KOBJECT_RELEASE this issue is not rare -
CONFIG_DEBUG_KOBJECT_RELEASE deliberately provokes it.
Nice to hear that it will be fixed. You should patch other drivers to use
this new function in the unload routine as well.
What is the name of the function? I didn't find it in linux-btrfs.git or
btrfs-next.git.
Mikulas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists