From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: [PATCH] ipr: don't doublefree pages from scatterlist Date: Mon, 06 Feb 2006 09:02:59 -0600 Message-ID: <1139238179.3022.2.camel@mulgrave.il.steeleye.com> References: <43E66FB6.6070303@us.ibm.com> <20060206.014608.22328385.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from stat9.steeleye.com ([209.192.50.41]:11404 "EHLO hancock.sc.steeleye.com") by vger.kernel.org with ESMTP id S932132AbWBFPD5 (ORCPT ); Mon, 6 Feb 2006 10:03:57 -0500 In-Reply-To: <20060206.014608.22328385.davem@davemloft.net> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: "David S. Miller" Cc: hugh@veritas.com, brking@us.ibm.com, akpm@osdl.org, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org On Mon, 2006-02-06 at 01:46 -0800, David S. Miller wrote: > That's a bug, frankly. Sparc64 doesn't need to do anything like > that. Spamming the page pointers is really really bogus and I'm > surprised this doesn't make more stuff explode. > > It was never the intention to allow the DMA mapping support code > to modify the page, offset, and length members of the scatterlist. > Only the DMA components. > > I'd really prefer that those assignments get fixed and an explicit > note added to Documentation/DMA-mapping.txt about this. > > It's rediculious that these generic subsystem drivers need to > know about this. :) I complained about this x86_64 behaviour ages ago. Andi claimed it was the only way they could get there merging algorithm to work. It actually triggered a bug in SCSI because in-flight I/O that was rejected gets unmapped and then remapped (which was, originally, not working). They finally fixed it by making the unmap put back the original scatterlist. Perhaps this should go to linux-arch just in case anyone else copied x86_64? James