[Webkit-unassigned] [Bug 155255] MemoryPressureHandler doesn't work if cgroups aren't present in Linux

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=155255

--- Comment #12 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to comment #10)
> (In reply to comment #0)
> > The current Linux implementation of MemoryPressureHandler relies on
> > /sys/fs/cgroup/memory to be properly setup. This isn't the case for all
> > desktop systems and embedded devices, so having an alternative method to
> > measure memory pressure should be desirable (especially on embedded devices
> > with limited memory).
> 
> As I rememeber, the polling way was considered but it was rejected because
> of a concern that regularly waking up cpu to read meminfo itself badly
> affected on performance. https://bugs.webkit.org/show_bug.cgi?id=123532#c2 .
> For the reason, it is intended that MemoryPressureHandler works for only
> cgroup avaiable platforms. I know there is a small difference between this
> approach and previous one, using a timer or sleep() in a separate thread. I
> am also glad to see this alernative memory handler though, I would like to
> ask if there is anything different from using timer way and this patch is
> actually useful on real embeded systems. How about performance there? Is it
> acceptable?

I agree, for embedded devices these wakeups could affect the battery life.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160312/1c482845/attachment.html>