All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: qemu-devel@nongnu.org, Luiz Capitulino <lcapitulino@redhat.com>,
	Christian Borntraeger <borntraeger@de.ibm.com>,
	Cornelia Huck <cornelia.huck@de.ibm.com>,
	Sascha Silbe <silbe@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH v2 0/3] add missing virtio aliases
Date: Fri, 04 Mar 2016 15:54:00 +0100	[thread overview]
Message-ID: <87k2licn07.fsf@blackfin.pond.sub.org> (raw)
In-Reply-To: <20160304145928-mutt-send-email-mst@redhat.com> (Michael S. Tsirkin's message of "Fri, 4 Mar 2016 15:01:18 +0200")

"Michael S. Tsirkin" <mst@redhat.com> 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:1455023713-104799-1-git-send-email-silbe@linux.vnet.ibm.com
>>     "[PATCH 0/3] qemu-iotests: fix 140 on s390x" by Sascha Silbe
>>     <silbe@linux.vnet.ibm.com> on 2016-02-09
>
> I don't see issues here:
>
> Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
>
> You should also Cc lcapitulino@redhat.com and eblake@redhat.com.

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.

  reply	other threads:[~2016-03-04 14:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-18 21:44 [Qemu-devel] [PATCH v2 0/3] add missing virtio aliases Sascha Silbe
2016-02-18 21:44 ` [Qemu-devel] [PATCH v2 1/3] qdev-monitor: improve error message when alias device is unavailable Sascha Silbe
2016-02-22 11:54   ` Cornelia Huck
2016-02-18 21:44 ` [Qemu-devel] [PATCH v2 2/3] qdev-monitor: sort alias table by typename Sascha Silbe
2016-02-22 12:00   ` Cornelia Huck
2016-02-18 21:44 ` [Qemu-devel] [PATCH v2 3/3] qdev-monitor: add missing aliases for virtio device classes Sascha Silbe
2016-02-22 12:09   ` Cornelia Huck
2016-02-22 17:38     ` Sascha Silbe
2016-03-04 12:44 ` [Qemu-devel] [PATCH v2 0/3] add missing virtio aliases Sascha Silbe
2016-03-04 13:01 ` Michael S. Tsirkin
2016-03-04 14:54   ` Markus Armbruster [this message]
2016-03-04 17:02 ` Markus Armbruster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87k2licn07.fsf@blackfin.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=borntraeger@de.ibm.com \
    --cc=cornelia.huck@de.ibm.com \
    --cc=lcapitulino@redhat.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=silbe@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.