ACK/cmnt: [SRU][Xenial][Yakkety][Zesty][Artful][PATCH 0/1] UBUNTU: SAUCE: pci-hyperv: Use only 16 bit integer for PCI domain

[Joshua R. Poulson]

Spanking received, but these three patches that affect totally
different areas of the Hyper-V device driver is at least a two beer
story some day.

On Mon, Apr 24, 2017 at 10:46 PM, Stefan Bader
<stefan.bader at canonical.com> wrote:
> On 25.04.2017 01:38, Joseph Salisbury wrote:
>> BugLink: http://bugs.launchpad.net/bugs/1684971
>>
>> == SRU Justification ==
>> This patch fixes a problem introduced by mainline commit 4a9b0933bdfc.
>> Some drivers were expecting a u16 instead of
>> a u32 for PCI device serial numbers, as observed by Oops and hangs in
>> Azure on NC and NV GPU instances.
>
> That is the 3rd time hyperv code needs urgent fixes and at least the 2nd time
> this is caused by broken upstream submissions. Maybe someone should tell those
> guys about the benefits of testing...
>
> Otherwise looks to be doing what it claims.
>
> -Stefan
>>
>> This patch has been submitted upstream, but is not in mainline or -next
>> yet, which is why I'm submitting it as SAUCE.
>>
>> Upstream thread:
>> https://lkml.org/lkml/2017/4/20/729
>>
>> == Fix ==
>> Author: Haiyang Zhang <haiyangz at microsoft.com>
>> Date:   Thu Apr 20 13:54:14 2017 -0400
>>
>>       pci-hyperv: Use only 16 bit integer for PCI domain
>>
>> == Test Case ==
>> A test kernel was built with this patch and tested by the original bug reporter.
>> The bug reporter states the test kernel resolved the bug.
>>
>>
>>
>> Haiyang Zhang (1):
>>   This patch uses the lower 16 bits of the serial number as PCI
>>     domain, otherwise some drivers may not be able to handle it.
>>
>>  drivers/pci/host/pci-hyperv.c | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>
>
>
> --
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>