Re: [PATCH 1/1] mm: introduce process_reap system call
From: Christian Brauner <hidden>
Date: 2021-07-09 08:59:06
Also in:
linux-mm, lkml
On Wed, Jul 07, 2021 at 02:14:23PM -0700, Suren Baghdasaryan wrote:
On Wed, Jul 7, 2021 at 5:38 AM Michal Hocko [off-list ref] wrote:quoted
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
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.quoted
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.
I think a syscall is fine and it's not we're running out of numbers (anymore). :) Christian