Re: [PATCH 1/1] mm: introduce process_reap system call
From: Suren Baghdasaryan <surenb@google.com>
Date: 2021-07-07 21:14:37
Also in:
linux-mm, lkml
From: Suren Baghdasaryan <surenb@google.com>
Date: 2021-07-07 21:14:37
Also in:
linux-mm, lkml
On Wed, Jul 7, 2021 at 5:38 AM Michal Hocko [off-list ref] wrote:
On Mon 05-07-21 09:41:54, David Hildenbrand wrote:quoted
On 02.07.21 17:27, Christian Brauner wrote:[...]quoted
quoted
That one was my favorite from the list I gave too but maybe we can satisfy Andy too if we use one of: - process_mfree() - process_mrelease()FWIW, I tend to like process_mrelease(), due to the implied "release" ("free the memory if there are no other references") semantics.Agreed.
Ok, sounds like process_mrelease() would be an acceptable compromise.
quoted
Further, a new syscall feels cleaner than some magic sysfs/procfs toggle. Just my 2 cents.Yeah, proc based interface is both tricky to use and kinda ugly now that pidfd can solve all at in once.
Sounds good. Will keep it as is then.
My original preference was a more generic kill syscall to allow flags but a dedicated syscall doesn't look really bad either.
Yeah, I have tried that direction unsuccessfully before arriving at this one. Hopefully it represents the right compromise which can satisfy everyone's usecase.
-- Michal Hocko SUSE Labs