[Top][All Lists]

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

Re: [Qemu-devel] [PATCH V2 0/4] machine: QemuOpts per machine

From: Andreas Färber
Subject: Re: [Qemu-devel] [PATCH V2 0/4] machine: QemuOpts per machine
Date: Mon, 26 May 2014 18:42:45 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0

Am 26.05.2014 14:48, schrieb Michael S. Tsirkin:
> On Mon, May 26, 2014 at 03:40:54PM +0300, Marcel Apfelbaum wrote:
>> V1 -> V2:
>>  - Rebased on qom-next queue. 
>>  - Patch 1/4:
>>    - Since there are a lot of discussions on the correct
>>      way to do the fix, added a FIXME comment to find a better way.
>>  - Patch 2/4:
>>    - Addressed Michael Roth's comments:
>>      - Added output visitor test
>>      - Tweaked it as advised
>>  - Patch 4/4
>>    - Addressed Andreas Farber's comments:
>>      - Renamed machine_state -> ms to conform with the other patches

In the other one it's "machine" now, but I agree that "ms" is
sufficiently understandable in machine.c. :)

>>    - Solved an issue discovered with make check:
>>      - Updated machine->kernel_cmdline to be an empty string
>>        if no string was supplied by the user.
>>  - All other comments were addressed in the mail thread (I hope...)
> For series:
> Acked-by: Michael S. Tsirkin <address@hidden>

I have them queued on qom-next, but giving mdroth a chance to ack/nack
the QAPI bits. I believe all his comments are addressed now, save for
deeper investigations with armbru that I hope can be done as follow-up.


Still build-testing on BSD.


SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

reply via email to

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