With the patch applied the compile complains with:
...
  CC      mm/vmalloc.o
^[[01m^[[K/home/ubuntu/focal-lp1874057/arch/s390/pci/pci.c:^[[m^[[K In function 
‘^[[01m^[[Kzpci_alloc_domain^[[m^[[K’:
^[[01m^[[K/home/ubuntu/focal-lp1874057/arch/s390/pci/pci.c:668:4:^[[m^[[K 
^[[01;31m^[[Kerror: ^[[m^[[Kimplicit declaration of function 
‘^[[01m^[[Kupdate_uid_checking^[[m^[[K’ 
[^[[01;31m^[[K-Werror=implicit-function-declaration^[[m^[[K]
  668 |    ^[[01;31m^[[Kupdate_uid_checking^[[m^[[K(false);
      |    ^[[01;31m^[[K^~~~~~~~~~~~~~~~~~~^[[m^[[K
  CC      mm/process_vm_access.o
  CC      security/apparmor/lib.o
cc1: some warnings being treated as errors
make[4]: *** [/home/ubuntu/focal-lp1874057/scripts/Makefile.build:273: 
arch/s390/pci/pci.o] Error 1
make[3]: *** [/home/ubuntu/focal-lp1874057/scripts/Makefile.build:522: 
arch/s390/pci] Error 2
make[2]: *** [/home/ubuntu/focal-lp1874057/Makefile:1719: arch/s390] Error 2

I double-checked with compiling the focal master-next tree w/o the
patch, which works fine.

-- 
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     : kernel-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kernel-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to