From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ie0-f169.google.com ([209.85.223.169]:35393 "EHLO mail-ie0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752233AbbEGPTJ (ORCPT ); Thu, 7 May 2015 11:19:09 -0400 Received: by ieczm2 with SMTP id zm2so41815335iec.2 for ; Thu, 07 May 2015 08:19:08 -0700 (PDT) Date: Thu, 7 May 2015 10:19:05 -0500 From: Bjorn Helgaas To: wdavis@nvidia.com Cc: joro@8bytes.org, iommu@lists.linux-foundation.org, linux-pci@vger.kernel.org, tripperda@nvidia.com, jhubbard@nvidia.com, jglisse@redhat.com, "David S. Miller" , Yinghai Lu Subject: Re: [PATCH 3/6] dma-mapping: pci: add pci_(un)map_resource Message-ID: <20150507151905.GL24643@google.com> References: <1430505138-2877-1-git-send-email-wdavis@nvidia.com> <1430505138-2877-4-git-send-email-wdavis@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1430505138-2877-4-git-send-email-wdavis@nvidia.com> Sender: linux-pci-owner@vger.kernel.org List-ID: [+cc Dave for sparc64, Yinghai] On Fri, May 01, 2015 at 01:32:15PM -0500, wdavis@nvidia.com wrote: > From: Will Davis > > Simply route these through to the new dma_(un)map_resource APIs. > > Signed-off-by: Will Davis > Reviewed-by: Terence Ripperda > Reviewed-by: John Hubbard > --- > include/asm-generic/pci-dma-compat.h | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/include/asm-generic/pci-dma-compat.h b/include/asm-generic/pci-dma-compat.h > index c110843..ac4a4ad 100644 > --- a/include/asm-generic/pci-dma-compat.h > +++ b/include/asm-generic/pci-dma-compat.h > @@ -61,6 +61,20 @@ pci_unmap_page(struct pci_dev *hwdev, dma_addr_t dma_address, > dma_unmap_page(hwdev == NULL ? NULL : &hwdev->dev, dma_address, size, (enum dma_data_direction)direction); > } > > +static inline dma_addr_t > +pci_map_resource(struct pci_dev *hwdev, struct resource *resource, > + unsigned long offset, size_t size, int direction) > +{ > + return dma_map_resource(hwdev == NULL ? NULL : &hwdev->dev, resource, offset, size, (enum dma_data_direction)direction); > +} On sparc64, PCI bus addresses, e.g., raw BAR values, can be 64 bits wide, but dma_addr_t is only 32 bits [1]. So dma_addr_t is a bit of a problem here. It's likely that we will add a pci_bus_addr_t, but that hasn't happened yet [2]. We do have existing problems already, e.g,. pci_bus_address() returns a dma_addr_t, so it has the same problem. So I guess this is just a heads-up that this needs to be fixed eventually. Bjorn [1] http://lkml.kernel.org/r/20150327.145016.86183910134380870.davem@davemloft.net [2] http://lkml.kernel.org/r/1427857069-6789-2-git-send-email-yinghai@kernel.org