On 04/24/2013 12:24 AM, Anton Vorontsov wrote: > Setting the variable on every event seems a bit wasteful... does it make > sense to set it in vmpressure_register_event()? We'll have to make it a > counter, but the good thing is that we won't need any additional locks for > the counter. My bad, I was not looking at the code. There are two variables here: One of them is an event variable: kernel_event. It is set to true upon registration when we are registering a kernel event. We use it to decide whether we should call a function or signal eventfd for that event. The other, is a vmpr event and should indeed, be set in all pressure isntances. It tells us if we should only trigger kernel events (if false), or userspace events as well (if true). This is because, for instance, userspace events may not always be triggered (for instance, due to flags mismatch) -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>