[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qe
From: |
Samuel Thibault |
Subject: |
Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu |
Date: |
Wed, 6 Aug 2008 13:50:55 +0100 |
User-agent: |
Mutt/1.5.12-2006-07-14 |
Markus Armbruster, le Wed 06 Aug 2008 08:43:49 -0400, a écrit :
> It *is* quite move, becauses it accomplishes a lot: it goes from a
> heavily modified fork of an oldish version all the way to merge with
> upstream, as far as PV is concerned.
Then why doing it in qemu before having it tested in the xen unstable
tree? That's not the way I usually see merging happen.
> For what it's worth, I went over significant parts of Gerd's patch
> (all the generic stuff + pvfb) with a fine comb, comparing it to what
> we have now. I consider it sound.
I'm not saying it's not fine. I had a look and the code looked fine
indeed. But what I'm afraid of is the delta with Ian would have to
bear when merging: is it save/restore safe, does it work with PCI
pass-through, VT-D, etc.?
> If that's where we want to go, we can of course still argue whether we
> should go in leaps or baby steps, and whether Gerd's leap lands in
> quite the right spot.
Baby steps are much easier to review. That's how things are usually
done, and here it looks to me like it is feasible to achieve in Xen (and
have it tested) before event thinking about importing a pile of code in
qemu where it won't receive as much testing as the xen-unstable tree
receives.
Samuel
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, (continued)
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Gerd Hoffmann, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Anthony Liguori, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Blue Swirl, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/05
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Gerd Hoffmann, 2008/08/06
- Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Markus Armbruster, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu,
Samuel Thibault <=
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Gerd Hoffmann, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Gerd Hoffmann, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Gerd Hoffmann, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Gerd Hoffmann, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Laurent Vivier, 2008/08/06
- Re: [Xen-devel] Re: [Qemu-devel] [PATCH 0/7] merge some xen bits into qemu, Samuel Thibault, 2008/08/06