[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4A0184F7.7070309@redhat.com>
Date: Wed, 06 May 2009 15:39:19 +0300
From: Izik Eidus <ieidus@...hat.com>
To: Rik van Riel <riel@...hat.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 2/6] ksm: dont allow overlap memory addresses registrations.
Rik van Riel wrote:
> Izik Eidus wrote:
>> Rik van Riel wrote:
>>> Izik Eidus wrote:
>>>> subjects say it all.
>>>
>>> Not a very useful commit message.
>>>
>>> This makes me wonder, though.
>>>
>>> What happens if a user mmaps a 30MB memory region, registers it
>>> with KSM and then unmaps the middle 10MB?
>>
>> User cant break 30MB into smaller one.
>
> The user can break up the underlying VMAs though.
So? KSM work on contigiouns virtual address, if user will break its
virtual address and will leave it to be registered inside ksm
get_user_pages() will just fail, and ksm will skip scanning this
addresses...
Normal usage of ksm is:
1) Allocating big chunck of memory.
2) registering it inside ksm
3) free the memory and remove it from ksm...
>
> I am just wondering out loud if we really want two
> VMA-like objects in the kernel, the VMA itself and
> a separate KSM object, with different semantics.
>
> Maybe this is fine, but I do think it's a question
> that needs to be thought about.
Yea, we had some talk about that issue, considering the fact that user
register its memory using ioctl and not systemcall, and considering the
fact that ksm is loadable module that the kernel doesnt depend on,
How would you prefer to see the interface?
--
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