Messages in this thread | | | Date | Wed, 24 May 2023 16:10:23 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH 3/3] fork, vhost: Use CLONE_THREAD to fix freezer/ps regression |
| |
On 05/23, Eric W. Biederman wrote: > > I want to point out that we need to consider not just SIGKILL, but > SIGABRT that causes a coredump, as well as the process peforming > an ordinary exit(2). All of which will cause get_signal to return > SIGKILL in this context.
Yes, but probably SIGABRT/exit doesn't really differ from SIGKILL wrt vhost_worker().
> It is probably not the worst thing in the world, but what this means > is now if you pass a copy of the vhost file descriptor to another > process the vhost_worker will persist, and thus the process will persist > until that copy of the file descriptor is closed.
Hadn't thought about it.
I am fighting with internal bugzillas today, will try to write another email tomorrow.
But before that, I would like to have an answer to my "main" question in my previois email. Otherwise I am still not sure I understand what exactly we need to fix.
Oleg.
| |