Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Thu, 17 Oct 2002 02:39:25 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Thu, 17 Oct 2002 02:39:25 -0400 Received: from inet-mail2.oracle.com ([148.87.2.202]:4266 "EHLO inet-mail2.oracle.com") by vger.kernel.org with ESMTP id ; Thu, 17 Oct 2002 02:39:24 -0400 Date: Wed, 16 Oct 2002 23:45:10 -0700 From: Joel Becker To: Jens Axboe Cc: Linux Kernel , "Stephen C. Tweedie" Subject: Re: [PATCH] superbh, fractured blocks, and grouped io Message-ID: <20021017064510.GX22117@nic1-pc.us.oracle.com> References: <20021014135100.GD28283@suse.de> <20021017005109.GV22117@nic1-pc.us.oracle.com> <20021017055942.GC9245@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20021017055942.GC9245@suse.de> User-Agent: Mutt/1.4i X-Burt-Line: Trees are cool. Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1126 Lines: 30 On Thu, Oct 17, 2002 at 07:59:42AM +0200, Jens Axboe wrote: > Eh no, it actually doesn't bounce anything! I added a test to bypass > bouncing temporarily in blk_queue_bounce() (see the BH_Super test in > there), but forgot to replace it with the real thing. The superbh > contains no data, so is not a candidate for bouncing. I understand that the superbh isn't a candidate for bouncing. The problem is that the rest of the bhs are, and they never get bounced. scsi_merge.c checks "if (PageHighMem(bh->b_page)) BUG();" in __init_io(), and that obviously crashes. Joel -- "Conservative, n. A statesman who is enamoured of existing evils, as distinguished from the Liberal, who wishes to replace them with others." - Ambrose Bierce, The Devil's Dictionary Joel Becker Senior Member of Technical Staff Oracle Corporation E-mail: joel.becker@oracle.com Phone: (650) 506-8127 - 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/