[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/3] add missing virtio aliases
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/3] add missing virtio aliases |
Date: |
Fri, 04 Mar 2016 15:54:00 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
"Michael S. Tsirkin" <address@hidden> writes:
> On Thu, Feb 18, 2016 at 10:44:11PM +0100, Sascha Silbe wrote:
>> Split out from the series fixing qemu-iotest 140 on s390x [1].
>>
>> This series makes crafting qemu command lines that work across
>> multiple (virtio-supporting) platforms at lot easier.
>>
>> Changes since the original series:
>> - included patch to improve the error message when the target device
>> class of an alias doesn't exist
>> - added comment about keeping alias table sorted
>> - covering all non-abstract virtio device classes rather than just
>> those already implemented on s390x
>>
>> Sascha Silbe (3):
>> qdev-monitor: improve error message when alias device is unavailable
>> qdev-monitor: sort alias table by typename
>> qdev-monitor: add missing aliases for virtio device classes
>>
>> qdev-monitor.c | 42 ++++++++++++++++++++++++++++++++++--------
>> 1 file changed, 34 insertions(+), 8 deletions(-)
>>
>> [1] mid:address@hidden
>> "[PATCH 0/3] qemu-iotests: fix 140 on s390x" by Sascha Silbe
>> <address@hidden> on 2016-02-09
>
> I don't see issues here:
>
> Reviewed-by: Michael S. Tsirkin <address@hidden>
>
> You should also Cc address@hidden and address@hidden
Cc'ing them now.
> Markus, will you take this series?
qdev-monitor falls in the crack between monitor (maintained by me) and
qdev (no maintainer, which is a bloody shame). I can take it through my
tree.