From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: [PATCH] block: Free queue resources at blk_release_queue() Date: Tue, 27 Sep 2011 20:02:08 -0600 Message-ID: <4E828020.4060205@kernel.dk> References: <201109221318.p8MDIXPP020015@pentland.suse.de> <20110928004722.GA27421@kernel.dk> <4E82751C.9050908@kernel.dk> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Linus Torvalds Cc: Hannes Reinecke , James Bottomley , "linux-scsi@vger.kernel.org" , Linux Kernel List-Id: linux-scsi@vger.kernel.org On 2011-09-27 19:59, Linus Torvalds wrote: > On Tue, Sep 27, 2011 at 6:15 PM, Jens Axboe wrote: >>> >>> But if you forward the actual patch to me (the one I see on lkml seems >>> to be broken and doesn't compile cleanly because it's assiging a >>> structure to a pointer), I'll try it out anyway. >> >> Thanks, that would be great. It's inlined below. > > Well, I did several USB eject events, and nothing bad happened. > > But as mentioned, I don't think that means much. Have you tried this > with slub debugging and poisoning? It might be worth some extra > testing that way. I'm traveling at the moment, and don't have a stick handy here. I'll try it tomorrow morning, before and after (both with pagealloc etc debuggin on). -- Jens Axboe