From: Alexander Graf <agraf@suse.de>
To: anthony.perard@citrix.com
Cc: Xen Devel <xen-devel@lists.xensource.com>,
QEMU-devel <qemu-devel@nongnu.org>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [Qemu-devel] [PATCH V11 00/15] Xen device model support
Date: Tue, 22 Mar 2011 15:23:02 +0100 [thread overview]
Message-ID: <4D88B0C6.6040506@suse.de> (raw)
In-Reply-To: <1299004529-31290-1-git-send-email-anthony.perard@citrix.com>
On 03/01/2011 07:35 PM, anthony.perard@citrix.com wrote:
> From: Anthony PERARD<anthony.perard@citrix.com>
>
> Hi all,
>
> Here is the few change since the V10:
>
> - Add braces for blocks with single statement in the clean-up patch;
> - the patch that builds Xen only for x86 have been removed, instead,
> xen_domainbuild is built with libhw and other Xen files are built for i386
> target only;
> - the redirection structure with function pointer have been removed, instead,
> there are few #define or static inline function use for the compatibility;
ARGH!
The point of the redirection structure was so I can plug in with xenner
and replace all the xen calls with in-qemu versions. If you remove it,
I'll have to put it back in in the xenner patch set :(.
We need some sort of abstraction between calling xs_ functions and
actually calling them. Wrapping all xs_ calls in static inlines would be
fine for that, as would the indirect calling table.
Alex
WARNING: multiple messages have this Message-ID (diff)
From: Alexander Graf <agraf@suse.de>
To: anthony.perard@citrix.com
Cc: Xen Devel <xen-devel@lists.xensource.com>,
QEMU-devel <qemu-devel@nongnu.org>,
Anthony Liguori <anthony@codemonkey.ws>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [Qemu-devel] [PATCH V11 00/15] Xen device model support
Date: Tue, 22 Mar 2011 15:23:02 +0100 [thread overview]
Message-ID: <4D88B0C6.6040506@suse.de> (raw)
In-Reply-To: <1299004529-31290-1-git-send-email-anthony.perard@citrix.com>
On 03/01/2011 07:35 PM, anthony.perard@citrix.com wrote:
> From: Anthony PERARD<anthony.perard@citrix.com>
>
> Hi all,
>
> Here is the few change since the V10:
>
> - Add braces for blocks with single statement in the clean-up patch;
> - the patch that builds Xen only for x86 have been removed, instead,
> xen_domainbuild is built with libhw and other Xen files are built for i386
> target only;
> - the redirection structure with function pointer have been removed, instead,
> there are few #define or static inline function use for the compatibility;
ARGH!
The point of the redirection structure was so I can plug in with xenner
and replace all the xen calls with in-qemu versions. If you remove it,
I'll have to put it back in in the xenner patch set :(.
We need some sort of abstraction between calling xs_ functions and
actually calling them. Wrapping all xs_ calls in static inlines would be
fine for that, as would the indirect calling table.
Alex
next prev parent reply other threads:[~2011-03-22 14:23 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-01 18:35 [Qemu-devel] [PATCH V11 00/15] Xen device model support anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 01/15] xen: Replace some tab-indents with spaces (clean-up) anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 02/15] xen: Make Xen build once anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-23 10:57 ` [Qemu-devel] " Alexander Graf
2011-03-23 10:57 ` Alexander Graf
2011-03-28 14:50 ` [Xen-devel] " Anthony PERARD
2011-03-28 14:50 ` Anthony PERARD
2011-03-28 17:29 ` [Xen-devel] " Alexander Graf
2011-03-28 17:29 ` Alexander Graf
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 03/15] xen: Support new libxc calls from xen unstable anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-23 10:43 ` [Qemu-devel] " Alexander Graf
2011-03-23 10:43 ` Alexander Graf
2011-03-28 15:22 ` [Xen-devel] " Anthony PERARD
2011-03-28 15:22 ` Anthony PERARD
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 04/15] xen: Add initialisation of Xen anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 05/15] xen: Add xenfv machine anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 06/15] xen: Add the Xen platform pci device anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-23 12:08 ` [Qemu-devel] " Alexander Graf
2011-03-23 12:08 ` Alexander Graf
2011-03-28 16:08 ` [Qemu-devel] " Anthony PERARD
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 07/15] piix_pci: Introduces Xen specific call for irq anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 08/15] xen: Introduce Xen Interrupt Controller anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 09/15] xen: Introduce the Xen mapcache anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-23 12:33 ` [Qemu-devel] " Alexander Graf
2011-03-23 12:33 ` Alexander Graf
2011-03-28 16:28 ` [Xen-devel] " Anthony PERARD
2011-03-28 16:28 ` Anthony PERARD
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 10/15] configure: Always use 64bits target physical addresses with xen enabled anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 11/15] Introduce qemu_put_ram_ptr anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-04-06 17:19 ` [Qemu-devel] " John Baboval
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 12/15] vl.c: Introduce getter for shutdown_requested and reset_requested anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 13/15] xen: Initialize event channels and io rings anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-23 12:36 ` [Qemu-devel] " Alexander Graf
2011-03-23 12:36 ` Alexander Graf
2011-03-28 16:30 ` [Xen-devel] " Anthony PERARD
2011-03-28 16:30 ` Anthony PERARD
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 14/15] xen: Set running state in xenstore anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-01 18:35 ` [Qemu-devel] [PATCH V11 15/15] xen: Add Xen hypercall for sleep state in the cmos_s3 callback anthony.perard
2011-03-01 18:35 ` anthony.perard
2011-03-11 16:29 ` [Qemu-devel] [PATCH V11 00/15] Xen device model support Anthony Liguori
2011-03-11 20:47 ` Alexander Graf
2011-03-22 14:23 ` Alexander Graf [this message]
2011-03-22 14:23 ` Alexander Graf
2011-03-22 14:47 ` [Xen-devel] " Anthony PERARD
2011-03-22 14:47 ` Anthony PERARD
2011-03-22 15:13 ` [Xen-devel] " Alexander Graf
2011-03-22 15:13 ` Alexander Graf
2011-03-22 15:40 ` [Xen-devel] " Alexander Graf
2011-03-22 15:40 ` [Xen-devel] " Alexander Graf
2011-03-22 15:52 ` [Xen-devel] Re: [Qemu-devel] " Alexander Graf
2011-03-22 15:52 ` Alexander Graf
2011-03-22 17:22 ` [Xen-devel] " Anthony PERARD
2011-03-22 17:22 ` Anthony PERARD
2011-03-22 17:23 ` [Xen-devel] " Alexander Graf
2011-03-22 17:23 ` [Xen-devel] " Alexander Graf
2011-03-23 12:37 ` [Qemu-devel] " Alexander Graf
2011-03-23 12:37 ` Alexander Graf
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=4D88B0C6.6040506@suse.de \
--to=agraf@suse.de \
--cc=anthony.perard@citrix.com \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xensource.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.