Comment 10 for bug 1869948

Revision history for this message
Guilherme G. Piccoli (gpiccoli) wrote :

I've verified this LP using AWS instances, so the kernel versions I tested were -aws flavored; this was necessary given the patch proposed here is related to the ena driver, which manages an AWS exclusive virtual NIC.

The versions tested were:

4.4.0.1105 xenial-updates
4.4.0.1106 xenial-proposed
4.15.0.1065 bionic-updates
4.15.0.1066 bionic-proposed
5.3.0.1016 eoan-updates (tested with Bionic-HWE analog)
5.3.0.1017 eoan-proposed (tested with Bionic-HWE analog)

The test was based in the [Test] section / comment #3, and I've managed to reproduce the issue in all versions on -updates, whereas no version in -proposed showed the issue (20 kexecs succeeded). In the failure case, in the 2nd or at most 3rd kexec, we've noticed the crash on boot, initrd corruption. In Xenial (kernel 4.4), due to the "small" size of initrd, it was needed to install linux-modules-extra to increase the size of the file and hence expose the memory corruption.

Also, I've checked in all kernels if the symbols added by the patch were there, with the following command:

# grep "ena_remov\|ena_shut" /proc/kallsyms
ffffffffc0005690 t __ena_shutoff [ena]
ffffffffc0005770 t ena_shutdown [ena]
ffffffffc0005790 t ena_remove [ena]

The above output is from a -proposed kernel; kernels in -updates only show ena_remove() symbol. Finally, I checked the patch in the generic flavors trees, for X/B/E/F, and they are present in the latest tag (corresponding to kernels in -proposed and tag Ubuntu-5.4.0-22.26 for Focal).

So, I'm hereby marking this LP as verified for all releases. See next comment for a note about Disco kernel.
Thanks,

Guilherme