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

From: Chris Wright
Date: Thu Apr 02 2009 - 11:21:57 EST


* Izik Eidus (ieidus@xxxxxxxxxx) 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/