All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
@ 2016-09-02 22:03 Thomas Huth
  2016-09-03  7:09 ` Fam Zheng
  2016-09-05  8:22 ` Markus Armbruster
  0 siblings, 2 replies; 7+ messages in thread
From: Thomas Huth @ 2016-09-02 22:03 UTC (permalink / raw)
  To: Stefan Hajnoczi; +Cc: qemu-block, qemu-devel

The folder does not exist anymore, thus should be removed from the
MAINTAINERS file, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index b6fb84e..ff45f8c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -902,7 +902,6 @@ L: qemu-block@nongnu.org
 S: Supported
 F: hw/block/virtio-blk.c
 F: hw/block/dataplane/*
-F: hw/virtio/dataplane/*
 T: git git://github.com/stefanha/qemu.git block
 
 virtio-ccw
-- 
1.8.3.1

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

* Re: [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
  2016-09-02 22:03 [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file Thomas Huth
@ 2016-09-03  7:09 ` Fam Zheng
  2016-09-05  8:22 ` Markus Armbruster
  1 sibling, 0 replies; 7+ messages in thread
From: Fam Zheng @ 2016-09-03  7:09 UTC (permalink / raw)
  To: Thomas Huth; +Cc: Stefan Hajnoczi, qemu-devel, qemu-block

On Sat, 09/03 00:03, Thomas Huth wrote:
> The folder does not exist anymore, thus should be removed from the
> MAINTAINERS file, too.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b6fb84e..ff45f8c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -902,7 +902,6 @@ L: qemu-block@nongnu.org
>  S: Supported
>  F: hw/block/virtio-blk.c
>  F: hw/block/dataplane/*
> -F: hw/virtio/dataplane/*
>  T: git git://github.com/stefanha/qemu.git block
>  
>  virtio-ccw
> -- 
> 1.8.3.1
> 
> 

Reviewed-by: Fam Zheng <famz@redhat.com>

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

* Re: [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
  2016-09-02 22:03 [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file Thomas Huth
  2016-09-03  7:09 ` Fam Zheng
@ 2016-09-05  8:22 ` Markus Armbruster
  2016-09-05  8:34   ` Thomas Huth
  1 sibling, 1 reply; 7+ messages in thread
From: Markus Armbruster @ 2016-09-05  8:22 UTC (permalink / raw)
  To: Thomas Huth; +Cc: Stefan Hajnoczi, qemu-devel, qemu-block

Thomas Huth <thuth@redhat.com> writes:

> The folder does not exist anymore, thus should be removed from the
> MAINTAINERS file, too.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b6fb84e..ff45f8c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -902,7 +902,6 @@ L: qemu-block@nongnu.org
>  S: Supported
>  F: hw/block/virtio-blk.c
>  F: hw/block/dataplane/*
> -F: hw/virtio/dataplane/*
>  T: git git://github.com/stefanha/qemu.git block
>  
>  virtio-ccw

Are there any other patterns in MAINTAINERS that don't match anything?

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

* Re: [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
  2016-09-05  8:22 ` Markus Armbruster
@ 2016-09-05  8:34   ` Thomas Huth
  2016-09-05 11:05     ` Markus Armbruster
  0 siblings, 1 reply; 7+ messages in thread
From: Thomas Huth @ 2016-09-05  8:34 UTC (permalink / raw)
  To: Markus Armbruster; +Cc: Stefan Hajnoczi, qemu-devel, qemu-block

On 05.09.2016 10:22, Markus Armbruster wrote:
> Thomas Huth <thuth@redhat.com> writes:
> 
>> The folder does not exist anymore, thus should be removed from the
>> MAINTAINERS file, too.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>  MAINTAINERS | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index b6fb84e..ff45f8c 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -902,7 +902,6 @@ L: qemu-block@nongnu.org
>>  S: Supported
>>  F: hw/block/virtio-blk.c
>>  F: hw/block/dataplane/*
>> -F: hw/virtio/dataplane/*
>>  T: git git://github.com/stefanha/qemu.git block
>>  
> 
> Are there any other patterns in MAINTAINERS that don't match anything?

for i in `grep "^[FX]: " MAINTAINERS | sed "s/^.: //"` ; do \
  if [ ! -e "$i" ]; then echo "$i" ; fi ; \
done

include/hw/xilinx.h
include/hw/*/xlnx*.c
include/hw/acpi/piix.h
hw/i386/*dsl
scripts/acpi*py
hw/virtio/dataplane/*
include/hw/cpu/icc_bus.h
hw/cpu/icc_bus.c
block/raw-aio.h

 Thomas

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

* Re: [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
  2016-09-05  8:34   ` Thomas Huth
@ 2016-09-05 11:05     ` Markus Armbruster
  2016-09-05 11:34       ` Thomas Huth
  0 siblings, 1 reply; 7+ messages in thread
From: Markus Armbruster @ 2016-09-05 11:05 UTC (permalink / raw)
  To: Thomas Huth; +Cc: qemu-block, qemu-devel, Stefan Hajnoczi

Thomas Huth <thuth@redhat.com> writes:

> On 05.09.2016 10:22, Markus Armbruster wrote:
>> Thomas Huth <thuth@redhat.com> writes:
>> 
>>> The folder does not exist anymore, thus should be removed from the
>>> MAINTAINERS file, too.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>>  MAINTAINERS | 1 -
>>>  1 file changed, 1 deletion(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index b6fb84e..ff45f8c 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -902,7 +902,6 @@ L: qemu-block@nongnu.org
>>>  S: Supported
>>>  F: hw/block/virtio-blk.c
>>>  F: hw/block/dataplane/*
>>> -F: hw/virtio/dataplane/*
>>>  T: git git://github.com/stefanha/qemu.git block
>>>  
>> 
>> Are there any other patterns in MAINTAINERS that don't match anything?
>
> for i in `grep "^[FX]: " MAINTAINERS | sed "s/^.: //"` ; do \
>   if [ ! -e "$i" ]; then echo "$i" ; fi ; \
> done
>
> include/hw/xilinx.h

Gone since commit d5001cf.

> include/hw/*/xlnx*.c

