Re: [PATCH v5 0/3] mm, proc: Implement /proc//totmaps

2016-09-19 Thread Robert Foss
On 2016-09-14 05:12 AM, Michal Hocko wrote: On Tue 13-09-16 13:27:39, Sonny Rao wrote: On Tue, Sep 13, 2016 at 12:12 AM, Michal Hocko wrote: On Mon 12-09-16 10:28:53, Sonny Rao wrote: On Mon, Sep 12, 2016 at 10:15 AM, Michal Hocko wrote: On Mon 12-09-16 08:31:36, Sonny Rao wrote: [...]

Re: [PATCH v5 0/3] mm, proc: Implement /proc//totmaps

2016-09-19 Thread Robert Foss
On 2016-09-19 03:32 PM, Michal Hocko wrote: On Mon 19-09-16 11:16:31, Robert Foss wrote: On 2016-09-14 05:12 AM, Michal Hocko wrote: On Tue 13-09-16 13:27:39, Sonny Rao wrote: [...] Given that smaps doesn't provide this in a straightforward way, what do you think is the right w

[PATCH] drm/docs: Move "scaling mode" property.

2016-05-02 Thread robert . foss
From: Robert Foss The "scaling mode" property has been moved to the DRM->Generic. It has also had a list of supported drivers added to it. Signed-off-by: Robert Foss --- Documentation/DocBook/gpu.tmpl | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-)

[PATCH v2] drm/docs: Move "scaling mode" property.

2016-05-02 Thread robert . foss
From: Robert Foss The "scaling mode" property has been moved to the DRM->Generic. It has also had a list of supported drivers added to it. Reviewed-by: Alex Deucher Signed-off-by: Robert Foss --- Changes since v1: - Added Reviewed-by: Alex Deucher Documentation/DocBook/

[PACTH v2 0/3] Implement /proc//totmaps

2016-08-12 Thread robert . foss
From: Robert Foss This series implements /proc/PID/totmaps, a tool for retrieving summarized information about the mappings of a process. Changes since v1: - Removed IS_ERR check from get_task_mm() function - Changed comment format - Moved proc_totmaps_operations declaration inside internal.h

[PACTH v2 2/3] Documentation/filesystems: Fixed typo

2016-08-12 Thread robert . foss
From: Robert Foss Fixed a -> an typo. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index e8d0075..7d001be 100644 --

[PACTH v2 3/3] Documentation/filesystems: Added /proc/PID/totmaps documentation

2016-08-12 Thread robert . foss
From: Robert Foss Added documentation covering /proc/PID/totmaps. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 21 + 1 file changed, 21 insertions(+) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index 7d001be

[PACTH v2 1/3] mm, proc: Implement /proc//totmaps

2016-08-12 Thread robert . foss
From: Robert Foss This is based on earlier work by Thiago Goncales. It implements a new per process proc file which summarizes the contents of the smaps file but doesn't display any addresses. It gives more detailed information than statm like the PSS (proprotional set size). It differs

Re: [PACTH v2 0/3] Implement /proc//totmaps

2016-08-15 Thread Robert Foss
On 2016-08-14 05:04 AM, Michal Hocko wrote: On Fri 12-08-16 18:04:19, robert.f...@collabora.com wrote: From: Robert Foss This series implements /proc/PID/totmaps, a tool for retrieving summarized information about the mappings of a process. The changelog is absolutely missing the usecase

Re: [PACTH v2 1/3] mm, proc: Implement /proc//totmaps

2016-08-15 Thread Robert Foss
On 2016-08-13 10:39 AM, Jann Horn wrote: On Fri, Aug 12, 2016 at 06:04:20PM -0400, robert.f...@collabora.com wrote: diff --git a/fs/proc/internal.h b/fs/proc/internal.h index aa27810..c55e1fe 100644 --- a/fs/proc/internal.h +++ b/fs/proc/internal.h @@ -281,6 +281,7 @@ struct proc_maps_private

Re: [PACTH v2 0/3] Implement /proc//totmaps

2016-08-15 Thread Robert Foss
On 2016-08-15 09:42 AM, Michal Hocko wrote: On Mon 15-08-16 09:00:04, Robert Foss wrote: On 2016-08-14 05:04 AM, Michal Hocko wrote: On Fri 12-08-16 18:04:19, robert.f...@collabora.com wrote: From: Robert Foss This series implements /proc/PID/totmaps, a tool for retrieving summarized

Re: [PACTH v2 1/3] mm, proc: Implement /proc//totmaps

