[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 3/3] xen_machine_pv: do not create a dummy CPU in machine->init

On Wed, 29 May 2013, Paolo Bonzini wrote:
> Il 29/05/2013 14:11, Andreas FÃrber ha scritto:
> >> xen_machine_pv uses cpu_x86_init, therefore it has been broken.
> >>
> >> This patch fixes the problem by removing the dummy CPU creation
> >> altogether from xen_init_pv, relying on the fact that QEMU can now cope
> >> with a machine without an emulated CPU.
> >>
> >> This fix should be backported to QEMU 1.5.
> >>
> >> Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> >> CC: imammedo@xxxxxxxxxx
> >> CC: qemu-stable@xxxxxxxxxx
> > 
> > Change looks okay,
> > 
> > Reviewed-by: Andreas FÃrber <afaerber@xxxxxxx>
> > 
> > But for the future please avoid "this patch" in the commit message
> > (because later on it's a "commit", not a patch) and also please put
> > additional notices such as for backporting below the --- line.
> I read it as a warning to backporters that the patch, even though it may
> apply, was not tested and may even be wrong in 1.4.

Yes, that's right.

In any case Andreas has a point about moving notes about backports or
patch revision changes under the --- line, I just have to find a way to
make it work with guilt (that would also cut those notes from the commit
messages therefore when I generate the patches they would go missing).
Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.