[SRU][U][PATCH v2 0/1] Enable CONFIG_HSA_AMD_P2P

[Andrea Righi]

On Thu, Jan 05, 2023 at 06:30:53PM +0200, Timo Aaltonen wrote:
> Andrea Righi kirjoitti 5.1.2023 klo 18.29:
> > On Thu, Jan 05, 2023 at 04:21:45PM +0200, Timo Aaltonen wrote:
> > > BugLink: https://bugs.launchpad.net/bugs/1987394
> > > 
> > > This is a config option that AMD requested us to enable, so it's time to enable it for
> > > the unstable kernel too.
> > 
> > I assume you want this both in lunar/linux and lunar/linux-unstable,
> > like [U][L], correct?
> 
> That'd be fine too, but since lunar will eventually get 6.2 via unstable, I
> thought it'd be enough to enable just there. But either way is ok.

Yeah, makes sense.

BTW, I think we can't just enable HSA_AMD_P2P, because it seems to
depend on other configs:

  config HSA_AMD_P2P
          bool "HSA kernel driver support for peer-to-peer for AMD GPU devices"
          depends on HSA_AMD && PCI_P2PDMA && DMABUF_MOVE_NOTIFY

I think the right patch should be something like the following:

 debian.master/config/annotations | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/debian.master/config/annotations b/debian.master/config/annotations
index 34995ba23a99..386f74a3decb 100644
--- a/debian.master/config/annotations
+++ b/debian.master/config/annotations
@@ -3071,7 +3071,7 @@ CONFIG_DMABUF_DEBUG                             policy<{'amd64': 'n', 'arm64': '
 CONFIG_DMABUF_HEAPS                             policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'y'}>
 CONFIG_DMABUF_HEAPS_CMA                         policy<{'arm64': 'y', 'armhf': 'y', 'riscv64': 'y'}>
 CONFIG_DMABUF_HEAPS_SYSTEM                      policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'y'}>
-CONFIG_DMABUF_MOVE_NOTIFY                       policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 'riscv64': 'n', 's390x': 'n'}>
+CONFIG_DMABUF_MOVE_NOTIFY                       policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'n', 'ppc64el': 'y', 'riscv64': 'n', 's390x': 'n'}>
 CONFIG_DMABUF_SELFTESTS                         policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 'riscv64': 'n', 's390x': 'n'}>
 CONFIG_DMABUF_SYSFS_STATS                       policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 'riscv64': 'n', 's390x': 'n'}>
 CONFIG_DMADEVICES                               policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'n'}>
@@ -5106,6 +5106,9 @@ CONFIG_HP_ILO                                   policy<{'amd64': 'm', 'arm64': '
 CONFIG_HP_WATCHDOG                              policy<{'amd64': 'm', 'arm64': 'm'}>
 CONFIG_HP_WMI                                   policy<{'amd64': 'm'}>
 CONFIG_HSA_AMD                                  policy<{'amd64': 'y', 'arm64': 'y', 'ppc64el': 'y'}>
+CONFIG_HSA_AMD_P2P                              policy<{'amd64': 'y', 'arm64': 'y', 'ppc64el': 'y'}>
+CONFIG_HSA_AMD_P2P                              note<'LP: #1987394'>
+
 CONFIG_HSA_AMD_SVM                              policy<{'amd64': 'y', 'arm64': 'y', 'ppc64el': 'y'}>
 CONFIG_HSI                                      policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'ppc64el': 'm', 'riscv64': 'm', 's390x': 'n'}>
 CONFIG_HSI_BOARDINFO                            policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y'}>
@@ -8882,7 +8885,7 @@ CONFIG_PCI_MVEBU                                policy<{'armhf': 'y'}>
 CONFIG_PCI_NR_FUNCTIONS                         policy<{'s390x': '512'}>
 CONFIG_PCI_NR_FUNCTIONS                         note<'LP: #1866056'>
 
-CONFIG_PCI_P2PDMA                               policy<{'amd64': 'n', 'arm64': 'n', 'ppc64el': 'n'}>
+CONFIG_PCI_P2PDMA                               policy<{'amd64': 'y', 'arm64': 'y', 'ppc64el': 'y'}>
 CONFIG_PCI_PASID                                policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'n'}>
 CONFIG_PCI_PF_STUB                              policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'ppc64el': 'm', 'riscv64': 'm', 's390x': 'm'}>
 CONFIG_PCI_PRI                                  policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'n'}>