2016-08-15 Thread Robert Foss
On 2016-08-15 09:57 AM, Robert Foss wrote: On 2016-08-13 10:39 AM, Jann Horn wrote: On Fri, Aug 12, 2016 at 06:04:20PM -0400, robert.f...@collabora.com wrote: diff --git a/fs/proc/internal.h b/fs/proc/internal.h index aa27810..c55e1fe 100644 --- a/fs/proc/internal.h +++ b/fs/proc

Re: [PACTH v2 0/3] Implement /proc//totmaps

2016-08-16 Thread Robert Foss
On 2016-08-16 03:12 AM, Michal Hocko wrote: On Mon 15-08-16 12:25:10, Robert Foss wrote: On 2016-08-15 09:42 AM, Michal Hocko wrote: [...] The use case is to speed up monitoring of memory consumption in environments where RSS isn't precise. For example Chrome tends to many proc

[PACTH v3 0/3] Implement /proc//totmaps

2016-08-16 Thread robert . foss
From: Robert Foss This series provides the /proc/PID/totmaps feature, which summarizes the information provided by /proc/PID/smaps for improved performance and usability reasons. A use case is to speed up monitoring of memory consumption in environments where RSS isn't precise. For ex

[PACTH v3 1/3] mm, proc: Implement /proc//totmaps

2016-08-16 Thread robert . foss
From: Robert Foss This is based on earlier work by Thiago Goncales. It implements a new per process proc file which summarizes the contents of the smaps file but doesn't display any addresses. It gives more detailed information than statm like the PSS (proprotional set size). It differs

[PACTH v3 3/3] Documentation/filesystems: Added /proc/PID/totmaps documentation

2016-08-16 Thread robert . foss
From: Robert Foss Added documentation covering /proc/PID/totmaps. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 21 + 1 file changed, 21 insertions(+) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index 7d001be

[PACTH v3 2/3] Documentation/filesystems: Fixed typo

2016-08-16 Thread robert . foss
From: Robert Foss Fixed a -> an typo. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index e8d0075..7d001be 100644 --

Re: [PACTH v3 3/3] Documentation/filesystems: Added /proc/PID/totmaps documentation

2016-08-16 Thread Robert Foss
On 2016-08-16 02:01 PM, Jann Horn wrote: nit: s/extenssion/extension/ Thanks :) -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PACTH v3 1/3] mm, proc: Implement /proc//totmaps

2016-08-16 Thread Robert Foss
On 2016-08-16 02:18 PM, Jann Horn wrote: On Tue, Aug 16, 2016 at 01:34:14PM -0400, robert.f...@collabora.com wrote: From: Robert Foss This is based on earlier work by Thiago Goncales. It implements a new per process proc file which summarizes the contents of the smaps file but doesn&#

[PACTH v4 3/3] Documentation/filesystems: Added /proc/PID/totmaps documentation

2016-08-16 Thread robert . foss
From: Robert Foss Added documentation covering /proc/PID/totmaps. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 21 + 1 file changed, 21 insertions(+) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index 7d001be

[PACTH v4 2/3] Documentation/filesystems: Fixed typo

2016-08-16 Thread robert . foss
From: Robert Foss Fixed a -> an typo. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index e8d0075..7d001be 100644 --

[PACTH v4 1/3] mm, proc: Implement /proc//totmaps

2016-08-16 Thread robert . foss
From: Robert Foss This is based on earlier work by Thiago Goncales. It implements a new per process proc file which summarizes the contents of the smaps file but doesn't display any addresses. It gives more detailed information than statm like the PSS (proprotional set size). It differs

[PACTH v4 0/3] Implement /proc//totmaps

2016-08-16 Thread robert . foss
From: Robert Foss This series provides the /proc/PID/totmaps feature, which summarizes the information provided by /proc/PID/smaps for improved performance and usability reasons. A use case is to speed up monitoring of memory consumption in environments where RSS isn't precise. For ex

Re: [PACTH v2 0/3] Implement /proc//totmaps

2016-08-17 Thread Robert Foss
On 2016-08-17 09:03 AM, Michal Hocko wrote: On Wed 17-08-16 11:31:25, Jann Horn wrote: On Wed, Aug 17, 2016 at 10:22:00AM +0200, Michal Hocko wrote: On Tue 16-08-16 12:46:51, Robert Foss wrote: [...] $ /usr/bin/time -v -p zsh -c "repeat 25 { awk '/^Rss/{rss+=\$2} /^Pss/{pss

Re: [PACTH v2 0/3] Implement /proc//totmaps

