ACK/cmnt: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073

[Stefan Bader]

On 08.09.2017 15:38, Joseph Salisbury wrote:
> BugLink: http://bugs.launchpad.net/bugs/1715073
> 
> == SRU Justification ==
> IBM reports that When doing SRIOV if they unload VFs they will see a warning
> that is coming from the fix for bugzilla: Bug 146479 (LP1625318).
> 
> Commit 5b0948dfe is a prereq commit for Zesty, but not for Artful, since that commit 
> was included in mainline as of 4.11-rc1.
> 
> Commit 0fc690a7c3f is not in mainline as of yet, and was a clean pick from linux-next.
> Commit 0fc690a7c3f is also need in Artful, but will be SRU'd seperatly, since Z needs
>  a prereq commit.
> 
> == Fixes ==
> commit 5b0948dfe138f0837699f46f5877f4f81c252dac
> Author: Emil Tantilov <emil.s.tantilov at intel.com>
> Date:   Fri Jan 6 13:59:08 2017 -0800
> 
>     PCI: Lock each enable/disable num_vfs operation in sysfs
> 
> commit 0fc690a7c3f7053613dcbab6a7613bb6586d8ee2
> Author: Gavin Shan <gwshan at linux.vnet.ibm.com>
> Date:   Fri Aug 11 18:19:33 2017 +1000
>     PCI: Disable VF decoding before pcibios_sriov_disable() updates resources
> 
> 
> 
> == Regression Potential ==
> The primary patch is only adding a call to pcibios_sriov_disable(), which 
> disables VF memory decoding before calling pcibios_sriov_disable().  No other major
> changes are made.
> 
> == Test Case ==
> A test kernel was built with these patches and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
> 
> 
> Emil Tantilov (1):
>   PCI: Lock each enable/disable num_vfs operation in sysfs
> 
> Gavin Shan (1):
>   PCI: Disable VF decoding before pcibios_sriov_disable() updates
>     resources
> 
>  drivers/pci/iov.c       | 14 ++++----------
>  drivers/pci/pci-sysfs.c | 23 ++++++++++++++++-------
>  drivers/pci/pci.h       |  2 +-
>  3 files changed, 21 insertions(+), 18 deletions(-)
> 
Acked-by: Stefan Bader <stefan.bader at canonical.com>

We can add the bug link to patch #2 when committing (can use this hopefully as
reminder).

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20170911/0318d25f/attachment.sig>