[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6599ad830708291520t2bc9ea20m2bdcd9e042b3a423@mail.gmail.com>
Date: Wed, 29 Aug 2007 15:20:09 -0700
From: "Paul Menage" <menage@...gle.com>
To: "Dave Hansen" <haveblue@...ibm.com>
Cc: balbir@...ux.vnet.ibm.com,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
"Linux MM Mailing List" <linux-mm@...ck.org>,
"David Rientjes" <rientjes@...gle.com>,
"Linux Containers" <containers@...ts.osdl.org>
Subject: Re: [-mm PATCH] Memory controller improve user interface
On 8/29/07, Dave Hansen <haveblue@...ibm.com> wrote:
> On Thu, 2007-08-30 at 03:34 +0530, Balbir Singh wrote:
> > I've thought about this before. The problem is that a user could
> > set his limit to 10000 bytes, but would then see the usage and
> > limit round to the closest page boundary. This can be confusing
> > to a user.
>
> True, but we're lying if we allow a user to set their limit there,
> because we can't actually enforce a limit at 8,192 bytes vs 10,000.
> They're the same limit as far as the kernel is concerned.
>
> Why not just -EINVAL if the value isn't page-aligned? There are plenty
> of interfaces in the kernel that require userspace to know the page
> size, so this shouldn't be too difficult.
I'd argue that having the user's specified limit be truncated to the
page size is less confusing than giving an EINVAL if it's not page
aligned.
Paul
-
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