Messages in this thread | | | From | Suren Baghdasaryan <> | Date | Wed, 7 Jul 2021 14:14:23 -0700 | Subject | Re: [PATCH 1/1] mm: introduce process_reap system call |
| |
On Wed, Jul 7, 2021 at 5:38 AM Michal Hocko <mhocko@suse.com> wrote: > > On Mon 05-07-21 09:41:54, David Hildenbrand wrote: > > On 02.07.21 17:27, Christian Brauner wrote: > [...] > > > 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.
> > > 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
| |