qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Avi Kivity <avi@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
	kvm@vger.kernel.org, qemu-devel@nongnu.org,
	Blue Swirl <blauwirbel@gmail.com>,
	ceph-devel@vger.kernel.org, Christian Brunner <chb@muc.de>
Subject: Re: [Qemu-devel] [RFC PATCH 1/1] ceph/rbd block driver for qemu-kvm
Date: Tue, 25 May 2010 11:21:25 -0500	[thread overview]
Message-ID: <4BFBF905.1030901@codemonkey.ws> (raw)
In-Reply-To: <4BFBF7D4.80900@redhat.com>

On 05/25/2010 11:16 AM, Avi Kivity wrote:
> On 05/25/2010 06:01 PM, Anthony Liguori wrote:
>> On 05/25/2010 10:00 AM, Avi Kivity wrote:
>>> The latter.  Why is it less important?  If you don't inherit the 
>>> memory, you can't access it.
>>>
>>>> You can also pass /dev/shm fd's via SCM_RIGHTs to establish shared 
>>>> memory segments dynamically.
>>>
>>> Doesn't work for anonymous memory.
>>
>> What's wrong with /dev/shm memory?
>
> The kernel treats anonymous and nonymous memory differently for 
> swapping (see /proc/sys/vm/swappiness); transparent hugepages won't 
> work for /dev/shm (though it may be argued that that's a problem with 
> thp); setup (/dev/shm defaults to half memory IIRC, we want mem+swap); 
> different cgroup handling; somewhat clunky (a minor concern to be sure).

Surely, with mmu notifiers, it wouldn't be that hard to share anonymous 
memory via an fd though, no?

Regards,

Anthony Liguori

>
> Nothing is a killer, but we should prefer anonymous memory.
>

  reply	other threads:[~2010-05-25 16:21 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-19 19:22 [Qemu-devel] [RFC PATCH 1/1] ceph/rbd block driver for qemu-kvm Christian Brunner
2010-05-20 20:31 ` Blue Swirl
2010-05-20 21:18   ` Christian Brunner
2010-05-20 21:29     ` Anthony Liguori
2010-05-20 22:16       ` Christian Brunner
2010-05-21  5:28         ` Stefan Hajnoczi
2010-05-21  6:13           ` MORITA Kazutaka
2010-05-21  5:54         ` MORITA Kazutaka
2010-05-23 12:01       ` Avi Kivity
2010-05-24  7:12         ` MORITA Kazutaka
2010-05-24 11:05           ` Avi Kivity
2010-05-24 11:42             ` MORITA Kazutaka
2010-05-24 11:56               ` Avi Kivity
2010-05-24 12:07                 ` Cláudio Martins
2010-05-24 14:01                 ` MORITA Kazutaka
2010-05-24 19:07                   ` Christian Brunner
2010-05-24 19:38                     ` Anthony Liguori
2010-05-25  9:14                       ` Avi Kivity
2010-05-25 13:17                         ` Anthony Liguori
2010-05-25 13:25                           ` Avi Kivity
2010-05-25 13:29                             ` Anthony Liguori
2010-05-25 13:36                               ` Avi Kivity
2010-05-25 13:54                                 ` Anthony Liguori
2010-05-25 13:57                                   ` Avi Kivity
2010-05-25 14:02                                     ` Anthony Liguori
2010-05-26  8:44                                       ` Avi Kivity
2010-05-25 14:01                             ` Kevin Wolf
2010-05-25 16:21                               ` Avi Kivity
2010-05-25 17:12                                 ` Sage Weil
2010-05-26  5:24                                   ` MORITA Kazutaka
2010-05-26  8:46                                   ` Avi Kivity
2010-05-24 19:16                 ` Anthony Liguori
2010-05-25  9:19                   ` Avi Kivity
2010-05-25 13:26                   ` MORITA Kazutaka
2010-05-24  8:27         ` Stefan Hajnoczi
2010-05-24 11:03           ` Avi Kivity
2010-05-24 19:19             ` Anthony Liguori
2010-05-25  9:22               ` Avi Kivity
2010-05-25 11:02         ` Kevin Wolf
2010-05-25 11:25           ` Avi Kivity
2010-05-25 12:03             ` Christoph Hellwig
2010-05-25 12:13               ` Avi Kivity
2010-05-25 13:25             ` Anthony Liguori
2010-05-25 13:31               ` Avi Kivity
2010-05-25 13:35                 ` Anthony Liguori
2010-05-25 13:38                   ` Avi Kivity
2010-05-25 13:55                     ` Anthony Liguori
2010-05-25 14:01                       ` Avi Kivity
2010-05-25 14:05                         ` Anthony Liguori
2010-05-25 15:00                           ` Avi Kivity
2010-05-25 15:01                             ` Anthony Liguori
2010-05-25 16:16                               ` Avi Kivity
2010-05-25 16:21                                 ` Anthony Liguori [this message]
2010-05-25 16:27                                   ` Avi Kivity
2010-05-25 13:53               ` Kevin Wolf
2010-05-25 13:55                 ` Avi Kivity
2010-05-25 14:03                   ` Anthony Liguori
2010-05-25 15:02                     ` Avi Kivity
2010-05-25 14:09                   ` Kevin Wolf
2010-05-25 15:01                     ` Avi Kivity
2010-05-20 23:02   ` Yehuda Sadeh Weinraub
2010-05-23  7:59     ` Blue Swirl
2010-05-24  2:17       ` Yehuda Sadeh Weinraub
2010-05-25 20:13         ` Blue Swirl

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=4BFBF905.1030901@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=avi@redhat.com \
    --cc=blauwirbel@gmail.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=chb@muc.de \
    --cc=kvm@vger.kernel.org \
    --cc=kwolf@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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).