From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by ozlabs.org (Postfix) with ESMTP id D5251B7112 for ; Sat, 25 Sep 2010 08:20:53 +1000 (EST) Subject: Re: [PATCH RFCv1 1/2] dmaengine: add support for scatterlist to scatterlist transfers From: Dan Williams To: "Ira W. Snyder" In-Reply-To: <20100924220419.GC24654@ovro.caltech.edu> References: <1285357571-23377-1-git-send-email-iws@ovro.caltech.edu> <1285357571-23377-2-git-send-email-iws@ovro.caltech.edu> <20100924212443.GA24654@ovro.caltech.edu> <1285365194.21375.22.camel@dwillia2-linux> <20100924220419.GC24654@ovro.caltech.edu> Content-Type: text/plain; charset="UTF-8" Date: Fri, 24 Sep 2010 15:20:14 -0700 Message-ID: <1285366814.21375.30.camel@dwillia2-linux> Mime-Version: 1.0 Cc: "linuxppc-dev@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 2010-09-24 at 15:04 -0700, Ira W. Snyder wrote: > On Fri, Sep 24, 2010 at 02:53:14PM -0700, Dan Williams wrote: > > What about overrun or underrun do we not care if src_total != dst_total? > > > > Otherwise looks ok. > > > > I don't know if we should care about that. The algorithm handles that > case just fine. It copies the maximum amount it can, which is exactly > min(src_total, dst_total). Whichever scatterlist runs out of entries > first is the shortest. > > As a real world example, my driver verifies that both scatterlists have > exactly the right number of bytes available before trying to program the > hardware. Ok, just handle the prep failure and I think we are good to go. -- Dan