Re: [PATCH 3/6] ksm: change the KSM_REMOVE_MEMORY_REGION ioctl.

Andrea Arcangeli
2009-05-07T13:28:18Z

On Thu, May 07, 2009 at 02:13:31PM +0100, Hugh Dickins wrote:
> No: "KSM" stands for "Kernel Samepage Merging", doesn't it?
> Or maybe someone can devise a better term for the "S" of it.
> I think we're all too familiar with "KSM" to want to outlaw that,
> just don't dwell too much on the "Kernel Shared Memory" expansion.

So you suggest to keep KSM acronym but meaning Kernel Samepage
Merging instead of Kernel Shared Memory ;).

> > In addition I consistently use the term "shared KSM pages" often,
> > should I rename all those instances to "merged pages"? I used the word
> > 'merging' only when describing the operation KSM does when it creates
> > shared pages, but never to name the generated pages themself.
>
> No, you and your audience and your readers will find it clearest
> if you remark on the change in naming upfront, then get on with
> describing it all in the way that comes most naturally to you.

Actually that's what I already did yesterday, but then I thought I
shall try to get it in sync to avoid confusion. But if KSM name is
here to stay, I'll stick with the KSM name and perhaps a few to
"merged pages" as you suggested.

> But do sprinkle in a few "merged pages", I suggest: perhaps by
> the time you deliver it, they'll be coming more naturally to you.
>
> (Actually, "shared KSM pages" makes more sense if that "S" is not

ehehe ;)

> for Shared. I keep wondering what the two "k"s in kksmd stand for.)

kernel KSM daemon.