[Webkit-unassigned] [Bug 200391] Implement memory monitoring functions for Linux OS

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 12 17:49:59 PDT 2019


Carlos Alberto Lopez Perez <clopez at igalia.com> changed:

           What    |Removed                     |Added
                 CC|                            |clopez at igalia.com

--- Comment #3 from Carlos Alberto Lopez Perez <clopez at igalia.com> ---
(In reply to Paulo Matos from comment #2)
> I wanted to post this for review/discussion. Initially I discussed this with
> my colleague Caio and he suggested to get this in WTF, except WTF already
> contains memory management functions for Linux under
> ./Source/WTF/wtf/linux/MemoryFootprintLinux.cpp but for a completely
> different purpose.
> Initially my reason for implementing this was to get results out of
> JetStream2 benchmark with memory information when ran with
> runMode='RAMification'.
> I would like to know if this is something that upstream would like to keep
> in jsc.cpp or move somewhere else (maybe WTF), possibly even merging it with
> current functionality? Suggestions welcome.

I think its reasonable to merge this with WTF/wtf/linux/MemoryFootprintLinux.cpp
and share the code.

For example, getCurrentMemoryUsage() can be implemented as returning ProcessMemoryStatus.resident after calling currentProcessMemoryStatus(ProcessMemoryStatus)

Adding resetPeak() to MemoryFootprintLinux.cpp is also reasonable.

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

More information about the webkit-unassigned mailing list