2016-08-18 Thread Robert Foss
On 2016-08-18 02:01 PM, Michal Hocko wrote: On Thu 18-08-16 10:47:57, Sonny Rao wrote: On Thu, Aug 18, 2016 at 12:44 AM, Michal Hocko wrote: On Wed 17-08-16 11:57:56, Sonny Rao wrote: [...] 2) User space OOM handling -- we'd rather do a more graceful shutdown than let the kernel's OOM kill

Re: [PACTH v2 0/3] Implement /proc//totmaps

2016-08-22 Thread Robert Foss
On 2016-08-22 10:12 AM, Minchan Kim wrote: On Mon, Aug 22, 2016 at 09:40:52AM +0200, Michal Hocko wrote: On Mon 22-08-16 09:07:45, Minchan Kim wrote: [...] #!/bin/sh ./smap_test & pid=$! for i in $(seq 25) do awk '/^Rss/{rss+=$2} /^Pss/{pss+=$2} END {}' \ /proc/$pid/smaps do

Re: [PACTH v4 1/3] mm, proc: Implement /proc//totmaps

2016-08-31 Thread Robert Foss
On 2016-08-31 05:45 AM, Jacek Anaszewski wrote: Hi Robert, On 08/17/2016 12:33 AM, robert.f...@collabora.com wrote: From: Robert Foss This is based on earlier work by Thiago Goncales. It implements a new per process proc file which summarizes the contents of the smaps file but doesn&#

Re: [PACTH v4 1/3] mm, proc: Implement /proc//totmaps

2016-09-01 Thread Robert Foss
On 2016-08-31 01:04 PM, Mateusz Guzik wrote: On Wed, Aug 31, 2016 at 12:36:26PM -0400, Robert Foss wrote: On 2016-08-31 05:45 AM, Jacek Anaszewski wrote: +static void *m_totmaps_start(struct seq_file *p, loff_t *pos) +{ +return NULL + (*pos == 0); +} + +static void *m_totmaps_next(struct

[PATCH v5 1/3] mm, proc: Implement /proc//totmaps

2016-09-05 Thread robert . foss
From: Robert Foss This is based on earlier work by Thiago Goncales. It implements a new per process proc file which summarizes the contents of the smaps file but doesn't display any addresses. It gives more detailed information than statm like the PSS (proprotional set size). It differs

[PATCH v5 0/3] mm, proc: Implement /proc//totmaps

2016-09-05 Thread robert . foss
From: Robert Foss This series provides the /proc/PID/totmaps feature, which summarizes the information provided by /proc/PID/smaps for improved performance and usability reasons. A use case is to speed up monitoring of memory consumption in environments where RSS isn't precise. For ex

[PATCH v5 3/3] Documentation/filesystems: Added /proc/PID/totmaps documentation

2016-09-05 Thread robert . foss
From: Robert Foss Added documentation covering /proc/PID/totmaps. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 21 + 1 file changed, 21 insertions(+) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index fcc1ac0

[PATCH v5 2/3] Documentation/filesystems: Fixed typo

2016-09-05 Thread robert . foss
From: Robert Foss Fixed a -> an typo. Signed-off-by: Robert Foss --- Documentation/filesystems/proc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index 68080ad..fcc1ac0 100644 --

Re: [PATCH v5 2/3] Documentation/filesystems: Fixed typo

2016-09-07 Thread Robert Foss
On 2016-09-07 07:22 PM, Kees Cook wrote: On Mon, Sep 5, 2016 at 1:14 PM, wrote: From: Robert Foss Fixed a -> an typo. Signed-off-by: Robert Foss Acked-by: Kees Cook This could be taken directly into the docs tree, I think -- no reason to make it depend on the rest of the ser

[PATCH v1] Documentation/filesystems: Fixed typo

2016-09-08 Thread robert . foss
From: Robert Foss Fixed a -> an typo. Signed-off-by: Robert Foss Acked-by: Kees Cook --- Documentation/filesystems/proc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/filesystems/proc.txt b/Documentation/filesystems/proc.txt index 68080ad..fcc1

Re: [PATCH v5 1/3] mm, proc: Implement /proc//totmaps

2016-09-12 Thread Robert Foss
Hey Oleg! Thanks for the feedback, I'll keep it in mind, but currently it looks like the patch is on ice for non-implementation related reasons. Rob. @@ -2854,6 +2854,7 @@ static const struct pid_entry tgid_base_stuff[] = { REG("clear_refs", S_IWUSR, proc_clear_refs_operations),