Date: Sat, 6 May 2023 20:56:40 +0300 From: Konstantin Belousov <kostikbel@gmail.com> To: Dag-Erling =?utf-8?B?U23DuHJncmF2?= <des@des.no> Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 5807f35c541c - main - dd: Fix SIGINT handling. Message-ID: <ZFaU2F0NRdRlXsYz@kib.kiev.ua> In-Reply-To: <86a5yhy5oe.fsf@ltc.des.no> References: <202305051245.345CjNLE048275@gitrepo.freebsd.org> <ZFVJMIKu4v4OnPZL@kib.kiev.ua> <86a5yhy5oe.fsf@ltc.des.no>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, May 06, 2023 at 03:01:53PM +0200, Dag-Erling Smørgrav wrote: > Konstantin Belousov <kostikbel@gmail.com> writes: > > This does not fix SIGINT handler, it only (significantly) reduces the window > > where the issue could occur. > > Yes, it does. I think you fundamentally misunderstand the intent of the > patch. I know there is a tiny race but it's still a vast improvement on > the original. > > > See my latest comment on the review. > > See my reply. I do not see a reply to my last comment in the review.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ZFaU2F0NRdRlXsYz>