On Sat, Jun 22, 2013 at 01:51:17PM +0900, Hyunhee Kim wrote: > And one more thing. > > vmpressure sends all of the lower events than the level (current). > > 155 list_for_each_entry(ev, &vmpr->events, node) { > 156 if (level >= ev->level) { > 157 eventfd_signal(ev->efd, 1); > 158 signalled = true; > 159 } > 160 } > > Isn't it wasteful? > When we registered "low" and "critical", and the current level is > critical, why should we signal "low" level together? > If we receive "critical" only, can't we just assume "low" level at the > user space? Agree. -- Kind regards, Minchan Kim -- 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>