Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Tue, 11 Mar 2003 20:22:36 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Tue, 11 Mar 2003 20:22:36 -0500 Received: from packet.digeo.com ([12.110.80.53]:53397 "EHLO packet.digeo.com") by vger.kernel.org with ESMTP id ; Tue, 11 Mar 2003 20:22:35 -0500 Date: Tue, 11 Mar 2003 17:28:15 -0800 From: Andrew Morton To: Luben Tuikov Cc: linux-kernel@vger.kernel.org, axboe@suse.de Subject: Re: [PATCH] fix kernel oops in generic_unplug_device() for md Message-Id: <20030311172815.46ac305d.akpm@digeo.com> In-Reply-To: <3E6E8B6D.1000501@splentec.com> References: <3E6E8B6D.1000501@splentec.com> X-Mailer: Sylpheed version 0.8.10 (GTK+ 1.2.10; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 12 Mar 2003 01:33:08.0960 (UTC) FILETIME=[565D5E00:01C2E837] Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1068 Lines: 33 Luben Tuikov wrote: > > The following patch fixes a kernel oops when doing > blk_unplug_work() (oopses in generic_unplug_device()) for md. > > The oops and the original report are here: > http://MARC.10East.com/?l=linux-kernel&m=104706400705154&w=2 > Yup, is a bug. I received the below fix from Neil today which looks simpler. diff -puN drivers/block/ll_rw_blk.c~auto-unplugging-fix drivers/block/ll_rw_blk.c --- 25/drivers/block/ll_rw_blk.c~auto-unplugging-fix Tue Mar 11 15:04:00 2003 +++ 25-akpm/drivers/block/ll_rw_blk.c Tue Mar 11 15:04:00 2003 @@ -1004,7 +1004,8 @@ void generic_unplug_device(void *data) static void blk_unplug_work(void *data) { - generic_unplug_device(data); + request_queue_t *q = data; + q->unplug_fn(q); } static void blk_unplug_timeout(unsigned long data) _ - 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/