Date: Tue, 14 Dec 2021 22:42:49 +0100 From: Kristof Provost <kp@FreeBSD.org> To: Gleb Smirnoff <glebius@FreeBSD.org> Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: eb93b99d6986 - main - in_pcb: delay crfree() down into UMA dtor Message-ID: <28AE53F1-2B22-444B-B1EC-1600FA741FE2@FreeBSD.org> In-Reply-To: <202112051847.1B5Il2GP030287@gitrepo.freebsd.org> References: <202112051847.1B5Il2GP030287@gitrepo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--=_MailMate_08E64701-C3C6-422E-84B5-8215067F1362_= Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable On 5 Dec 2021, at 19:47, Gleb Smirnoff wrote: > The branch main has been updated by glebius: > > URL: = > https://cgit.FreeBSD.org/src/commit/?id=3Deb93b99d698674e3b1cc7139fda98= e2b175b8c5b > > commit eb93b99d698674e3b1cc7139fda98e2b175b8c5b > Author: Gleb Smirnoff <glebius@FreeBSD.org> > AuthorDate: 2021-12-05 16:47:24 +0000 > Commit: Gleb Smirnoff <glebius@FreeBSD.org> > CommitDate: 2021-12-05 18:46:37 +0000 > > in_pcb: delay crfree() down into UMA dtor > > inpcb lookups, which check inp_cred, work with pcbs that = > potentially went > through in_pcbfree(). So inp_cred should stay valid until SMR = > guarantees > its invisibility to lookups. > > While here, put the whole inpcb destruction sequence of = > in_pcbfree(), > inpcb_dtor() and inpcb_fini() sequentially. > > Submitted by: markj > Differential revision: https://reviews.freebsd.org/D33273 > For some reason it looks like this commit causes jails to fail to get = fully cleaned up. I can reproduce that trivially with `cd /usr/tests/sys/net ; kyua test = if_bridge_test:bridge_transmit_ipv4_unicast ; jls -na`. Note the jails in dying state. The jails created by that test never go away. It=E2=80=99s as if = `crfree(inp->inp_cred);` doesn=E2=80=99t actually get called. And indeed,= it = looks like inpcb_dtor() does not get called at all. Does Kristof --=_MailMate_08E64701-C3C6-422E-84B5-8215067F1362_=--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?28AE53F1-2B22-444B-B1EC-1600FA741FE2>