[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <49FD5E96.6060301@redhat.com>
Date: Sun, 03 May 2009 12:06:30 +0300
From: Izik Eidus <ieidus@...hat.com>
To: Rik van Riel <riel@...riel.com>
CC: akpm@...ux-foundation.org, linux-kernel@...r.kernel.org,
aarcange@...hat.com, chrisw@...hat.com, alan@...rguk.ukuu.org.uk,
device@...ana.org, linux-mm@...ck.org, hugh@...itas.com,
nickpiggin@...oo.com.au
Subject: Re: [PATCH 1/6] ksm: limiting the num of mem regions user can register
per fd.
Rik van Riel wrote:
> On Sun, 3 May 2009 01:16:07 +0300
> Izik Eidus <ieidus@...hat.com> wrote:
>
>
>> Right now user can open /dev/ksm fd and register unlimited number of
>> regions, such behavior may allocate unlimited amount of kernel memory
>> and get the whole host into out of memory situation.
>>
>
> How many times can a process open /dev/ksm?
>
> If a process can open /dev/ksm a thousand times and then
> register 1000 regions through each file descriptor, this
> patch does not help all that much...
>
>
The idea is that the limitation is now on the maximum file descriptors
user can open.
So for each such file descriptor user can open 1024 structures that are
just few bytes each.
The whole propose of this patch is to avoid while (1) {
IOCTL(REGISTER_MEMORY_REGION) } and oom the host.
--
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