diff mbox series

MAINTAINERS: Add unmaintained bitmap file to related field

Message ID 20190329202003.30988-1-chen.zhang@intel.com
State New
Headers show
Series MAINTAINERS: Add unmaintained bitmap file to related field | expand

Commit Message

Zhang, Chen March 29, 2019, 8:20 p.m. UTC
From: Zhang Chen <chen.zhang@intel.com>

Add bitmap.h and bitops.h to Dirty Bitmaps field.

Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

Comments

John Snow April 2, 2019, 9:07 p.m. UTC | #1
On 3/29/19 4:20 PM, Zhang Chen wrote:
> From: Zhang Chen <chen.zhang@intel.com>
> 
> Add bitmap.h and bitops.h to Dirty Bitmaps field.
> 
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
> ---
>  MAINTAINERS | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 56139ac8ab..360af3e8ca 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1773,6 +1773,8 @@ S: Supported
>  F: util/hbitmap.c
>  F: block/dirty-bitmap.c
>  F: include/qemu/hbitmap.h
> +F: include/qemu/bitmap.h
> +F: include/qemu/bitops.h
>  F: include/block/dirty-bitmap.h
>  F: tests/test-hbitmap.c
>  F: docs/interop/bitmaps.rst
> 

Actually, these bitmaps aren't the same kind as is used by the block
storage module -- these are a more generic, primitive kind that I have
no association with.
Zhang, Chen April 3, 2019, 1:25 a.m. UTC | #2
> -----Original Message-----
> From: John Snow [mailto:jsnow@redhat.com]
> Sent: Wednesday, April 3, 2019 5:07 AM
> To: Zhang, Chen <chen.zhang@intel.com>; Laurent Vivier <laurent@vivier.eu>;
> qemu-dev <qemu-devel@nongnu.org>; Fam Zheng <fam@euphon.net>
> Subject: Re: [Qemu-devel] [PATCH] MAINTAINERS: Add unmaintained bitmap
> file to related field
> 
> 
> 
> On 3/29/19 4:20 PM, Zhang Chen wrote:
> > From: Zhang Chen <chen.zhang@intel.com>
> >
> > Add bitmap.h and bitops.h to Dirty Bitmaps field.
> >
> > Signed-off-by: Zhang Chen <chen.zhang@intel.com>
> > ---
> >  MAINTAINERS | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS index 56139ac8ab..360af3e8ca
> > 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1773,6 +1773,8 @@ S: Supported
> >  F: util/hbitmap.c
> >  F: block/dirty-bitmap.c
> >  F: include/qemu/hbitmap.h
> > +F: include/qemu/bitmap.h
> > +F: include/qemu/bitops.h
> >  F: include/block/dirty-bitmap.h
> >  F: tests/test-hbitmap.c
> >  F: docs/interop/bitmaps.rst
> >
> 
> Actually, these bitmaps aren't the same kind as is used by the block storage
> module -- these are a more generic, primitive kind that I have no association
> with.

OK, but these two files are lack of maintenance.
Which maintenance area do you think these two files should belong to?


Thanks
Zhang Chen
John Snow April 3, 2019, 6:53 p.m. UTC | #3
On 4/2/19 9:25 PM, Zhang, Chen wrote:
> 
>> -----Original Message-----
>> From: John Snow [mailto:jsnow@redhat.com]
>> Sent: Wednesday, April 3, 2019 5:07 AM
>> To: Zhang, Chen <chen.zhang@intel.com>; Laurent Vivier <laurent@vivier.eu>;
>> qemu-dev <qemu-devel@nongnu.org>; Fam Zheng <fam@euphon.net>
>> Subject: Re: [Qemu-devel] [PATCH] MAINTAINERS: Add unmaintained bitmap
>> file to related field
>>
>>
>>
>> On 3/29/19 4:20 PM, Zhang Chen wrote:
>>> From: Zhang Chen <chen.zhang@intel.com>
>>>
>>> Add bitmap.h and bitops.h to Dirty Bitmaps field.
>>>
>>> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
>>> ---
>>>  MAINTAINERS | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS index 56139ac8ab..360af3e8ca
>>> 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -1773,6 +1773,8 @@ S: Supported
>>>  F: util/hbitmap.c
>>>  F: block/dirty-bitmap.c
>>>  F: include/qemu/hbitmap.h
>>> +F: include/qemu/bitmap.h
>>> +F: include/qemu/bitops.h
>>>  F: include/block/dirty-bitmap.h
>>>  F: tests/test-hbitmap.c
>>>  F: docs/interop/bitmaps.rst
>>>
>>
>> Actually, these bitmaps aren't the same kind as is used by the block storage
>> module -- these are a more generic, primitive kind that I have no association
>> with.
> 
> OK, but these two files are lack of maintenance.
> Which maintenance area do you think these two files should belong to?
> 
> 
> Thanks
> Zhang Chen
> 
> 

Ehm, I'm not sure. Looks like Paolo and Stefan are the active developers
who have touched it the most.

Do the two of you want me to fold this into the bitmaps section so that
it has a point of contact?

--js
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 56139ac8ab..360af3e8ca 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1773,6 +1773,8 @@  S: Supported
 F: util/hbitmap.c
 F: block/dirty-bitmap.c
 F: include/qemu/hbitmap.h
+F: include/qemu/bitmap.h
+F: include/qemu/bitops.h
 F: include/block/dirty-bitmap.h
 F: tests/test-hbitmap.c
 F: docs/interop/bitmaps.rst