<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:PMingLiU;
panose-1:2 2 5 0 0 0 0 0 0 0;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Aptos;
panose-1:2 11 0 4 2 2 2 2 2 4;}
@font-face
{font-family:"\@PMingLiU";
panose-1:2 1 6 1 0 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:12.0pt;
font-family:"Aptos",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Aptos",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
mso-ligatures:none;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style></head><body lang=EN-US link=blue vlink=purple style='word-wrap:break-word'><div class=WordSection1><p class=MsoNormal><span style='font-size:11.0pt'>Thanks, will follow the right cover-letter format next time!<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt'>William<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt'><o:p> </o:p></span></p><p class=MsoNormal style='margin-bottom:12.0pt'>On 4/3/24, 8:36 AM, "Andrei Gherzan" <andrei.gherzan@canonical.com> wrote:<o:p></o:p></p><div><p class=MsoNormal>On 24/04/01 11:40AM, William Tu wrote:<o:p></o:p></p></div><div><p class=MsoNormal>> intro<o:p></o:p></p></div><div><p class=MsoNormal>> -----<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> Our internal test triggers a kernel crash dump below<o:p></o:p></p></div><div><p class=MsoNormal>> [ 888.690348] Sun Mar 24 23:51:59 2024: DriVerTest - Start Test<o:p></o:p></p></div><div><p class=MsoNormal>> [ 888.691834] ----------------------------------------------------------------------------------------------------<o:p></o:p></p></div><div><p class=MsoNormal>> [ 888.983912] mlx5_core 0000:08:00.1<o:p></o:p></p></div><div><p class=MsoNormal>> eth3: Link up<o:p></o:p></p></div><div><p class=MsoNormal>> [ 888.987644] IPv6: ADDRCONF(NETDEV_CHANGE): eth3: link becomes ready<o:p></o:p></p></div><div><p class=MsoNormal>> [ 889.336577] mlx5_core 0000:08:00.0 eth2: Link up<o:p></o:p></p></div><div><p class=MsoNormal>> [ 894.635836] Sun Mar 24 11:52:04 PM IST 2024 - DriVerTest Debug Heartbeat<o:p></o:p></p></div><div><p class=MsoNormal>> [ 940.431644] general protection fault, probably for non-canonical address 0x8002001400000000: 0000 [#1] SMP NOPTI<o:p></o:p></p></div><div><p class=MsoNormal>> [ 940.432866] CPU: 7 PID: 94305 Comm: ethtool Tainted: G OE 5.15.0-1039.17.g0d63875-bluefield #1<o:p></o:p></p></div><div><p class=MsoNormal>> [ 940.433970] Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS rel-1.13.0-0-gf21b5a4aeb02-prebuilt.qemu.org 04/01/2014<o:p></o:p></p></div><div><p class=MsoNormal>> [ 940.435220] RIP: 0010:netlink_policy_dump_add_policy+0x95/0x160<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> fix<o:p></o:p></p></div><div><p class=MsoNormal>> ---<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> Need to cherry-pick the following patch<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> commit c1b05105573b2cd5845921eb0d2caa26e2144a34<o:p></o:p></p></div><div><p class=MsoNormal>> Author: Jakub Kicinski <<a href="mailto:kuba@kernel.org">kuba@kernel.org</a>><o:p></o:p></p></div><div><p class=MsoNormal>> Date: Wed Nov 9 10:32:54 2022 -0800<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> genetlink: fix single op policy dump when do is present<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> Jonathan reports crashes when running net-next in Meta's fleet.<o:p></o:p></p></div><div><p class=MsoNormal>> Stats collection uses ethtool -I which does a per-op policy dump<o:p></o:p></p></div><div><p class=MsoNormal>> to check if stats are supported. We don't initialize the dumpit<o:p></o:p></p></div><div><p class=MsoNormal>> information if doit succeeds due to evaluation short-circuiting.<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> The crash may look like this:<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> BUG: kernel NULL pointer dereference, address: 0000000000000cc0<o:p></o:p></p></div><div><p class=MsoNormal>> RIP: 0010:netlink_policy_dump_add_policy+0x174/0x2a0<o:p></o:p></p></div><div><p class=MsoNormal>> ctrl_dumppolicy_start+0x19f/0x2f0<o:p></o:p></p></div><div><p class=MsoNormal>> genl_start+0xe7/0x140<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> Jakub Kicinski (1):<o:p></o:p></p></div><div><p class=MsoNormal>> genetlink: fix single op policy dump when do is present<o:p></o:p></p></div><div><p class=MsoNormal>> <o:p></o:p></p></div><div><p class=MsoNormal>> net/netlink/genetlink.c | 30 +++++++++++++++++++++---------<o:p></o:p></p></div><div><p class=MsoNormal>> 1 file changed, 21 insertions(+), 9 deletions(-)<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>There is a format/template that the cover letter should have. See<o:p></o:p></p></div><div><p class=MsoNormal><a href="https://wiki.ubuntu.com/Kernel/Dev/StablePatchFormat">https://wiki.ubuntu.com/Kernel/Dev/StablePatchFormat</a> for more info. Even<o:p></o:p></p></div><div><p class=MsoNormal>so, the submission looks good, so I am happy to ack it:<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Acked-by: Andrei Gherzan <<a href="mailto:andrei.gherzan@canonical.com">andrei.gherzan@canonical.com</a>><o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>-- <o:p></o:p></p></div><div><p class=MsoNormal>Andrei Gherzan<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div></div></body></html>