On Fri 11-08-17 00:09:57, Colm MacCárthaigh wrote: > On Thu, Aug 10, 2017 at 7:01 PM, Michal Hocko <mhocko@xxxxxxxxxx> wrote: > > Does anybody actually do that using the minherit BSD interface? > > I can't find any OSS examples. I just thought of it in response to > your question, but now that I have, I do want to use it that way in > privsep code. > > As a mere user, fwiw it would make /my/ code less complex (in > Kolmogorov terms) to be an madvise option. Here's what that would look > like in user space: > > mmap() > > #if MAP_INHERIT_ZERO > minherit() || pthread_atfork(workaround_fptr); > #elif MADVISE_WIPEONFORK > madvise() || pthread_atfork(workaround_fptr); > #else > pthread_atfork(workaround_fptr); > #endif > > Vs: > > #if MAP_WIPEONFORK > mmap( ... WIPEONFORK) || pthread_atfork(workaround_fptr); > #else > mmap() > #endif > > #if MAP_INHERIT_ZERO > madvise() || pthread_atfork(workaround_fptr); > #endif > > #if !defined(MAP_WIPEONFORK) && !defined(MAP_INHERIT_ZERO) > pthread_atfork(workaround_fptr); > #endif > > The former is neater, and also a lot easier to stay structured if the > code is separated across different functional units. Allocation is > often handled in special functions. OK, I guess I see your point. Thanks for the clarification. > For me, madvise() is the principle of least surprise, following > existing DONTDUMP semantics. I am sorry to look too insisting here (I have still hard time to reconcile myself with the madvise (ab)use) but if we in fact want minherit like interface why don't we simply add minherit and make the code which wants to use that interface easier to port? Is the only reason that hooking into madvise is less code? If yes is that a sufficient reason to justify the (ab)use of madvise? If there is a general consensus on that part I will shut up and won't object anymore. Arguably MADV_DONTFORK would fit into minherit API better as well. MADV_DONTDUMP is a differnet storry of course. -- Michal Hocko SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