Bug, should be .h.

> include/hw/acpi/piix.h

Bug, should be piix4.h.

> hw/i386/*dsl
> scripts/acpi*py

Gone since commit 9fc6502.

> hw/virtio/dataplane/*

Gone since commit fee089e.

> include/hw/cpu/icc_bus.h
> hw/cpu/icc_bus.c

Gone since commit dfeb867.

> block/raw-aio.h

Moved to include/block/raw-aio.h in commit 0187f5c.

>  Thomas

Let's fix all this.  You or I?

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

* Re: [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
  2016-09-05 11:05     ` Markus Armbruster
@ 2016-09-05 11:34       ` Thomas Huth
  2016-09-05 12:29         ` Markus Armbruster
  0 siblings, 1 reply; 7+ messages in thread
From: Thomas Huth @ 2016-09-05 11:34 UTC (permalink / raw)
  To: Markus Armbruster; +Cc: qemu-block, qemu-devel, Stefan Hajnoczi

On 05.09.2016 13:05, Markus Armbruster wrote:
> Thomas Huth <thuth@redhat.com> writes:
> 
>> On 05.09.2016 10:22, Markus Armbruster wrote:
>>> Thomas Huth <thuth@redhat.com> writes:
>>>
>>>> The folder does not exist anymore, thus should be removed from the
>>>> MAINTAINERS file, too.
>>>>
>>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>>> ---
>>>>  MAINTAINERS | 1 -
>>>>  1 file changed, 1 deletion(-)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index b6fb84e..ff45f8c 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -902,7 +902,6 @@ L: qemu-block@nongnu.org
>>>>  S: Supported
>>>>  F: hw/block/virtio-blk.c
>>>>  F: hw/block/dataplane/*
>>>> -F: hw/virtio/dataplane/*
>>>>  T: git git://github.com/stefanha/qemu.git block
>>>>  
>>>
>>> Are there any other patterns in MAINTAINERS that don't match anything?
>>
>> for i in `grep "^[FX]: " MAINTAINERS | sed "s/^.: //"` ; do \
>>   if [ ! -e "$i" ]; then echo "$i" ; fi ; \
>> done
>>
>> include/hw/xilinx.h
> 
> Gone since commit d5001cf.
> 
>> include/hw/*/xlnx*.c
> 
> Bug, should be .h.
> 
>> include/hw/acpi/piix.h
> 
> Bug, should be piix4.h.
> 
>> hw/i386/*dsl
>> scripts/acpi*py
> 
> Gone since commit 9fc6502.
> 
>> hw/virtio/dataplane/*
> 
> Gone since commit fee089e.
> 
>> include/hw/cpu/icc_bus.h
>> hw/cpu/icc_bus.c
> 
> Gone since commit dfeb867.
> 
>> block/raw-aio.h
> 
> Moved to include/block/raw-aio.h in commit 0187f5c.
> 
>>  Thomas
> 
> Let's fix all this.  You or I?

I don't mind. I'm currently busy hunting some bugs ... So feel free to
send the patches for these issues. (and in case you also don't have time
for that now, I've also put an entry on my TODO list, so I could do that
later if you prefer that)

 Thomas

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

* Re: [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file
  2016-09-05 11:34       ` Thomas Huth
@ 2016-09-05 12:29         ` Markus Armbruster
  0 siblings, 0 replies; 7+ messages in thread
From: Markus Armbruster @ 2016-09-05 12:29 UTC (permalink / raw)
  To: Thomas Huth; +Cc: Stefan Hajnoczi, qemu-devel, qemu-block

Thomas Huth <thuth@redhat.com> writes:

> On 05.09.2016 13:05, Markus Armbruster wrote:
[...]
>> Let's fix all this.  You or I?
>
> I don't mind. I'm currently busy hunting some bugs ... So feel free to
> send the patches for these issues. (and in case you also don't have time
> for that now, I've also put an entry on my TODO list, so I could do that
> later if you prefer that)

Patch sent.  Thanks!

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

end of thread, other threads:[~2016-09-05 12:29 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-02 22:03 [Qemu-devel] [PATCH] virtio-blk: Remove hw/virtio/dataplane folder from MAINTAINERS file Thomas Huth
2016-09-03  7:09 ` Fam Zheng
2016-09-05  8:22 ` Markus Armbruster
2016-09-05  8:34   ` Thomas Huth
2016-09-05 11:05     ` Markus Armbruster
2016-09-05 11:34       ` Thomas Huth
2016-09-05 12:29         ` Markus Armbruster

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.