[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <661de9470706222048g638a06dboc0b2e85975a9ba70@mail.gmail.com>
Date: Sat, 23 Jun 2007 09:18:07 +0530
From: "Balbir Singh" <balbir@...ux.vnet.ibm.com>
To: "Paul Menage" <menage@...gle.com>
Cc: "Pavel Emelianov" <xemul@...nvz.org>,
"Andrew Morton" <akpm@...l.org>,
"Linux Containers" <containers@...ts.osdl.org>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>
Subject: Re: [patch -rss] Make RSS accounting display more user friendly
On 6/22/07, Paul Menage <menage@...gle.com> wrote:
> On 6/21/07, Pavel Emelianov <xemul@...nvz.org> wrote:
> >
> > 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.
>
> This exposes more implementation detail than I think is good.
> Something like a memory controller should use a more abstract
> interface like bytes, and do the conversion to/from its internal
> "natural" units like pages internally. E.g. the example cpu accounting
> controller that I included with my patch set reports usage in
> milliseconds, even though it counts internally in jiffies.
>
Hi, Paul,
Bytes are too fine grained. Do we also accept the input in bytes? kB
are probably the best, but then with the limit set to LONG_MAX, kB
wrap around (if the input is in units of pages) on 64 bit machines
(the sameapplies to bytes as well).
The problem with input in bytes is that the user will have to ensure
that the input is
a multiple of page size, which implies that she would need to use the
calculator every time.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists