Date: Sun, 23 Jan 2022 19:24:36 +0100 From: Wojciech Macek <wma@semihalf.com> To: Gleb Smirnoff <glebius@freebsd.org> Cc: Kristof Provost <kp@freebsd.org>, Wojciech Macek <wma@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-main@freebsd.org Subject: Re: git: 9ce46cbc95d7 - main - ip_mroute: move ip_mrouter_done outside lock Message-ID: <CANsEV8cEWc9GsTZk%2BVm-_%2B9rsU-=yXLjRkZZz--dg7-ZJRgOnw@mail.gmail.com> In-Reply-To: <Ye2FOvi32Xu23qIB@FreeBSD.org> References: <202201210519.20L5J1mJ029323@gitrepo.freebsd.org> <7A991003-0F24-4C8C-81FD-84F39FC78D8A@FreeBSD.org> <C25EE57B-BE5A-4BC8-AD00-C8D0D582D073@FreeBSD.org> <CANsEV8cmZcQKh3U8w2S37cg9heFd=icuEmF7dVwrE486OM3Yug@mail.gmail.com> <Ye2FOvi32Xu23qIB@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000095288505d643f733 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable This is an orginal fix. The issue was visible due to race between ip input/output and mrouter uninit. https://reviews.freebsd.org/D29946 niedz., 23 sty 2022, 17:41 u=C5=BCytkownik Gleb Smirnoff <glebius@freebsd.o= rg> napisa=C5=82: > On Sat, Jan 22, 2022 at 08:50:54AM +0100, Wojciech Macek wrote: > W> I will prepare a fix later today. Basically, the whole "if" condition > must > W> be inside rwlock block to avoid race during module unload. > W> Maybe adding else and rwunlock in rip_detach will suffice.... I need t= o > try > W> it out. > > May I ask you to post a review and add me as reviewer? Please also includ= e > a description of the problem we are fighting with. At least, a short > version > of it, for example "fix race between module unload and packet received". > > -- > Gleb Smirnoff > --00000000000095288505d643f733 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto">This is an orginal fix. The issue was visible due to race= between ip input/output and mrouter uninit.=C2=A0<div dir=3D"auto"><a href= =3D"https://reviews.freebsd.org/D29946">https://reviews.freebsd.org/D29946<= /a><br></div></div><br><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D= "gmail_attr">niedz., 23 sty 2022, 17:41 u=C5=BCytkownik Gleb Smirnoff <<= a href=3D"mailto:glebius@freebsd.org">glebius@freebsd.org</a>> napisa=C5= =82:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;= border-left:1px #ccc solid;padding-left:1ex">On Sat, Jan 22, 2022 at 08:50:= 54AM +0100, Wojciech Macek wrote:<br> W> I will prepare a fix later today. Basically, the whole "if"= condition must<br> W> be inside rwlock block to avoid race during module unload.<br> W> Maybe adding else and rwunlock in rip_detach will suffice.... I need = to try<br> W> it out.<br> <br> May I ask you to post a review and add me as reviewer? Please also include<= br> a description of the problem we are fighting with. At least, a short versio= n<br> of it, for example "fix race between module unload and packet received= ".<br> <br> -- <br> Gleb Smirnoff<br> </blockquote></div> --00000000000095288505d643f733--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANsEV8cEWc9GsTZk%2BVm-_%2B9rsU-=yXLjRkZZz--dg7-ZJRgOnw>