From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756856AbZEJVgY (ORCPT ); Sun, 10 May 2009 17:36:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755651AbZEJVgA (ORCPT ); Sun, 10 May 2009 17:36:00 -0400 Received: from mail-bw0-f174.google.com ([209.85.218.174]:60750 "EHLO mail-bw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754663AbZEJVf7 (ORCPT ); Sun, 10 May 2009 17:35:59 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to :mime-version:content-disposition:message-id:content-type :content-transfer-encoding; b=cnf9wOYLnORaB7eJuZ4UXsT494eY3eW2YhDX51wjSwgiOrZ4s5so9og6BXzFCMEvTk 8PqkVE/tB+Kdg5b2ZZ9JtJedtsCbobnxaqzWrOEgHJiXugSiG33Q5UWnQefdCq7yszWj 6iHdxsePJ62rP6Cm3WVXJsUKa/GGGbWnvlQBs= From: Bartlomiej Zolnierkiewicz To: Borislav Petkov Subject: Re: [PATCH 02/14] ide-atapi: switch to blk_rq_bytes() on do_request() path Date: Sun, 10 May 2009 23:32:29 +0200 User-Agent: KMail/1.11.2 (Linux/2.6.30-rc3-00442-ga3668b0; KDE/4.2.2; i686; ; ) Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org References: <1241855134-4984-1-git-send-email-petkovbb@gmail.com> <1241855134-4984-3-git-send-email-petkovbb@gmail.com> In-Reply-To: <1241855134-4984-3-git-send-email-petkovbb@gmail.com> MIME-Version: 1.0 Content-Disposition: inline Message-Id: <200905102332.29839.bzolnier@gmail.com> Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Saturday 09 May 2009 09:45:22 Borislav Petkov wrote: > After the recent struct request cleanups, blk_rq_bytes() is guaranteed > to be valid and is the current total length of the rq's bio. Use that > instead of pc->req_xfer in the do_request() path after the command has > been queued. > > The remaining usage of pc->req_xfer now is only until we map the rq to a > bio. > > While at it: > > - remove local caching of rq completion length in ide_tape_issue_pc() > > Signed-off-by: Borislav Petkov > --- > drivers/ide/ide-atapi.c | 12 +++++++----- > drivers/ide/ide-floppy.c | 8 ++++---- > drivers/ide/ide-tape.c | 12 ++++++------ > 3 files changed, 17 insertions(+), 15 deletions(-) [...] > @@ -592,8 +593,7 @@ static void ide_tape_create_rw_cmd(idetape_tape_t *tape, > put_unaligned(cpu_to_be32(length), (unsigned int *) &pc->c[1]); > pc->c[1] = 1; > pc->buf = NULL; > - pc->buf_size = length * tape->blk_size; > - pc->req_xfer = pc->buf_size; > + pc->buf_size = blk_rq_bytes(rq); > if (pc->req_xfer == tape->buffer_size) Seems like this chunk also needs s/pc->req_xfer/pc->buf_size/ fixup.