Re: [PATCH 4/4] add ksm kernel shared memory driver.

Izik Eidus
2009-04-01T22:59:22Z

Anthony Liguori wrote:
> Andrea Arcangeli wrote:
>> On Tue, Mar 31, 2009 at 10:54:57AM -0500, Anthony Liguori wrote:
>>
>>> You can still disable ksm and simply return ENOSYS for the MADV_
>>> flag. You
>>
>>
Anthony, the biggest problem about madvice() is that it is a real system
call api, i wouldnt want in that stage of ksm commit into api changes of
linux...

The ioctl itself is restricting, madvice is much more...,

Can we draft this issue to after ksm is merged, and after all the big
new fetures that we want to add to ksm will be merge....
(then the api would be much more stable, and we will be able to ask ppl
in the list about changing of api, but for new driver that it yet to be
merged, it is kind of overkill to add api to linux)

What do you think?