All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Christoph Hellwig <hch@lst.de>,
	iommu@lists.linux-foundation.org,
	Russell King <linux@armlinux.org.uk>,
	Santosh Shilimkar <ssantosh@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jim Quinlan <james.quinlan@broadcom.com>,
	Nathan Chancellor <natechancellor@gmail.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Robin Murphy <robin.murphy@arm.com>,
	Rob Herring <robh+dt@kernel.org>,
	Frank Rowand <frowand.list@gmail.com>,
	Ohad Ben-Cohen <ohad@wizery.com>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	linux-remoteproc@vger.kernel.org, linux-usb@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-sh@vger.kernel.org,
	linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org,
	devicetree@vger.kernel.org, arnaud.pouliquen@st.com,
	loic.pallardy.st.com@lst.de
Subject: Re: [PATCH 6/6] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset
Date: Wed, 16 Sep 2020 06:13:59 +0000	[thread overview]
Message-ID: <20200916061359.GA8424@lst.de> (raw)
In-Reply-To: <20200915195501.GA3666944@xps15>

On Tue, Sep 15, 2020 at 01:55:01PM -0600, Mathieu Poirier wrote:
> That did the trick - the stm32 platform driver's probe() function completes and
> the remote processor is operatinal. 
> 
> That being said the value returned by function dma_to_pfn()
> is 0x137fff in the original code and 0xfffff with your patches applied.

Yes, that is intentional.  The old code just applied the range and got
an out of range offset, the new one reports the max offset.

WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Christoph Hellwig <hch@lst.de>,
	iommu@lists.linux-foundation.org,
	Russell King <linux@armlinux.org.uk>,
	Santosh Shilimkar <ssantosh@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jim Quinlan <james.quinlan@broadcom.com>,
	Nathan Chancellor <natechancellor@gmail.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Robin Murphy <robin.murphy@arm.com>,
	Rob Herring <robh+dt@kernel.org>,
	Frank Rowand <frowand.list@gmail.com>,
	Ohad Ben-Cohen <ohad@wizery.com>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	linux-remoteproc@vger.kernel.org, linux-usb@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-sh@vger.kernel.org,
	linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org,
	devicetree@vger.kernel.org, arnaud.pouliquen@st.com,
	loic.pallardy.st.com@lst.de
Subject: Re: [PATCH 6/6] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset
Date: Wed, 16 Sep 2020 08:13:59 +0200	[thread overview]
Message-ID: <20200916061359.GA8424@lst.de> (raw)
In-Reply-To: <20200915195501.GA3666944@xps15>

On Tue, Sep 15, 2020 at 01:55:01PM -0600, Mathieu Poirier wrote:
> That did the trick - the stm32 platform driver's probe() function completes and
> the remote processor is operatinal. 
> 
> That being said the value returned by function dma_to_pfn()
> is 0x137fff in the original code and 0xfffff with your patches applied.

Yes, that is intentional.  The old code just applied the range and got
an out of range offset, the new one reports the max offset.

WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: linux-sh@vger.kernel.org, linux-pci@vger.kernel.org,
	linux-remoteproc@vger.kernel.org,
	Frank Rowand <frowand.list@gmail.com>,
	Christoph Hellwig <hch@lst.de>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Russell King <linux@armlinux.org.uk>,
	linux-acpi@vger.kernel.org, loic.pallardy.st.com@lst.de,
	Ohad Ben-Cohen <ohad@wizery.com>,
	devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	Santosh Shilimkar <ssantosh@kernel.org>,
	Nathan Chancellor <natechancellor@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-usb@vger.kernel.org, arnaud.pouliquen@st.com,
	linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org,
	Jim Quinlan <james.quinlan@broadcom.com>,
	Robin Murphy <robin.murphy@arm.com>
Subject: Re: [PATCH 6/6] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset
Date: Wed, 16 Sep 2020 08:13:59 +0200	[thread overview]
Message-ID: <20200916061359.GA8424@lst.de> (raw)
In-Reply-To: <20200915195501.GA3666944@xps15>

On Tue, Sep 15, 2020 at 01:55:01PM -0600, Mathieu Poirier wrote:
> That did the trick - the stm32 platform driver's probe() function completes and
> the remote processor is operatinal. 
> 
> That being said the value returned by function dma_to_pfn()
> is 0x137fff in the original code and 0xfffff with your patches applied.

