[Qemu-devel] pending 1.5 patches

[Qemu-devel] pending 1.5 patches

[Paolo Bonzini]

I see the following patches on the list that IMO deserve inclusion into
1.5.  All of them should be low-risk.

<1368718348-15199-1-git-send-email-emaste@freebsd.org>
[PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict

<1369046780-17498-1-git-send-email-pbonzini@redhat.com>
[PATCH for-1.5 resend] vl: new runstate transition:
RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE

<1369045989-14016-1-git-send-email-hpoussin@reactos.org>
[PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
on 32 bit hosts)

<1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
[PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
(not the rest of the series)

There is also <20130520081814.GA8162@redhat.com>, but I think it's
too intrusive.

Paolo

Re: [Qemu-devel] pending 1.5 patches

[Stefan Weil]

[-- Attachment #1: Type: text/plain, Size: 1038 bytes --]

Am 20.05.2013 13:00, schrieb Paolo Bonzini:
> I see the following patches on the list that IMO deserve inclusion into
> 1.5.  All of them should be low-risk.
>
> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict
>
> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
> [PATCH for-1.5 resend] vl: new runstate transition:
> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
>
> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
> on 32 bit hosts)

See also http://patchwork.ozlabs.org/patch/244757/

> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
> (not the rest of the series)
>
> There is also <20130520081814.GA8162@redhat.com>, but I think it's
> too intrusive.
>
> Paolo

Plus this one, please:

http://patchwork.ozlabs.org/patch/244758/
spitz: Fix compiler warning (MinGW-w64)

Stefan


[-- Attachment #2: Type: text/html, Size: 2464 bytes --]

Re: [Qemu-devel] pending 1.5 patches

[Paolo Bonzini]

Il 20/05/2013 13:28, Stefan Weil ha scritto:
> Am 20.05.2013 13:00, schrieb Paolo Bonzini:
>> I see the following patches on the list that IMO deserve inclusion into
>> 1.5.  All of them should be low-risk.
>>
>> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
>> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict
>>
>> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
>> [PATCH for-1.5 resend] vl: new runstate transition:
>> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
>>
>> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
>> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
>> on 32 bit hosts)
> 
> See also http://patchwork.ozlabs.org/patch/244757/

Yes, they are identical.

>> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
>> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
>> (not the rest of the series)
>>
>> There is also <20130520081814.GA8162@redhat.com>, but I think it's
>> too intrusive.
> 
> Plus this one, please:
> 
> http://patchwork.ozlabs.org/patch/244758/
> spitz: Fix compiler warning (MinGW-w64)

Agreed.

Paolo

Re: [Qemu-devel] pending 1.5 patches

[Anthony Liguori]

Paolo Bonzini <pbonzini@redhat.com> writes:

> I see the following patches on the list that IMO deserve inclusion into
> 1.5.  All of them should be low-risk.
>
> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict

Ack.

> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
> [PATCH for-1.5 resend] vl: new runstate transition:
> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE

Ack.

> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
> on 32 bit hosts)

Ack.

> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
> (not the rest of the series)

Ack.

> There is also <20130520081814.GA8162@redhat.com>, but I think it's
> too intrusive.

It's untested so there's no way it can go in.  Since it only impacts
WHQL testing I don't think it's a bit problem to wait for stable.

Regards,

Anthony Liguori

>
> Paolo

Re: [Qemu-devel] pending 1.5 patches

[Anthony Liguori]

Paolo Bonzini <pbonzini@redhat.com> writes:

> Il 20/05/2013 13:28, Stefan Weil ha scritto:
>> Am 20.05.2013 13:00, schrieb Paolo Bonzini:
>>> I see the following patches on the list that IMO deserve inclusion into
>>> 1.5.  All of them should be low-risk.
>>>
>>> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
>>> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict
>>>
>>> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
>>> [PATCH for-1.5 resend] vl: new runstate transition:
>>> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
>>>
>>> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
>>> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
>>> on 32 bit hosts)
>> 
>> See also http://patchwork.ozlabs.org/patch/244757/
>
> Yes, they are identical.
>
>>> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
>>> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
>>> (not the rest of the series)
>>>
>>> There is also <20130520081814.GA8162@redhat.com>, but I think it's
>>> too intrusive.
>> 
>> Plus this one, please:
>> 
>> http://patchwork.ozlabs.org/patch/244758/
>> spitz: Fix compiler warning (MinGW-w64)
>
> Agreed.

We don't build with -Werror on MinGW-w64 so a warning fix is very low
priority.  It needs to wait for stable.

Regards,

Anthony Liguori

>
> Paolo

Re: [Qemu-devel] pending 1.5 patches

[Michael S. Tsirkin]

On Mon, May 20, 2013 at 01:00:56PM +0200, Paolo Bonzini wrote:
> I see the following patches on the list that IMO deserve inclusion into
> 1.5.  All of them should be low-risk.
> 
> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict
> 
> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
> [PATCH for-1.5 resend] vl: new runstate transition:
> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
> 
> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
> on 32 bit hosts)
> 
> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
> (not the rest of the series)
> 
> There is also <20130520081814.GA8162@redhat.com>, but I think it's
> too intrusive.
> 
> Paolo

Message-ID: <20130513201840.5430.86331.stgit@bling.home>
[PATCH v2] pci-assign: Add MSI affinity support

low risk and fixes a nasty bug

-- 
MST

Re: [Qemu-devel] pending 1.5 patches

[Stefan Weil]

