Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755576Ab3GLXIe (ORCPT ); Fri, 12 Jul 2013 19:08:34 -0400 Received: from mail-lb0-f169.google.com ([209.85.217.169]:34996 "EHLO mail-lb0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751345Ab3GLXIc (ORCPT ); Fri, 12 Jul 2013 19:08:32 -0400 Message-ID: <51E08C6B.2080802@cogentembedded.com> Date: Sat, 13 Jul 2013 03:08:27 +0400 From: Sergei Shtylyov Organization: Cogent Embedded User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 MIME-Version: 1.0 To: Russell King - ARM Linux CC: Santosh Shilimkar , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-scsi@vger.kernel.org Subject: Re: [RFC/RFT PATCH 3/5] scsi: Use dma_max_pfn(dev) helper for bounce_limit calculations References: <1373665694-7580-1-git-send-email-santosh.shilimkar@ti.com> <1373665694-7580-4-git-send-email-santosh.shilimkar@ti.com> <51E07B6E.5090900@cogentembedded.com> <20130712222506.GI24642@n2100.arm.linux.org.uk> In-Reply-To: <20130712222506.GI24642@n2100.arm.linux.org.uk> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1635 Lines: 45 Hello. On 07/13/2013 02:25 AM, Russell King - ARM Linux wrote: >>> DMA bounce limit is the maximum direct DMA'able memory beyond which >>> bounce buffers has to be used to perform dma operations. SCSI driver >>> relies on dma_mask but its calculation is based on max_*pfn which >>> don't have uniform meaning across architectures. So make use of >>> dma_max_pfn() which is expected to return the DMAable maximum pfn >>> value across architectures. >>> Cc: Russell King >>> Cc: linux-scsi@vger.kernel.org >>> Signed-off-by: Santosh Shilimkar >>> --- >>> drivers/scsi/scsi_lib.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c >>> index 86d5220..e8275fa 100644 >>> --- a/drivers/scsi/scsi_lib.c >>> +++ b/drivers/scsi/scsi_lib.c >>> @@ -1668,7 +1668,7 @@ u64 scsi_calculate_bounce_limit(struct Scsi_Host *shost) >>> >>> host_dev = scsi_get_device(shost); >>> if (host_dev && host_dev->dma_mask) >>> - bounce_limit = *host_dev->dma_mask; >>> + bounce_limit = dma_max_pfn(host_dev) << PAGE_SHIFT; >> You definitely forgot -1 here. > Please explain your point. Previously, 'bounce_limit' would look like 0xffffffff (unless I'm mistaken), now it would look like 0xfffff000 which is hardly what we're looking for, no? WBR, Sergei -- 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/