Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261654AbVB1Pcr (ORCPT ); Mon, 28 Feb 2005 10:32:47 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261651AbVB1Pcg (ORCPT ); Mon, 28 Feb 2005 10:32:36 -0500 Received: from fire.osdl.org ([65.172.181.4]:58299 "EHLO smtp.osdl.org") by vger.kernel.org with ESMTP id S261650AbVB1Pc3 (ORCPT ); Mon, 28 Feb 2005 10:32:29 -0500 Subject: Re: [PATCH] Fix panic in 2.6 with bounced bio and dm From: Mark Haverkamp To: Jens Axboe Cc: Linus Torvalds , Andrew Morton , linux-kernel , dm-devel@redhat.com In-Reply-To: <20050226123934.GA1254@suse.de> References: <1109351021.5014.10.camel@markh1.pdx.osdl.net> <20050225161947.5fd6d343.akpm@osdl.org> <20050226123934.GA1254@suse.de> Content-Type: text/plain Date: Mon, 28 Feb 2005 07:32:17 -0800 Message-Id: <1109604737.30227.3.camel@markh1.pdx.osdl.net> Mime-Version: 1.0 X-Mailer: Evolution 2.0.2 (2.0.2-4) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1324 Lines: 36 On Sat, 2005-02-26 at 13:39 +0100, Jens Axboe wrote: > On Fri, Feb 25 2005, Linus Torvalds wrote: > > > > > > On Fri, 25 Feb 2005, Andrew Morton wrote: > > > > > > It seems very weird for dm to be shoving NULL page*'s into the middle of a > > > bio's bvec array, so your fix might end up being a workaround pending a > > > closer look at what's going on in there. > > > > Yes. I don't see how this patch can be anything but bandaid to hide the > > real bug. Where do these "non-page" bvec's originate? > > Yep that's the fishy part, there should not be NULL pages in the middle > (or empty bios, for that matter) submitted for io. > > Mark, what was the bug that triggered you to write this patch? It happened when some pages of IO from a dm device were bounced. It looks to me when bio's are cloned in the dm code to split it for physical devices that only the pointers to pages that apply to that device are copied and th bi_idx is adjusted to point to the start, leaving some NULL pointers at the start of the bio_vec. Mark, > -- Mark Haverkamp - 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/