public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Pavel Emelianov <xemul@openvz.org>
To: Paul Menage <menage@google.com>
Cc: Balbir Singh <balbir@linux.vnet.ibm.com>,
	Andrew Morton <akpm@osdl.org>, Pavel Emelianov <xemul@openvz.org>,
	Linux Containers <containers@lists.osdl.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [patch -rss] Make RSS accounting display more user friendly
Date: Fri, 22 Jun 2007 06:09:19 +0400	[thread overview]
Message-ID: <467B2F4F.2030101@openvz.org> (raw)
In-Reply-To: <6599ad830706211217l65d76d4ehb42f9165bcf45dd@mail.gmail.com>

Paul Menage wrote:
> On 6/20/07, Balbir Singh <balbir@linux.vnet.ibm.com> wrote:
>>
>>
>> Display the current usage and limit in a more user friendly manner.
>> Number
>> of pages can be confusing if the page size is different. Some systems
>> can choose a page size of 64KB.
> 
> I'm not sure that's such a great idea. "Human-friendly"
> representations would make programmatic parsing harder.
> 
> What's wrong with just showing counts in bytes in the control files?

Nothing wrong, but currently they are shown in "natural" points, i.e. in
those that the controller accounts them in. For RSS controller the natural
point is "page", but auto-converting them from pages to bytes is wrong, as
not all the controllers account in pages. I think that the better way is 
to show the value in the internal units and specify (somehow) what these 
units are. The userspace then can read them and convert accordingly.

> Paul

Thanks,
Pavel

  reply	other threads:[~2007-06-22  7:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-20 15:46 [patch -rss] Make RSS accounting display more user friendly Balbir Singh
2007-06-21 19:17 ` Paul Menage
2007-06-22  2:09   ` Pavel Emelianov [this message]
2007-06-22 15:39     ` Paul Menage
2007-06-23  3:48       ` Balbir Singh
2007-06-25  7:19         ` Paul Menage
2007-06-26 12:39           ` Kirill Korotaev
2007-06-26 12:54             ` Balbir Singh
2007-06-26 14:05               ` Kirill Korotaev
2028-02-27  7:39           ` Balbir Singh
2007-07-09 11:38             ` Peter Zijlstra
2007-07-10  4:44               ` Balbir Singh
2007-07-10 15:29                 ` Randy Dunlap
2007-07-10 15:40                   ` Balbir Singh

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=467B2F4F.2030101@openvz.org \
    --to=xemul@openvz.org \
    --cc=akpm@osdl.org \
    --cc=balbir@linux.vnet.ibm.com \
    --cc=containers@lists.osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=menage@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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox