<div dir="ltr"><div>Oops -- that should be Unstable, as requested by Cascardo. I realized later than U could also refer to the old Utopic.</div><div>Sorry for the confusion.<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 12, 2018 at 3:07 AM Khaled Elmously <<a href="mailto:khalid.elmously@canonical.com">khalid.elmously@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm not sure what is meant by [U] in the subject - what release is this intended for?<br>
<br>
<br>
On 2018-11-08 19:07:18 , Mauricio Faria de Oliveira wrote:<br>
> BugLink: <a href="https://bugs.launchpad.net/bugs/1797990" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/1797990</a><br>
> <br>
> [Changelog]<br>
>  * v2:<br>
>    - Reorder patch 1 as 3 to allow for the next change:<br>
>    - Gate the bus-scan differences with the cmdline option<br>
>      (patch 3 only).  Now all functional changes are gated.<br>
> <br>
> [Impact]<br>
> <br>
>  * A kexec/crash kernel might get stuck and fail to boot<br>
>    (for crash kernel, kdump fails to collect a crashdump)<br>
>    if a PCI device is buggy/stuck/looping and triggers a<br>
>    continuous flood of MSI(X) interrupts (that the kernel<br>
>    does not yet know about).<br>
> <br>
>  * This fix allowed to obtain crashdumps when debugging a<br>
>    heavy-load scenario, in which a (heavy-loaded) network<br>
>    adapter wouldn't stop triggering MSI-X interrupts ever<br>
>    after panic()->kdump kicked in.<br>
> <br>
>  * This fix disables MSI(X) in all PCI devices on early<br>
>    boot (this is OK as it's (re-)enabled normally later)<br>
>    with a kernel cmdline parameter (disabled by default).<br>
> <br>
> [Test Case]<br>
> <br>
>  * A synthetic test-case is not yet available, however,<br>
>    this particular system/workload triggered the problem<br>
>    consistently, and it was used for development/testing.<br>
> <br>
>  * We'll update this bug once a synthetic test-case is<br>
>    available; we're working on patching QEMU for this.<br>
> <br>
>  * $ dmesg | grep 'Clearing MSI'<br>
>    [    0.000000] Clearing MSI/MSI-X enable bits early in boot (quirk)<br>
> <br>
>  * The comparison of 'dmesg -t | sort' has been reviewed<br>
>    between option disabled/enabled on boot & kexec modes,<br>
>    and only expected differences found (MHz, PIDs, MIPS).<br>
> <br>
> [Regression Potential] <br>
> <br>
>  * The potential area for regressions is early boot,<br>
>    particularly effects of applying quirks during PCI<br>
>    bus scan, which is changed/broader w/ these patches.<br>
> <br>
>  * However, all quirks are applied based on PCI ID<br>
>    matching, so would only apply if actually targeting<br>
>    a new device.<br>
> <br>
>  * Moreover, the new quirk is only applied based on<br>
>    a kernel cmdline parameter that is disabled by<br>
>    default, which constraints even more when this<br>
>    is actually in effect.<br>
> <br>
> [Other Info]<br>
>  <br>
>  * The patch series is still under review/discussion<br>
>    upstream, but it's relatively important for Ubuntu<br>
>    users at this point, and after internal discussions<br>
>    we decided to submit it for SRU.<br>
> <br>
>  * These are links to the linux-pci archive with the<br>
>    patches [1, 2, 3]<br>
> <br>
>    [1] [PATCH 1/3] x86/quirks: Scan all busses for early PCI quirks<br>
>        <a href="https://lore.kernel.org/linux-pci/20181018183721.27467-1-gpiccoli@canonical.com/" rel="noreferrer" target="_blank">https://lore.kernel.org/linux-pci/20181018183721.27467-1-gpiccoli@canonical.com/</a><br>
> <br>
>    [2] [PATCH 2/3] x86/PCI: Export find_cap() to be used in early PCI code<br>
>        <a href="https://lore.kernel.org/linux-pci/20181018183721.27467-2-gpiccoli@canonical.com/" rel="noreferrer" target="_blank">https://lore.kernel.org/linux-pci/20181018183721.27467-2-gpiccoli@canonical.com/</a><br>
> <br>
>    [3] [PATCH 3/3] x86/quirks: Add parameter to clear MSIs early on boot<br>
>        <a href="https://lore.kernel.org/linux-pci/20181018183721.27467-3-gpiccoli@canonical.com/" rel="noreferrer" target="_blank">https://lore.kernel.org/linux-pci/20181018183721.27467-3-gpiccoli@canonical.com/</a><br>
> <br>
> Guilherme G. Piccoli (3):<br>
>   UBUNTU: SAUCE: x86/PCI: Export find_cap() to be used in early PCI code<br>
>   UBUNTU: SAUCE: x86/quirks: Add parameter to clear MSIs early on boot<br>
>   UBUNTU: SAUCE: x86/quirks: Scan all busses for early PCI quirks<br>
> <br>
>  .../admin-guide/kernel-parameters.txt         |  6 +++<br>
>  arch/x86/include/asm/pci-direct.h             |  2 +<br>
>  arch/x86/kernel/aperture_64.c                 | 30 +-------------<br>
>  arch/x86/kernel/early-quirks.c                | 41 +++++++++++++++++++<br>
>  arch/x86/pci/common.c                         |  4 ++<br>
>  arch/x86/pci/early.c                          | 25 +++++++++++<br>
>  6 files changed, 80 insertions(+), 28 deletions(-)<br>
> <br>
> -- <br>
> 2.17.1<br>
> <br>
> <br>
> -- <br>
> kernel-team mailing list<br>
> <a href="mailto:kernel-team@lists.ubuntu.com" target="_blank">kernel-team@lists.ubuntu.com</a><br>
> <a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span><div><div dir="ltr"><div><div dir="ltr"><div>Mauricio Faria de Oliveira</div></div></div></div></div></span></div></div>