From: Primiano Tucci <primiano@chromium.org> To: David Rientjes <rientjes@google.com> Cc: "Kirill A. Shutemov" <kirill@shutemov.name>, Petr Cermak <petrcermak@chromium.org>, linux-kernel@vger.kernel.org, linux-mm@kvack.org, Andrew Morton <akpm@linux-foundation.org>, Bjorn Helgaas <bhelgaas@google.com>, Hugh Dickins <hughd@google.com> Subject: Re: [PATCH v2 2/2] task_mmu: Add user-space support for resetting mm->hiwater_rss (peak RSS) Date: Fri, 23 Jan 2015 00:28:38 +0000 [thread overview] Message-ID: <CA+yH71e2ewvA41BNyb=TTPn+yx2zWzY6rn09hRVVgWKoeMgwXQ@mail.gmail.com> (raw) In-Reply-To: <alpine.DEB.2.10.1501221523390.27807@chino.kir.corp.google.com> On Thu, Jan 22, 2015 at 11:27 PM, David Rientjes <rientjes@google.com> wrote: > If you reset the hwm for a process, rss grows to 100MB, another process > resets the hwm, and you see a hwm of 2MB, that invalidates the hwm > entirely. Not sure I follow this scenario. Where does the 2MB come from? How can you see a hwm of 2MB, under which conditions? HVM can never be < RSS. Again, what you are talking about is the case of two profilers racing for using the same interface (hwm). This is the same case today of the PG_referenced bit. > The hwm is already defined as the > highest rss the process has attained, resetting it and trying to make any > inference from the result is racy and invalidates the actual value which > is useful. The counter arugment is: once you have one very high peak, the hvm becomes essentially useless for the rest of the lifetime of the process (until a higher peak comes). This makes very hard to understand what is going on in the meanwhile (from userspace). Anyways, are you proposing to pursue a different approach? Is the approach 2. that petrcermark@ proposed in the beginning of the thread going to address this concern?
WARNING: multiple messages have this Message-ID (diff)
From: Primiano Tucci <primiano@chromium.org> To: David Rientjes <rientjes@google.com> Cc: "Kirill A. Shutemov" <kirill@shutemov.name>, Petr Cermak <petrcermak@chromium.org>, linux-kernel@vger.kernel.org, linux-mm@kvack.org, Andrew Morton <akpm@linux-foundation.org>, Bjorn Helgaas <bhelgaas@google.com>, Hugh Dickins <hughd@google.com> Subject: Re: [PATCH v2 2/2] task_mmu: Add user-space support for resetting mm->hiwater_rss (peak RSS) Date: Fri, 23 Jan 2015 00:28:38 +0000 [thread overview] Message-ID: <CA+yH71e2ewvA41BNyb=TTPn+yx2zWzY6rn09hRVVgWKoeMgwXQ@mail.gmail.com> (raw) In-Reply-To: <alpine.DEB.2.10.1501221523390.27807@chino.kir.corp.google.com> On Thu, Jan 22, 2015 at 11:27 PM, David Rientjes <rientjes@google.com> wrote: > If you reset the hwm for a process, rss grows to 100MB, another process > resets the hwm, and you see a hwm of 2MB, that invalidates the hwm > entirely. Not sure I follow this scenario. Where does the 2MB come from? How can you see a hwm of 2MB, under which conditions? HVM can never be < RSS. Again, what you are talking about is the case of two profilers racing for using the same interface (hwm). This is the same case today of the PG_referenced bit. > The hwm is already defined as the > highest rss the process has attained, resetting it and trying to make any > inference from the result is racy and invalidates the actual value which > is useful. The counter arugment is: once you have one very high peak, the hvm becomes essentially useless for the rest of the lifetime of the process (until a higher peak comes). This makes very hard to understand what is going on in the meanwhile (from userspace). Anyways, are you proposing to pursue a different approach? Is the approach 2. that petrcermark@ proposed in the beginning of the thread going to address this concern? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2015-01-23 0:28 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-01-07 17:06 [PATCH v2 0/2] task_mmu: Add user-space support for resetting mm->hiwater_rss (peak RSS) Petr Cermak 2015-01-07 17:06 ` Petr Cermak 2015-01-07 17:06 ` [PATCH v2 1/2] task_mmu: Reduce excessive indentation in clear_refs_write Petr Cermak 2015-01-07 17:06 ` Petr Cermak 2015-01-07 17:06 ` [PATCH v2 2/2] task_mmu: Add user-space support for resetting mm->hiwater_rss (peak RSS) Petr Cermak 2015-01-07 17:06 ` Petr Cermak 2015-01-07 17:24 ` Kirill A. Shutemov 2015-01-07 17:24 ` Kirill A. Shutemov 2015-01-14 15:22 ` Petr Cermak 2015-01-14 15:22 ` Petr Cermak 2015-01-14 23:36 ` Kirill A. Shutemov 2015-01-14 23:36 ` Kirill A. Shutemov 2015-01-21 22:58 ` David Rientjes 2015-01-21 22:58 ` David Rientjes 2015-01-22 0:22 ` Primiano Tucci 2015-01-22 0:22 ` Primiano Tucci 2015-01-22 23:27 ` David Rientjes 2015-01-22 23:27 ` David Rientjes 2015-01-23 0:28 ` Primiano Tucci [this message] 2015-01-23 0:28 ` Primiano Tucci 2015-01-27 0:00 ` David Rientjes 2015-01-27 0:00 ` David Rientjes 2015-02-03 3:26 ` Petr Cermak 2015-02-03 3:26 ` Petr Cermak 2015-02-03 15:51 ` Minchan Kim 2015-02-03 15:51 ` Minchan Kim 2015-02-03 20:16 ` David Rientjes 2015-02-03 20:16 ` David Rientjes 2015-01-14 23:39 ` Kirill A. Shutemov 2015-01-14 23:39 ` Kirill A. Shutemov 2015-01-15 16:46 ` Petr Cermak 2015-01-15 16:46 ` Petr Cermak
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CA+yH71e2ewvA41BNyb=TTPn+yx2zWzY6rn09hRVVgWKoeMgwXQ@mail.gmail.com' \ --to=primiano@chromium.org \ --cc=akpm@linux-foundation.org \ --cc=bhelgaas@google.com \ --cc=hughd@google.com \ --cc=kirill@shutemov.name \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=petrcermak@chromium.org \ --cc=rientjes@google.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.