<div dir="ltr"><div>Applied to kinetic:linux master-next</div><div><br></div><div>Thanks!</div><div>- Luke<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 7, 2023 at 9:04 PM Po-Hsu Lin <<a href="mailto:po-hsu.lin@canonical.com">po-hsu.lin@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">[Impact]<br>
The arp_ndisc_evict_nocarrier.sh in selftests/net didn't pass a proper<br>
return value for failed test cases. Thus we might get false negatives<br>
if the last command runs successfully.<br>
<br>
[Fix]<br>
* 1856628baa selftests: net: return non-zero for failures reported in<br>
arp_ndisc_evict_nocarrier<br>
<br>
This patch can be cherry-picked into affected kernels. Lunar has been<br>
patched already. J-hwe-5.17 will be deprecated thus it will be skipped.<br>
<br>
[Test]<br>
Run the patched test, this should not cause any regression as sub<br>
tests are good with our kernels (except there is a cleanup issue in<br>
bug 1968310)<br>
<br>
One can deliberately fail a test by hijacking the if statement for the<br>
return value check to make sure this patch is working as expected.<br>
<br>
[Where problems could occur]<br>
This patch improves the result interpretation, we might see new<br>
failures being reported.<br>
<br>
Po-Hsu Lin (1):<br>
  selftests: net: return non-zero for failures reported in<br>
    arp_ndisc_evict_nocarrier<br>
<br>
 tools/testing/selftests/net/arp_ndisc_evict_nocarrier.sh | 11 +++++++++--<br>
 1 file changed, 9 insertions(+), 2 deletions(-)<br>
<br>
-- <br>
2.7.4<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></div>