Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760979AbYGOOVu (ORCPT ); Tue, 15 Jul 2008 10:21:50 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753757AbYGOOVk (ORCPT ); Tue, 15 Jul 2008 10:21:40 -0400 Received: from sh.osrg.net ([192.16.179.4]:44128 "EHLO sh.osrg.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753238AbYGOOVj (ORCPT ); Tue, 15 Jul 2008 10:21:39 -0400 Date: Tue, 15 Jul 2008 23:20:14 +0900 To: mpatocka@redhat.com Cc: fujita.tomonori@lab.ntt.co.jp, jens.axboe@oracle.com, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, davem@davemloft.net, linux-parisc@vger.kernel.org Subject: Re: [PATCH] block: fix q->max_segment_size checking in blk_recalc_rq_segments about VMERGE From: FUJITA Tomonori In-Reply-To: References: <1216118676-13625-1-git-send-email-fujita.tomonori@lab.ntt.co.jp> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-Id: <20080715231956A.fujita.tomonori@lab.ntt.co.jp> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1506 Lines: 33 On Tue, 15 Jul 2008 09:37:05 -0400 (EDT) Mikulas Patocka wrote: > On Tue, 15 Jul 2008, FUJITA Tomonori wrote: > > > blk_recalc_rq_segments assumes that any segments can be merged in the > > case of BIOVEC_VIRT_MERGEABLE && !BIOVEC_VIRT_OVERSIZE. However, an > > IOMMU can't merge segments if the total length of the segments is > > larger than max_segment_size (the LLD restriction). > > > > Due to this bug, a LLD may get the larger number of segments than > > nr_hw_segments because the block layer puts more segments in a request > > than it should do. > > > > This bug could happen on alpha, parisc, and sparc, which use VMERGE. > > Parisc doesn't use virtual merge accounting (there is variable for it but > it's always 0). Hmm, really? Looks like PARISC IOMMUs (ccio-dma.c and sba_iomm.c) set parisc_vmerge_boundary (CC'ed PARISC mailing list). > On sparc64 it is broken anyway with or without your patch. Yeah, we need to modify SPARC64 IOMMU code (I'm not sure that it's worth). Right now, the best fix is setting BIO_VMERGE_BOUNDARY to 0. > And alpha alone doesn't justify substantial code bloat in generic block > layer. So I propose this patch to drop it at all. Jens, what do you think about removing VMERGE code? -- 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/