[SRU][F][G][PATCH 0/1] s390x/pci: do not allow to create more pci functions than configured via CONFIG_PCI_NR_FUNCTIONS (LP: 1874057)
frank.heimes at canonical.com
frank.heimes at canonical.com
Mon Apr 27 18:55:53 UTC 2020
Buglink: https://bugs.launchpad.net/bugs/1874057
SRU Justification:
[Impact]
* PCI Functions with UIDs >128 are currently not accounted correctly in the s390x/pci (zPCI) code.
* Furthermore, the code allows that more than CONFIG_PCI_NR_FUNCTIONS are created.
* This can lead to issues with data structures which were only allocated for CONFIG_PCI_NR_FUNCTIONS.
[Fix]
* https://bugs.launchpad.net/ubuntu-z-systems/+bug/1874057/+attachment/5357850/+files/0001-s390-pci-Fix-zpci_alloc_domain-over-allocation-for-focal.patch
[Test Case]
* Set the kernel parameter CONFIG_PCI_NR_FUNCTIONS to a specific (reasonablly low) number.
* And check if more PCI functions can be created than specified by CONFIG_PCI_NR_FUNCTIONS (e.g. using a RoCE adapter) and watch for kernel message 'Adding PCI function ... failed'
[Regression Potential]
* There is regression potential can be considered as low, since:
* the zPCI cards are less wide spread than for example ccw adapters on s390x
* the fix got already upstream accepted in 5.7, hence upstream reviewed, too
* the modifications span just two files and both are s390x arch specific
[Other Info]
* the above patch-file is based on commit 969ae01bab2fe938b4c8324836038b5ac1c78fac ("s390/pci: Fix zpci_alloc_domain() over allocation"), but this backport was needed for getting this applied to focal master-next
* and this patch got upstream accepted with kernel v5.7-rc1, hence on the long term it should be in 'gorilla'
Niklas Schnelle (1):
s390/pci: Fix zpci_alloc_domain() over allocation
arch/s390/include/asm/pci.h | 1 +
arch/s390/pci/pci.c | 40 +++++++++++++++++++++++++------------
2 files changed, 28 insertions(+), 13 deletions(-)
--
2.25.1
More information about the kernel-team
mailing list