Yes, that is intentional.  The old code just applied the range and got
an out of range offset, the new one reports the max offset.
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: linux-sh@vger.kernel.org, linux-pci@vger.kernel.org,
	linux-remoteproc@vger.kernel.org,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Frank Rowand <frowand.list@gmail.com>,
	Christoph Hellwig <hch@lst.de>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Russell King <linux@armlinux.org.uk>,
	linux-acpi@vger.kernel.org, loic.pallardy.st.com@lst.de,
	Ohad Ben-Cohen <ohad@wizery.com>,
	devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	Santosh Shilimkar <ssantosh@kernel.org>,
	Nathan Chancellor <natechancellor@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-usb@vger.kernel.org, arnaud.pouliquen@st.com,
	linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org,
	Jim Quinlan <james.quinlan@broadcom.com>,
	Robin Murphy <robin.murphy@arm.com>
Subject: Re: [PATCH 6/6] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset
Date: Wed, 16 Sep 2020 08:13:59 +0200	[thread overview]
Message-ID: <20200916061359.GA8424@lst.de> (raw)
In-Reply-To: <20200915195501.GA3666944@xps15>

On Tue, Sep 15, 2020 at 01:55:01PM -0600, Mathieu Poirier wrote:
> That did the trick - the stm32 platform driver's probe() function completes and
> the remote processor is operatinal. 
> 
> That being said the value returned by function dma_to_pfn()
> is 0x137fff in the original code and 0xfffff with your patches applied.

Yes, that is intentional.  The old code just applied the range and got
an out of range offset, the new one reports the max offset.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-09-16  6:13 UTC|newest]

