------- Comment on attachment From [email protected] 2020-05-04 09:52
EDT-------
Oh thank you for your patience, I totally screwed up the backport patch
as I accidentally included some code from the next commit
7a11c67a1ff9b0231eaaaa6a28294776d55b569a ("s390/pci: Improve handling of
unset UID")
Actually that now applies cleanly on top and I think we might
want to include that as well, escpecially since it will also simplify
the backporting of the multi-function patch series.
** Attachment added: "s390/pci: Fix zpci_alloc_domain() over allocation (for
focal 2)"
https://bugs.launchpad.net/bugs/1874057/+attachment/5366222/+files/0001-s390-pci-Fix-zpci_alloc_domain-over-allocation.patch
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1874057
Title:
[UBUNTU 20.04] s390x/pci: do not allow to create more pci functions
than configured via CONFIG_PCI_NR_FUNCTIONS
Status in Ubuntu on IBM z Systems:
In Progress
Status in linux package in Ubuntu:
In Progress
Bug description:
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'
__________
PCI Functions with UIDs >128 are currently not accounted correctly in
the s390x/pci 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.
This has been fixed in the following upstream commit:
969ae01bab2fe938b4c8324836038b5ac1c78fac
("s390/pci: Fix zpci_alloc_domain() over allocation")
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-z-systems/+bug/1874057/+subscriptions
--
Mailing list: https://launchpad.net/~kernel-packages
Post to : [email protected]
Unsubscribe : https://launchpad.net/~kernel-packages
More help : https://help.launchpad.net/ListHelp
