Re: [PATCH 5/4] update ksm userspace interfaces

Chris Wright
2009-04-02T15:21:56Z

* Izik Eidus (ieidus@redhat.com) wrote:
> So if we move into madvice and we remove the get_task_mm() usage, we
> will have to add notification to exit_mm() so ksm will know it should
> stop using this mm strcture, and drop it from all the trees data...

Technically it's needed already. This example is currently semi-broken:

main()
ksm_register_memory
execve() <-- no notifiction unless fd is proactively marked cloexec
(which it isn't)

new proc...do stuff (it's ->mm isn't registered)
eventually exit() <-- close fd and clear up the old stale ->mm registered