Thread overview: 148+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-10  5:40 support range based offsets in dma-direct Christoph Hellwig
2020-09-10  5:40 ` Christoph Hellwig
2020-09-10  5:40 ` Christoph Hellwig
2020-09-10  5:40 ` Christoph Hellwig
2020-09-10  5:40 ` [PATCH 1/3] ARM/dma-mapping: move various helpers from dma-mapping.h to dma-direct.h Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10 18:02   ` Robin Murphy
2020-09-10 18:02     ` Robin Murphy
2020-09-10 18:02     ` Robin Murphy
2020-09-10 18:02     ` Robin Murphy
2020-09-11  6:25     ` Christoph Hellwig
2020-09-11  6:25       ` Christoph Hellwig
2020-09-11  6:25       ` Christoph Hellwig
2020-09-11  6:25       ` Christoph Hellwig
2020-09-10  5:40 ` [PATCH 2/3] ARM/keystone: move the DMA offset handling under ifdef CONFIG_ARM_LPAE Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-11 11:12   ` Robin Murphy
2020-09-11 11:12     ` Robin Murphy
2020-09-11 11:12     ` Robin Murphy
2020-09-11 11:12     ` Robin Murphy
2020-09-11 11:15   ` Russell King - ARM Linux admin
2020-09-11 11:15     ` Russell King - ARM Linux admin
2020-09-11 11:15     ` Russell King - ARM Linux admin
2020-09-11 11:15     ` Russell King - ARM Linux admin
2020-09-11 11:27     ` Robin Murphy
2020-09-11 11:27       ` Robin Murphy
2020-09-11 11:27       ` Robin Murphy
2020-09-11 11:27       ` Robin Murphy
2020-09-11 18:00     ` santosh.shilimkar
2020-09-11 18:00       ` santosh.shilimkar
2020-09-11 18:00       ` santosh.shilimkar
2020-09-11 18:00       ` santosh.shilimkar
2020-09-10  5:40 ` [PATCH 3/3] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  5:40   ` Christoph Hellwig
2020-09-10  7:53   ` Greg KH
2020-09-10  7:53     ` Greg KH
2020-09-10  7:53     ` Greg KH
2020-09-10  7:53     ` Greg KH
2020-09-10  9:13     ` Christoph Hellwig
2020-09-10  9:13       ` Christoph Hellwig
2020-09-10  9:13       ` Christoph Hellwig
2020-09-10  9:13       ` Christoph Hellwig
2020-09-10 16:12       ` Greg KH
2020-09-10 16:12         ` Greg KH
2020-09-10 16:12         ` Greg KH
2020-09-10 16:12         ` Greg KH
2020-09-11 16:12   ` Robin Murphy
2020-09-11 16:12     ` Robin Murphy
2020-09-11 16:12     ` Robin Murphy
2020-09-11 16:12     ` Robin Murphy
2020-09-12  6:46     ` Christoph Hellwig
2020-09-12  6:46       ` Christoph Hellwig
2020-09-12  6:46       ` Christoph Hellwig
2020-09-12  6:46       ` Christoph Hellwig
2020-09-14  7:33 ` support range based offsets in dma-direct v2 Christoph Hellwig
2020-09-14  7:33   ` Christoph Hellwig
2020-09-14  7:33   ` Christoph Hellwig
2020-09-14  7:33   ` Christoph Hellwig
2020-09-14  7:33   ` [PATCH 1/6] ARM/dma-mapping: remove a __arch_page_to_dma #error Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33   ` [PATCH 2/6] ARM/dma-mapping: remove dma_to_virt Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33   ` [PATCH 3/6] ARM/dma-mapping: move various helpers from dma-mapping.h to dma-direct.h Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33   ` [PATCH 4/6] ARM/keystone: move the DMA offset handling under ifdef CONFIG_ARM_LPAE Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33   ` [PATCH 5/6] usb: don't inherity DMA properties for USB devices Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:51     ` Greg Kroah-Hartman
2020-09-14  7:51       ` Greg Kroah-Hartman
2020-09-14  7:51       ` Greg Kroah-Hartman
2020-09-14  7:51       ` Greg Kroah-Hartman
2020-09-14  7:33   ` [PATCH 6/6] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14  7:33     ` Christoph Hellwig
2020-09-14 23:01     ` Mathieu Poirier
2020-09-14 23:01       ` Mathieu Poirier
2020-09-14 23:01       ` Mathieu Poirier
2020-09-14 23:01       ` Mathieu Poirier
2020-09-15  5:41       ` Christoph Hellwig
2020-09-15  5:41         ` Christoph Hellwig
2020-09-15  5:41         ` Christoph Hellwig
2020-09-15  5:41         ` Christoph Hellwig
2020-09-15 19:55         ` Mathieu Poirier
2020-09-15 19:55           ` Mathieu Poirier
2020-09-15 19:55           ` Mathieu Poirier
2020-09-15 19:55           ` Mathieu Poirier
2020-09-16  6:13           ` Christoph Hellwig [this message]
2020-09-16  6:13             ` Christoph Hellwig
2020-09-16  6:13             ` Christoph Hellwig
2020-09-16  6:13             ` Christoph Hellwig
2020-09-16  6:14 ` support range based offsets in dma-direct v3 Christoph Hellwig
2020-09-16  6:14   ` Christoph Hellwig
2020-09-16  6:14   ` Christoph Hellwig
2020-09-16  6:14   ` Christoph Hellwig
2020-09-16  6:14   ` [PATCH 1/6] ARM/dma-mapping: remove a __arch_page_to_dma #error Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14   ` [PATCH 2/6] ARM/dma-mapping: remove dma_to_virt Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14   ` [PATCH 3/6] ARM/dma-mapping: move various helpers from dma-mapping.h to dma-direct.h Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14   ` [PATCH 4/6] ARM/keystone: move the DMA offset handling under ifdef CONFIG_ARM_LPAE Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14   ` [PATCH 5/6] usb: don't inherity DMA properties for USB devices Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14   ` [PATCH 6/6] dma-mapping: introduce DMA range map, supplanting dma_pfn_offset Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16  6:14     ` Christoph Hellwig
2020-09-16 17:36     ` Mathieu Poirier
2020-09-16 17:36       ` Mathieu Poirier
2020-09-16 17:36       ` Mathieu Poirier
2020-09-16 17:36       ` Mathieu Poirier
2020-10-26 15:33     ` Geert Uytterhoeven
2020-10-26 15:33       ` Geert Uytterhoeven
2020-10-26 15:33       ` Geert Uytterhoeven
2020-10-26 15:33       ` Geert Uytterhoeven
2020-09-17 16:45   ` support range based offsets in dma-direct v3 Christoph Hellwig
2020-09-17 16:45     ` Christoph Hellwig
2020-09-17 16:45     ` Christoph Hellwig
2020-09-17 16:45     ` Christoph Hellwig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200916061359.GA8424@lst.de \
    --to=hch@lst.de \
    --cc=arnaud.pouliquen@st.com \
    --cc=bjorn.andersson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=f.fainelli@gmail.com \
    --cc=frowand.list@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=iommu@lists.linux-foundation.org \
    --cc=james.quinlan@broadcom.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-remoteproc@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=loic.pallardy.st.com@lst.de \
    --cc=mathieu.poirier@linaro.org \
    --cc=natechancellor@gmail.com \
    --cc=ohad@wizery.com \
    --cc=robh+dt@kernel.org \
    --cc=robin.murphy@arm.com \
    --cc=ssantosh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.