qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests
@ 2016-09-09  8:31 Greg Kurz
  2016-09-10  8:20 ` Amit Shah
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Greg Kurz @ 2016-09-09  8:31 UTC (permalink / raw)
  To: qemu-devel
  Cc: Amit Shah, Paolo Bonzini, Jason Wang, Stefan Hajnoczi,
	Michael S. Tsirkin

Except virtio-9p, all virtio-* tests are orphan. This patch tries to fix
it, according to the following logic:

- when the related subsystem has its own section in MAINTAINERS, the test
  is added there
- otherwise it is added to the "parent" section (aka. SCSI, Network devices,
  virtio)

Signed-off-by: Greg Kurz <groug@kaod.org>
---
 MAINTAINERS |    7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index b6fb84e8267c..cc6d1cdc9165 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -826,6 +826,7 @@ Network devices
 M: Jason Wang <jasowang@redhat.com>
 S: Odd Fixes
 F: hw/net/
+F: tests/virtio-net-test.c
 T: git git://github.com/jasowang/qemu.git net
 
 SCSI
@@ -833,6 +834,7 @@ M: Paolo Bonzini <pbonzini@redhat.com>
 S: Supported
 F: include/hw/scsi*
 F: hw/scsi/*
+F: tests/virtio-scsi-test.c
 T: git git://github.com/bonzini/qemu.git scsi-next
 
 LSI53C895A
@@ -885,6 +887,7 @@ S: Supported
 F: hw/*/virtio*
 F: net/vhost-user.c
 F: include/hw/virtio/
+F: tests/virtio-balloon-test.c
 
 virtio-9p
 M: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
@@ -903,6 +906,7 @@ S: Supported
 F: hw/block/virtio-blk.c
 F: hw/block/dataplane/*
 F: hw/virtio/dataplane/*
+F: tests/virtio-blk-test.c
 T: git git://github.com/stefanha/qemu.git block
 
 virtio-ccw
@@ -925,6 +929,8 @@ S: Supported
 F: hw/char/virtio-serial-bus.c
 F: hw/char/virtio-console.c
 F: include/hw/virtio/virtio-serial.h
+F: tests/virtio-console-test.c
+F: tests/virtio-serial-test.c
 
 virtio-rng
 M: Amit Shah <amit.shah@redhat.com>
@@ -933,6 +939,7 @@ F: hw/virtio/virtio-rng.c
 F: include/hw/virtio/virtio-rng.h
 F: include/sysemu/rng*.h
 F: backends/rng*.c
+F: tests/virtio-rng-test.c
 
 nvme
 M: Keith Busch <keith.busch@intel.com>

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests
  2016-09-09  8:31 [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests Greg Kurz
@ 2016-09-10  8:20 ` Amit Shah
  2016-09-14 15:21 ` Stefan Hajnoczi
  2016-09-14 15:48 ` Michael S. Tsirkin
  2 siblings, 0 replies; 4+ messages in thread
From: Amit Shah @ 2016-09-10  8:20 UTC (permalink / raw)
  To: Greg Kurz
  Cc: qemu-devel, Paolo Bonzini, Jason Wang, Stefan Hajnoczi,
	Michael S. Tsirkin

On (Fri) 09 Sep 2016 [10:31:20], Greg Kurz wrote:
> Except virtio-9p, all virtio-* tests are orphan. This patch tries to fix
> it, according to the following logic:
> 
> - when the related subsystem has its own section in MAINTAINERS, the test
>   is added there
> - otherwise it is added to the "parent" section (aka. SCSI, Network devices,
>   virtio)
> 
> Signed-off-by: Greg Kurz <groug@kaod.org>

I'm fine with these.

For places where my name appears:

