<div dir="ltr"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi Seth,</div><div>yes, I've added a comment to the justification that it got upstream accepted wit 5.7-rc1 (that's usually the 1st thing I check).</div><div>With that it should automatically land (at some point in time) in 'gorilla',</div><div>but since gorilla's kernel is as of today still based on the focal, I've added 'gorilla' to this request, too. But feel free to drop it if not needed ...</div><div><br></div><div>And apologize that I forgot the provenance - using 'git cherry-pick -x' indeed largely adds it automatically,<br>but in this case I needed to use 'git am' since a backport was needed, because the upstream commit didn't applied cleanly while trying to cherry-pick.</div><div>(And I think there is no 'git am -x' ...)</div><div><br></div><div>Anyway, I'll submit a v2 shortly with proper provenance of the backport ...</div><div><br></div><div>Thx for your feedback ... bye, Frank</div></div></div></div></div></div></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 30, 2020 at 11:47 PM Seth Forshee <<a href="mailto:seth.forshee@canonical.com">seth.forshee@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Apr 27, 2020 at 08:55:53PM +0200, <a href="mailto:frank.heimes@canonical.com" target="_blank">frank.heimes@canonical.com</a> wrote:<br>
> Buglink: <a href="https://bugs.launchpad.net/bugs/1874057" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/1874057</a><br>
> <br>
> SRU Justification:<br>
> <br>
> [Impact]<br>
> <br>
> * PCI Functions with UIDs >128 are currently not accounted correctly in the s390x/pci (zPCI) code.<br>
> <br>
> * Furthermore, the code allows that more than CONFIG_PCI_NR_FUNCTIONS are created.<br>
> <br>
> * This can lead to issues with data structures which were only allocated for CONFIG_PCI_NR_FUNCTIONS.<br>
> <br>
> [Fix]<br>
> <br>
> * <a href="https://bugs.launchpad.net/ubuntu-z-systems/+bug/1874057/+attachment/5357850/+files/0001-s390-pci-Fix-zpci_alloc_domain-over-allocation-for-focal.patch" rel="noreferrer" target="_blank">https://bugs.launchpad.net/ubuntu-z-systems/+bug/1874057/+attachment/5357850/+files/0001-s390-pci-Fix-zpci_alloc_domain-over-allocation-for-focal.patch</a><br>
> <br>
> [Test Case]<br>
> <br>
> * Set the kernel parameter CONFIG_PCI_NR_FUNCTIONS to a specific (reasonablly low) number.<br>
> <br>
> * 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'<br>
> <br>
> <br>
> [Regression Potential] <br>
> <br>
> * There is regression potential can be considered as low, since:<br>
> <br>
> * the zPCI cards are less wide spread than for example ccw adapters on s390x<br>
> <br>
> * the fix got already upstream accepted in 5.7, hence upstream reviewed, too<br>
> <br>
> * the modifications span just two files and both are s390x arch specific<br>
> <br>
> [Other Info]<br>
> <br>
> * 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<br>
> <br>
> * and this patch got upstream accepted with kernel v5.7-rc1, hence on the long term it should be in 'gorilla'<br>
<br>
This patch is from 5.7-rc1, but it lacks the upstream provenance. Please<br>
use 'git cherry-pick -x' to include this information.<br>
<br>
This patch is not required for unstable, which is already based on<br>
5.7-rc.<br>
<br>
Thanks,<br>
Seth<br>
</blockquote></div>