All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Antonios Motakis <a.motakis@virtualopensystems.com>,
	"Michael S. Tsirkin" <mst@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	snabb-devel@googlegroups.com,
	"Anthony Liguori" <aliguori@amazon.com>,
	"Juan Quintela" <quintela@redhat.com>,
	"Michael Tokarev" <mjt@tls.msk.ru>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Nikolay Nikolaev" <n.nikolaev@virtualopensystems.com>,
	"qemu-devel qemu-devel" <qemu-devel@nongnu.org>,
	"Alexander Graf" <agraf@suse.de>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Luke Gorrie" <lukego@gmail.com>,
	"VirtualOpenSystems Technical Team" <tech@virtualopensystems.com>,
	"Andreas Färber" <afaerber@suse.de>,
	"Richard Henderson" <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH v8 01/17] Convert -mem-path to QemuOpts and add prealloc and share properties
Date: Mon, 17 Feb 2014 18:27:03 +0100	[thread overview]
Message-ID: <53024667.2060103@redhat.com> (raw)
In-Reply-To: <CAG8rG2zvo4pxstqT53MEbMM1=D6QAWv_it_bdwBGt16KbZ7D7A@mail.gmail.com>

Il 17/02/2014 14:04, Antonios Motakis ha scritto:
>
>     Hmm in that case, let's not add prealloc as a property here.
>     Stick to existing flag for that, this way we don't need
>     to support 3 ways to do this.
>
>
> We'll remove the prealloc property then for the next version.
>
> Otherwise, I wonder if the long term memdev discussion can impose
> additional constraints on our current implementation. It seems to me
> that this shouldn't be the case, right?

No, on the contrary it will be more flexible.  Just this patch will be 
dropped and a different command-line will be required on your part.

Paolo

  reply	other threads:[~2014-02-17 17:27 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-13 12:03 [Qemu-devel] [PATCH v8 00/17] Vhost and vhost-net support for userspace based backends Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 01/17] Convert -mem-path to QemuOpts and add prealloc and share properties Antonios Motakis
2014-02-15 18:10   ` Michael Tokarev
2014-02-16 23:42     ` Paolo Bonzini
2014-02-17  7:56       ` Michael S. Tsirkin
2014-02-17 13:04         ` Antonios Motakis
2014-02-17 17:27           ` Paolo Bonzini [this message]
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 02/17] Add chardev API qemu_chr_fe_read_all Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 03/17] Add chardev API qemu_chr_fe_set_msgfds Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 04/17] Add chardev API qemu_chr_fe_get_msgfds Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 05/17] Add G_IO_HUP handler for socket chardev Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 06/17] vhost_net should call the poll callback only when it is set Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 07/17] Refactor virtio-net to use generic get_vhost_net Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 08/17] vhost_net_init will use VhostNetOptions to get all its arguments Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 09/17] Add vhost_ops to vhost_dev struct and replace all relevant ioctls Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 10/17] Add mandatory_features to vhost_dev Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 11/17] Add vhost-backend and VhostBackendType Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 12/17] Add vhost-user as a vhost backend Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 13/17] Add new vhost-user netdev backend Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 14/17] Add the vhost-user netdev backend to the command line Antonios Motakis
2014-02-17 17:17   ` Eric Blake
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 15/17] Add vhost-user protocol documentation Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 16/17] libqemustub: add stubs to be able to use qemu-char.c Antonios Motakis
2014-02-13 12:03 ` [Qemu-devel] [PATCH v8 17/17] Add qtest for vhost-user Antonios Motakis
2014-02-26 16:44 ` [Qemu-devel] [PATCH v8 00/17] Vhost and vhost-net support for userspace based backends Michael S. Tsirkin

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=53024667.2060103@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=a.motakis@virtualopensystems.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=aliguori@amazon.com \
    --cc=armbru@redhat.com \
    --cc=lukego@gmail.com \
    --cc=mjt@tls.msk.ru \
    --cc=mst@redhat.com \
    --cc=n.nikolaev@virtualopensystems.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=rth@twiddle.net \
    --cc=snabb-devel@googlegroups.com \
    --cc=stefanha@redhat.com \
    --cc=tech@virtualopensystems.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.