linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Izik Eidus <ieidus@redhat.com>,
	akpm@linux-foundation.org, linux-kernel@vger.kernel.org,
	linux-mm@kvack.org, kvm@vger.kernel.org, aarcange@redhat.com,
	chrisw@redhat.com, mtosatti@redhat.com, hugh@veritas.com
Subject: Re: [PATCH 5/5] add ksm kernel shared memory driver.
Date: Mon, 20 Apr 2009 14:11:07 +0300	[thread overview]
Message-ID: <49EC584B.3060809@redhat.com> (raw)
In-Reply-To: <20090420110223.76ba4593@lxorguk.ukuu.org.uk>

Alan Cox wrote:
> The minor number you are using already belongs to another project.
>
> 10,234 is free but it would be good to know what device naming is
> proposed. I imagine other folks would like to know why you aren't using
> sysfs or similar or extending /dev/kvm ?
>   

ksm was deliberately made independent of kvm.  While there may or may 
not be uses of ksm without kvm (you could run ordinary qemu, but no one 
would do this in a production deployment), keeping them separate helps 
avoid unnecessary interdependencies.  For example all tlb flushes are 
mediated through mmu notifiers instead of ksm hooking directly into kvm.

-- 
error compiling committee.c: too many arguments to function

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2009-04-20 11:11 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-20  1:36 [PATCH 0/5] ksm - dynamic page sharing driver for linux v4 Izik Eidus
2009-04-20  1:36 ` [PATCH 1/5] MMU_NOTIFIERS: add set_pte_at_notify() Izik Eidus
2009-04-20  1:36   ` [PATCH 2/5] add get_pte(): helper function: fetching pte for va Izik Eidus
2009-04-20  1:36     ` [PATCH 3/5] add page_wrprotect(): write protecting page Izik Eidus
2009-04-20  1:36       ` [PATCH 4/5] add replace_page(): change the page pte is pointing to Izik Eidus
2009-04-20  1:36         ` [PATCH 5/5] add ksm kernel shared memory driver Izik Eidus
2009-04-20 10:02           ` Alan Cox
2009-04-20 11:11             ` Avi Kivity [this message]
2009-04-20 12:52               ` Izik Eidus
2009-04-27 22:34           ` Andrew Morton
2009-04-27 23:12             ` Izik Eidus
2009-04-30 17:46               ` Izik Eidus
2009-04-30 17:58                 ` Andrew Morton
2009-05-13 23:17           ` Andrew Morton
2009-05-13 23:25             ` Chris Wright
2009-05-14  0:14               ` Anthony Liguori
2009-05-14  0:11                 ` Izik Eidus
2009-05-14  0:15             ` Izik Eidus
2009-05-14  1:40               ` Tony Breeds

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49EC584B.3060809@redhat.com \
    --to=avi@redhat.com \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=chrisw@redhat.com \
    --cc=hugh@veritas.com \
    --cc=ieidus@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mtosatti@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).