Am 20.05.2013 14:25, schrieb Anthony Liguori:
> Paolo Bonzini <pbonzini@redhat.com> writes:
>
>> Il 20/05/2013 13:28, Stefan Weil ha scritto:
>>> Am 20.05.2013 13:00, schrieb Paolo Bonzini:
>>>> I see the following patches on the list that IMO deserve inclusion into
>>>> 1.5.  All of them should be low-risk.
>>>>
>>>> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
>>>> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict
>>>>
>>>> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
>>>> [PATCH for-1.5 resend] vl: new runstate transition:
>>>> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
>>>>
>>>> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
>>>> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
>>>> on 32 bit hosts)
>>> See also http://patchwork.ozlabs.org/patch/244757/
>> Yes, they are identical.
>>
>>>> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
>>>> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
>>>> (not the rest of the series)
>>>>
>>>> There is also <20130520081814.GA8162@redhat.com>, but I think it's
>>>> too intrusive.
>>> Plus this one, please:
>>>
>>> http://patchwork.ozlabs.org/patch/244758/
>>> spitz: Fix compiler warning (MinGW-w64)
>> Agreed.
> We don't build with -Werror on MinGW-w64 so a warning fix is very low
> priority.  It needs to wait for stable.
>
> Regards,
>
> Anthony Liguori

I don't mind waiting. For your information: MinGW-w64
normally should compile without warnings when a recent
set of MinGW-w64 include files is used.

Please don't forget http://patchwork.ozlabs.org/patch/244757/
_before_ Paolo's list.

Thanks,
Stefan Weil

Re: [Qemu-devel] pending 1.5 patches

[Eric Blake]

[-- Attachment #1: Type: text/plain, Size: 1083 bytes --]

On 05/20/2013 05:00 AM, Paolo Bonzini wrote:
> I see the following patches on the list that IMO deserve inclusion into
> 1.5.  All of them should be low-risk.
> 
> <1368718348-15199-1-git-send-email-emaste@freebsd.org>
> [PATCH v2] Rename hexdump to avoid FreeBSD libutil conflict
> 
> <1369046780-17498-1-git-send-email-pbonzini@redhat.com>
> [PATCH for-1.5 resend] vl: new runstate transition:
> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
> 
> <1369045989-14016-1-git-send-email-hpoussin@reactos.org>
> [PATCH for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation
> on 32 bit hosts)
> 
> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com>
> [PATCH 1/3] chardev: Make consistent with udp device for new qapi backend
> (not the rest of the series)

Actually, I argue that patch 2/3 of that series would also be an ABI
change if it is not included NOW, since it renames a union member that
to date has not been released.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 621 bytes --]

Re: [Qemu-devel] pending 1.5 patches

[Paolo Bonzini]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Il 20/05/2013 17:07, Eric Blake ha scritto:
> On 05/20/2013 05:00 AM, Paolo Bonzini wrote:
>> I see the following patches on the list that IMO deserve
>> inclusion into 1.5.  All of them should be low-risk.
>> 
>> <1368718348-15199-1-git-send-email-emaste@freebsd.org> [PATCH v2]
>> Rename hexdump to avoid FreeBSD libutil conflict
>> 
>> <1369046780-17498-1-git-send-email-pbonzini@redhat.com> [PATCH
>> for-1.5 resend] vl: new runstate transition: 
>> RUN_STATE_GUEST_PANICKED -> RUN_STATE_FINISH_MIGRATE
>> 
>> <1369045989-14016-1-git-send-email-hpoussin@reactos.org> [PATCH
>> for-1.5] osdep: fix qemu_anon_ram_free trace (+ fix compilation 
>> on 32 bit hosts)
>> 
>> <1369032665-18159-2-git-send-email-lilei@linux.vnet.ibm.com> 
>> [PATCH 1/3] chardev: Make consistent with udp device for new qapi
>> backend (not the rest of the series)
> 
> Actually, I argue that patch 2/3 of that series would also be an
> ABI change if it is not included NOW, since it renames a union
> member that to date has not been released.

As written, patch 2/3 is an ABI change compared to 1.4, so it cannot
go in anyway, at least not in this shape.

Paolo
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRmj5NAAoJEBvWZb6bTYbykzMQAJzBl1g27TW/vyei9e0SI7Ap
V2h/A+eCBiPaxJQisem6DA7YOlQLdTEgZXMRrYgqwR6r5tKTYd3lokW5jCmXTZ6Q
dD6cfw7Z4QTRl0FLmVey0rzna4iCf89xyBAB68Xt8ZnZGxc0DiR/82pjZCZESZ9k
bTDheIjdW7jJEdEUFx7/gDWTfun4rwvD6j/bTXfBKB6DNQ5s559wc2aiVRyZhBcZ
v2abMCK2atKmyjST61GOEUKniQtRzMmvMoorBcvytPv03sY9KFc6awINTZ3VlhhI
g1ht3XepSYD41Z1cnQL0vZe7ILw8eZtVLsRwaV6u0rlp6dcq8s6MoLtwTUp8Lfcc
b0+bzk30eND16x1IsMKQ2LKyCC5dRVnVNtNrmDnoDMpjkDjbI3TFmZcH3icseEc7
h/OmxxWpsfEghwCQe0fnch8ruV69Ba0FFxnBt5DWYKuv6aEuEAmKHtxFakrlT41P
lSz++AetLSy2tm7ssn+L3eb6a7EMhf+L0i62KvzK8Eewhqkl7krGzAmjhXwItk2r
VK2v+KtCUMFB3D3Y4FXp3RKsLsFvx8xaprsE+5sDfrmKFqCVL6sMbSfckdfOEuhg
sstB9AzB6u/+N3mLh49172vhq63sSEcW1It1HuCJIlvvOtZY1mimATo6lsrZGz0K
+oOBg8Fol5itqXtbdPuE
=aOgl
-----END PGP SIGNATURE-----