* [PATCH] arm64: Enable PCI write-combine resources under sysfs
@ 2020-08-21 15:51 Clint Sbisa
2020-08-27 14:41 ` Clint Sbisa
2020-08-31 15:22 ` Clint Sbisa
0 siblings, 2 replies; 5+ messages in thread
From: Clint Sbisa @ 2020-08-21 15:51 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon
Cc: Clint Sbisa, Benjamin Herrenschmidt, Ali Saidi, David Woodhouse,
Guy Tzalik, linux-arm-kernel, linux-kernel
Using write-combine is crucial for performance of PCI devices where
significant amounts of transactions go over PCI BARs.
arm64 supports write-combine PCI mappings, so the appropriate define
has been added which will expose write-combine mappings under sysfs
for prefetchable PCI resources.
Signed-off-by: Clint Sbisa <csbisa@amazon.com>
---
arch/arm64/include/asm/pci.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/include/asm/pci.h b/arch/arm64/include/asm/pci.h
index 70b323cf8300..b33ca260e3c9 100644
--- a/arch/arm64/include/asm/pci.h
+++ b/arch/arm64/include/asm/pci.h
@@ -17,6 +17,7 @@
#define pcibios_assign_all_busses() \
(pci_has_flag(PCI_REASSIGN_ALL_BUS))
+#define arch_can_pci_mmap_wc() 1
#define ARCH_GENERIC_PCI_MMAP_RESOURCE 1
extern int isa_dma_bridge_buggy;
--
2.23.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: Enable PCI write-combine resources under sysfs
2020-08-21 15:51 [PATCH] arm64: Enable PCI write-combine resources under sysfs Clint Sbisa
@ 2020-08-27 14:41 ` Clint Sbisa
2020-08-31 15:22 ` Clint Sbisa
1 sibling, 0 replies; 5+ messages in thread
From: Clint Sbisa @ 2020-08-27 14:41 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon
Cc: Benjamin Herrenschmidt, Ali Saidi, David Woodhouse, Guy Tzalik,
linux-arm-kernel, linux-kernel
On Fri, Aug 21, 2020 at 03:51:54PM +0000, Clint Sbisa wrote:
> Using write-combine is crucial for performance of PCI devices where
> significant amounts of transactions go over PCI BARs.
>
> arm64 supports write-combine PCI mappings, so the appropriate define
> has been added which will expose write-combine mappings under sysfs
> for prefetchable PCI resources.
>
> Signed-off-by: Clint Sbisa <csbisa@amazon.com>
> ---
> arch/arm64/include/asm/pci.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/include/asm/pci.h b/arch/arm64/include/asm/pci.h
> index 70b323cf8300..b33ca260e3c9 100644
> --- a/arch/arm64/include/asm/pci.h
> +++ b/arch/arm64/include/asm/pci.h
> @@ -17,6 +17,7 @@
> #define pcibios_assign_all_busses() \
> (pci_has_flag(PCI_REASSIGN_ALL_BUS))
>
> +#define arch_can_pci_mmap_wc() 1
> #define ARCH_GENERIC_PCI_MMAP_RESOURCE 1
>
> extern int isa_dma_bridge_buggy;
> --
> 2.23.3
>
Ping. Let me know if any more info is needed.
Clint
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: Enable PCI write-combine resources under sysfs
2020-08-21 15:51 [PATCH] arm64: Enable PCI write-combine resources under sysfs Clint Sbisa
2020-08-27 14:41 ` Clint Sbisa
@ 2020-08-31 15:22 ` Clint Sbisa
1 sibling, 0 replies; 5+ messages in thread
From: Clint Sbisa @ 2020-08-31 15:22 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel; +Cc: Catalin Marinas, Will Deacon
On Fri, Aug 21, 2020 at 03:51:54PM +0000, Clint Sbisa wrote:
> Using write-combine is crucial for performance of PCI devices where
> significant amounts of transactions go over PCI BARs.
>
> arm64 supports write-combine PCI mappings, so the appropriate define
> has been added which will expose write-combine mappings under sysfs
> for prefetchable PCI resources.
>
> Signed-off-by: Clint Sbisa <csbisa@amazon.com>
> ---
> arch/arm64/include/asm/pci.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/include/asm/pci.h b/arch/arm64/include/asm/pci.h
> index 70b323cf8300..b33ca260e3c9 100644
> --- a/arch/arm64/include/asm/pci.h
> +++ b/arch/arm64/include/asm/pci.h
> @@ -17,6 +17,7 @@
> #define pcibios_assign_all_busses() \
> (pci_has_flag(PCI_REASSIGN_ALL_BUS))
>
> +#define arch_can_pci_mmap_wc() 1
> #define ARCH_GENERIC_PCI_MMAP_RESOURCE 1
>
> extern int isa_dma_bridge_buggy;
> --
> 2.23.3
>
Please disregard this submission, I'm resubmitting it to more appropriate
maintainers than what was suggested by the get_maintainer script.
Clint
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: Enable PCI write-combine resources under sysfs
2020-09-01 23:22 ` Benjamin Herrenschmidt
@ 2020-09-02 8:57 ` Will Deacon
0 siblings, 0 replies; 5+ messages in thread
From: Will Deacon @ 2020-09-02 8:57 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Bjorn Helgaas, Clint Sbisa, linux-pci, Lorenzo Pieralisi,
Catalin Marinas, linux-kernel
On Wed, Sep 02, 2020 at 09:22:53AM +1000, Benjamin Herrenschmidt wrote:
> On Tue, 2020-09-01 at 13:37 -0500, Bjorn Helgaas wrote:
> > On Mon, Aug 31, 2020 at 03:18:27PM +0000, Clint Sbisa wrote:
> > > Using write-combine is crucial for performance of PCI devices where
> > > significant amounts of transactions go over PCI BARs.
> > >
> > > arm64 supports write-combine PCI mappings, so the appropriate
> > > define
> > > has been added which will expose write-combine mappings under sysfs
> > > for prefetchable PCI resources.
> > >
> > > Signed-off-by: Clint Sbisa <csbisa@amazon.com>
> >
> > Fine with me, I assume Will or Catalin will apply this.
>
> Haha ! Client had sent it to them originally and I told him to resend
> it to linux-pci, yourself and Lorenzo :-)
>
> So the confusion is on me.
>
> Will, Catalin, it's all yours. You should have the original patch in
> your mbox already, otherwise:
>
> https://patchwork.kernel.org/patch/11729875/
Yup, it's not the radar. We don't usually start queuing stuff until -rc3, so
I'm working through the backlog this week. Would like an Ack from Lorenzo,
though.
Will
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: Enable PCI write-combine resources under sysfs
[not found] <20200901183702.GA196025@bjorn-Precision-5520>
@ 2020-09-01 23:22 ` Benjamin Herrenschmidt
2020-09-02 8:57 ` Will Deacon
0 siblings, 1 reply; 5+ messages in thread
From: Benjamin Herrenschmidt @ 2020-09-01 23:22 UTC (permalink / raw)
To: Bjorn Helgaas, Clint Sbisa
Cc: linux-pci, Lorenzo Pieralisi, Catalin Marinas, Will Deacon, linux-kernel
On Tue, 2020-09-01 at 13:37 -0500, Bjorn Helgaas wrote:
> On Mon, Aug 31, 2020 at 03:18:27PM +0000, Clint Sbisa wrote:
> > Using write-combine is crucial for performance of PCI devices where
> > significant amounts of transactions go over PCI BARs.
> >
> > arm64 supports write-combine PCI mappings, so the appropriate
> > define
> > has been added which will expose write-combine mappings under sysfs
> > for prefetchable PCI resources.
> >
> > Signed-off-by: Clint Sbisa <csbisa@amazon.com>
>
> Fine with me, I assume Will or Catalin will apply this.
Haha ! Client had sent it to them originally and I told him to resend
it to linux-pci, yourself and Lorenzo :-)
So the confusion is on me.
Will, Catalin, it's all yours. You should have the original patch in
your mbox already, otherwise:
https://patchwork.kernel.org/patch/11729875/
Cheers,
Ben.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-09-02 8:57 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-21 15:51 [PATCH] arm64: Enable PCI write-combine resources under sysfs Clint Sbisa
2020-08-27 14:41 ` Clint Sbisa
2020-08-31 15:22 ` Clint Sbisa
[not found] <20200901183702.GA196025@bjorn-Precision-5520>
2020-09-01 23:22 ` Benjamin Herrenschmidt
2020-09-02 8:57 ` Will Deacon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).