Reviewed-by: Amit Shah <amit.shah@redhat.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests
  2016-09-09  8:31 [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests Greg Kurz
  2016-09-10  8:20 ` Amit Shah
@ 2016-09-14 15:21 ` Stefan Hajnoczi
  2016-09-14 15:48 ` Michael S. Tsirkin
  2 siblings, 0 replies; 4+ messages in thread
From: Stefan Hajnoczi @ 2016-09-14 15:21 UTC (permalink / raw)
  To: Greg Kurz
  Cc: qemu-devel, Amit Shah, Paolo Bonzini, Jason Wang,
	Michael S. Tsirkin

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

On Fri, Sep 09, 2016 at 10:31:20AM +0200, Greg Kurz wrote:
> Except virtio-9p, all virtio-* tests are orphan. This patch tries to fix
> it, according to the following logic:
> 
> - when the related subsystem has its own section in MAINTAINERS, the test
>   is added there
> - otherwise it is added to the "parent" section (aka. SCSI, Network devices,
>   virtio)
> 
> Signed-off-by: Greg Kurz <groug@kaod.org>
> ---
>  MAINTAINERS |    7 +++++++
>  1 file changed, 7 insertions(+)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests
  2016-09-09  8:31 [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests Greg Kurz
  2016-09-10  8:20 ` Amit Shah
  2016-09-14 15:21 ` Stefan Hajnoczi
@ 2016-09-14 15:48 ` Michael S. Tsirkin
  2 siblings, 0 replies; 4+ messages in thread
From: Michael S. Tsirkin @ 2016-09-14 15:48 UTC (permalink / raw)
  To: Greg Kurz
  Cc: qemu-devel, Amit Shah, Paolo Bonzini, Jason Wang, Stefan Hajnoczi

On Fri, Sep 09, 2016 at 10:31:20AM +0200, Greg Kurz wrote:
> Except virtio-9p, all virtio-* tests are orphan. This patch tries to fix
> it, according to the following logic:
> 
> - when the related subsystem has its own section in MAINTAINERS, the test
>   is added there
> - otherwise it is added to the "parent" section (aka. SCSI, Network devices,
>   virtio)
> 
> Signed-off-by: Greg Kurz <groug@kaod.org>

I applied this, thanks.

> ---
>  MAINTAINERS |    7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b6fb84e8267c..cc6d1cdc9165 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -826,6 +826,7 @@ Network devices
>  M: Jason Wang <jasowang@redhat.com>
>  S: Odd Fixes
>  F: hw/net/
> +F: tests/virtio-net-test.c
>  T: git git://github.com/jasowang/qemu.git net
>  
>  SCSI
> @@ -833,6 +834,7 @@ M: Paolo Bonzini <pbonzini@redhat.com>
>  S: Supported
>  F: include/hw/scsi*
>  F: hw/scsi/*
> +F: tests/virtio-scsi-test.c
>  T: git git://github.com/bonzini/qemu.git scsi-next
>  
>  LSI53C895A
> @@ -885,6 +887,7 @@ S: Supported
>  F: hw/*/virtio*
>  F: net/vhost-user.c
>  F: include/hw/virtio/
> +F: tests/virtio-balloon-test.c
>  
>  virtio-9p
>  M: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
> @@ -903,6 +906,7 @@ S: Supported
>  F: hw/block/virtio-blk.c
>  F: hw/block/dataplane/*
>  F: hw/virtio/dataplane/*
> +F: tests/virtio-blk-test.c
>  T: git git://github.com/stefanha/qemu.git block
>  
>  virtio-ccw
> @@ -925,6 +929,8 @@ S: Supported
>  F: hw/char/virtio-serial-bus.c
>  F: hw/char/virtio-console.c
>  F: include/hw/virtio/virtio-serial.h
> +F: tests/virtio-console-test.c
> +F: tests/virtio-serial-test.c
>  
>  virtio-rng
>  M: Amit Shah <amit.shah@redhat.com>
> @@ -933,6 +939,7 @@ F: hw/virtio/virtio-rng.c
>  F: include/hw/virtio/virtio-rng.h
>  F: include/sysemu/rng*.h
>  F: backends/rng*.c
> +F: tests/virtio-rng-test.c
>  
>  nvme
>  M: Keith Busch <keith.busch@intel.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-09-14 15:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-09  8:31 [Qemu-devel] [PATCH] MAINTAINERS: add virtio-* tests Greg Kurz
2016-09-10  8:20 ` Amit Shah
2016-09-14 15:21 ` Stefan Hajnoczi
2016-09-14 15:48 ` Michael S. Tsirkin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).