Re: [PATCH v6] memcg: event control at vmpressure.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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>




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]