[Top][All Lists]

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

Re: [Qemu-devel] [PATCH 00/54] WIP: chardev: qom-ify

From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH 00/54] WIP: chardev: qom-ify
Date: Wed, 4 Jan 2017 15:50:24 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1

On 01/04/2017 03:20 PM, Marc-André Lureau wrote:
> Hi
> On Mon, Jan 2, 2017 at 11:27 AM Paolo Bonzini <address@hidden> wrote:
>> On 12/12/2016 23:42, Marc-André Lureau wrote:
>>> Note: this series has been tested with Linux and cross-building with
>>> mingw. It mostly breaks on other platforms,
>> What breaks exactly?
> Not much that I know. I was afraid it would break on BSD (in particular the
> move to own file and last patch), so I tested on FreeBSD. The last version
> of the series works on win32, linux and freebsd. It can be found on
> https://github.com/elmarco/qemu/commits/chrfe It's slightly over 80 patches
> now, and needs some win32 fixes I sent earlier (subprocess tests, and
> main-loop busy loop fix)
> Since Erik started reviewing the series, I suggest to resend the first part
> until "qom-ify" patch. Erik, feel free to review the remaining part of the
> WIP series, but more changes happened after iirc. What do you think?

It's Eric (but you're not the first, and probably not the last).  The
qom-ify patch is 15/54, right? Okay, I'll complete my review up to that
point, and then you can split the series into two: a v2 through qom-ify
(which should probably be ready to go, or only very little to change),
and the second half which is still the WIP.

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

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]