[Qemu-devel] [PATCH] Update MAINTAINERS

[Qemu-devel] [PATCH] Update MAINTAINERS

[Anthony Liguori]

All of Paul's emails are bouncing and he hasn't been active for
some time.

Signed-off-by: Anthony Liguori <anthony@codemonkey.ws>
---
 MAINTAINERS | 21 +++++----------------
 1 file changed, 5 insertions(+), 16 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 5c3c70c..ab8166a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -51,7 +51,6 @@ Descriptions of section entries:
 General Project Administration
 ------------------------------
 M: Anthony Liguori <anthony@codemonkey.ws>
-M: Paul Brook <paul@codesourcery.com>
 
 Guest CPU cores (TCG):
 ----------------------
@@ -62,7 +61,6 @@ F: target-alpha/
 F: hw/alpha/
 
 ARM
-M: Paul Brook <paul@codesourcery.com>
 M: Peter Maydell <peter.maydell@linaro.org>
 S: Maintained
 F: target-arm/
@@ -83,8 +81,7 @@ F: hw/lm32/
 F: hw/char/lm32_*
 
 M68K
-M: Paul Brook <paul@codesourcery.com>
-S: Odd Fixes
+S: Orphan
 F: target-m68k/
 F: hw/m68k/
 
@@ -248,7 +245,6 @@ F: hw/*/imx*
 F: hw/arm/kzm.c
 
 Integrator CP
-M: Paul Brook <paul@codesourcery.com>
 M: Peter Maydell <peter.maydell@linaro.org>
 S: Maintained
 F: hw/arm/integratorcp.c
@@ -274,7 +270,6 @@ S: Maintained
 F: hw/arm/palm.c
 
 Real View
-M: Paul Brook <paul@codesourcery.com>
 M: Peter Maydell <peter.maydell@linaro.org>
 S: Maintained
 F: hw/arm/realview*
@@ -285,13 +280,11 @@ S: Maintained
 F: hw/arm/spitz.c
 
 Stellaris
-M: Paul Brook <paul@codesourcery.com>
 M: Peter Maydell <peter.maydell@linaro.org>
 S: Maintained
 F: hw/*/stellaris*
 
 Versatile PB
-M: Paul Brook <paul@codesourcery.com>
 M: Peter Maydell <peter.maydell@linaro.org>
 S: Maintained
 F: hw/*/versatile*
@@ -327,18 +320,15 @@ F: hw/lm32/milkymist.c
 M68K Machines
 -------------
 an5206
-M: Paul Brook <paul@codesourcery.com>
-S: Maintained
+S: Orphan
 F: hw/m68k/an5206.c
 
 dummy_m68k
-M: Paul Brook <paul@codesourcery.com>
-S: Maintained
+S: Orphan
 F: hw/m68k/dummy_m68k.c
 
 mcf5208
-M: Paul Brook <paul@codesourcery.com>
-S: Maintained
+S: Orphan
 F: hw/m68k/mcf5208.c
 
 MicroBlaze Machines
@@ -567,8 +557,7 @@ F: hw/scsi/*
 T: git git://github.com/bonzini/qemu.git scsi-next
 
 LSI53C895A
-M: Paul Brook <paul@codesourcery.com>
-S: Odd Fixes
+S: Orphan
 F: hw/scsi/lsi53c895a.c
 
 SSI
-- 
1.8.0

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Andreas Färber]

Am 02.10.2013 19:09, schrieb Anthony Liguori:
> All of Paul's emails are bouncing and he hasn't been active for
> some time.
> 
> Signed-off-by: Anthony Liguori <anthony@codemonkey.ws>
> ---
>  MAINTAINERS | 21 +++++----------------
>  1 file changed, 5 insertions(+), 16 deletions(-)

$ echo -n paul@codesourcery.com | md5sum
e38859654f155fa02b0996bb01b883fe  -

=> http://www.gravatar.com/avatar/e38859654f155fa02b0996bb01b883fe

Same image can be found on
https://github.com/pbrook?tab=activity
which shows recent activity, so he's not on offline vacation.

I had sent an email to the address used in his commit messages (CC'ed)
on Sep 4th asking to update or remove his email address in our
MAINTAINERS file, without response nearly a month later.

Pings on OFTC #qemu remained unanswered, too.

Therefore removal seems the right thing to do under the circumstances,

Reviewed-by: Andreas Färber <afaerber@suse.de>

Thanks.

> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5c3c70c..ab8166a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
[...]
> @@ -567,8 +557,7 @@ F: hw/scsi/*
>  T: git git://github.com/bonzini/qemu.git scsi-next
>  
>  LSI53C895A
> -M: Paul Brook <paul@codesourcery.com>
> -S: Odd Fixes
> +S: Orphan
>  F: hw/scsi/lsi53c895a.c
>  
>  SSI

CC'ing Paolo as SCSI maintainer.

Regards,
Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Paolo Bonzini]

Il 02/10/2013 19:53, Andreas Färber ha scritto:
>> >  LSI53C895A
>> > -M: Paul Brook <paul@codesourcery.com>
>> > -S: Odd Fixes
>> > +S: Orphan
>> >  F: hw/scsi/lsi53c895a.c
>> >  
>> >  SSI
> CC'ing Paolo as SCSI maintainer.

I definitely don't object to this, but it's also fine to just remove the
whole stanza.

Paolo

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Rob Landley]

On 10/02/2013 12:09:58 PM, Anthony Liguori wrote:
> All of Paul's emails are bouncing and he hasn't been active for
> some time.
...
>  M68K
> -M: Paul Brook <paul@codesourcery.com>
> -S: Odd Fixes
> +S: Orphan
>  F: target-m68k/
>  F: hw/m68k/

Laurent Vivier has an m68k gitorious branch to add the q800 target,  
which I've occasionally tested and would really really like to see  
finished and merged.

Alas, last time I tested it the sucker died during the kernel boot as  
soon as mmu setup tried to enable the page tables. But before that I  
got three lines printk'd!  (Woo! Progress!)

Still: possible m68k guy? Maybe? If he's interested?

Rob

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Laurent Vivier]

Le 02/10/2013 20:42, Rob Landley a écrit :
> On 10/02/2013 12:09:58 PM, Anthony Liguori wrote:
>> All of Paul's emails are bouncing and he hasn't been active for
>> some time.
> ...
>>  M68K
>> -M: Paul Brook <paul@codesourcery.com>
>> -S: Odd Fixes
>> +S: Orphan
>>  F: target-m68k/
>>  F: hw/m68k/
>
> Laurent Vivier has an m68k gitorious branch to add the q800 target, 
> which I've occasionally tested and would really really like to see 
> finished and merged.
>
> Alas, last time I tested it the sucker died during the kernel boot as 
> soon as mmu setup tried to enable the page tables. But before that I 
> got three lines printk'd!  (Woo! Progress!)
Now, kernel is able to try to load its first userspace process but fails 
in ld.so somewhere when it is trying  to map process memory (another MMU 
bug...)

> Still: possible m68k guy? Maybe? If he's interested?

Yes, I can do that. There is not a lot of activity on this, anyway.

Regards,
Laurent

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Rob Landley]

On 10/02/2013 06:40:18 PM, Laurent Vivier wrote:
> Le 02/10/2013 20:42, Rob Landley a écrit :
>> Laurent Vivier has an m68k gitorious branch to add the q800 target,  
>> which
>> I've occasionally tested and would really really like to see  
>> finished and
>> merged.
>> 
>> Alas, last time I tested it the sucker died during the kernel boot  
>> as soon
>> as mmu setup tried to enable the page tables. But before that I got  
>> three
>> lines printk'd!  (Woo! Progress!)
> 
> Now, kernel is able to try to load its first userspace process but  
> fails in
> ld.so somewhere when it is trying  to map process memory (another MMU  
> bug...)

Progress!

>> Still: possible m68k guy? Maybe? If he's interested?
> 
> Yes, I can do that. There is not a lot of activity on this, anyway.

I still have m68k images built form current kernels, I just can't test  
them.

I'd poke at it and report bugs, but you know what the bugs are as well  
as I do. I don't know what the thing _should_ be doing, so...

Rob

[Qemu-devel] [PATCH] Update MAINTAINERS

[Anthony Liguori]

The goal of this file is not to establish strict ownership of the code base but
rather to provide a place for people to direct questions to and for names of
people to CC on patches.

I'm basing this update on my observations on how things currently work today.
If you think your name should be on something and it's not, don't read anything
into it and just correct me :-)

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

diff --git a/MAINTAINERS b/MAINTAINERS
index e5165fb..ee84f3f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4,13 +4,12 @@ QEMU Maintainers
 Project leaders:
 ----------------
 
-Fabrice Bellard
 Paul Brook
 
-CPU cores:
+TCG cores:
 ----------
 
-x86                Fabrice Bellard
+x86                ?
 ARM                Paul Brook
 SPARC              Blue Swirl
 MIPS               ?
@@ -26,7 +25,7 @@ Machines (sorted by CPU):
 -------------------------
 
 x86
-  pc.c                    Fabrice Bellard (new maintainer needed)
+  pc.c                    Anthony Liguori
 ARM
   integratorcp.c          Paul Brook
   versatilepb.c           Paul Brook
@@ -70,19 +69,20 @@ S390
 Generic Subsystems:
 -------------------  
 
-Dynamic translator        Fabrice Bellard
-Main loop                 Fabrice Bellard (new maintainer needed)
-TCG                       Fabrice Bellard
-IDE device                ?
-SCSI device               Paul Brook
+Main loop                 Anthony Liguori
+TCG                       ?
+KVM                       Marcelo Tosatti/Avi Kivity
+Virtio devices            Anthony Liguori
+IDE devices               ?
+SCSI devices              Paul Brook
 PCI layer                 Michael S. Tsirkin
 USB layer                 ?
-Block layer               ?
-Graphic layer             ?
+Block layer               Kevin Wolf
+Graphic layer             Anthony Liguori
 Audio device layer        Vassili Karpov (malc)
-Character device layer    ?
-Network device layer      ?
+Character device layer    Anthony Liguori
+Network device layer      Anthony Liguori/Mark McLoughlin
 GDB stub                  ?
-Linux user                ?
-Darwin user               ?
-SLIRP                     ?
+Linux user                Riku Voipio
+Darwin user               Unmaintained (likely to be removed)
+SLIRP                     Unmaintained
-- 
1.7.0.4

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Stefan Weil]

Am 09.09.2010 21:54, schrieb Anthony Liguori:
> The goal of this file is not to establish strict ownership of the code base but
> rather to provide a place for people to direct questions to and for names of
> people to CC on patches.
>    

This text should be added to MAINTAINERS.

> I'm basing this update on my observations on how things currently work today.
> If you think your name should be on something and it's not, don't read anything
> into it and just correct me :-)
>
> Signed-off-by: Anthony Liguori<aliguori@us.ibm.com>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e5165fb..ee84f3f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4,13 +4,12 @@ QEMU Maintainers
>   Project leaders:
>   ----------------
>
> -Fabrice Bellard
>   Paul Brook
>
> -CPU cores:
> +TCG cores:
>   ----------
>
> -x86                Fabrice Bellard
> +x86                ?
>   ARM                Paul Brook
>   SPARC              Blue Swirl
>   MIPS               ?
> @@ -26,7 +25,7 @@ Machines (sorted by CPU):
>   -------------------------
>
>   x86
> -  pc.c                    Fabrice Bellard (new maintainer needed)
> +  pc.c                    Anthony Liguori
>   ARM
>     integratorcp.c          Paul Brook
>     versatilepb.c           Paul Brook
> @@ -70,19 +69,20 @@ S390
>   Generic Subsystems:
>   -------------------
>
> -Dynamic translator        Fabrice Bellard
> -Main loop                 Fabrice Bellard (new maintainer needed)
> -TCG                       Fabrice Bellard
> -IDE device                ?
> -SCSI device               Paul Brook
> +Main loop                 Anthony Liguori
> +TCG                       ?
> +KVM                       Marcelo Tosatti/Avi Kivity
> +Virtio devices            Anthony Liguori
> +IDE devices               ?
> +SCSI devices              Paul Brook
>   PCI layer                 Michael S. Tsirkin
>   USB layer                 ?
> -Block layer               ?
> -Graphic layer             ?
> +Block layer               Kevin Wolf
> +Graphic layer             Anthony Liguori
>   Audio device layer        Vassili Karpov (malc)
> -Character device layer    ?
> -Network device layer      ?
> +Character device layer    Anthony Liguori
> +Network device layer      Anthony Liguori/Mark McLoughlin
>   GDB stub                  ?
> -Linux user                ?
> -Darwin user               ?
> -SLIRP                     ?
> +Linux user                Riku Voipio
> +Darwin user               Unmaintained (likely to be removed)
> +SLIRP                     Unmaintained
>    

Hello Anthony,

I feel responsible for any file with "Copyright (c) 20xx Stefan Weil" in 
the header.

Maybe a general remark like this would be good:

For individual files, there is usually a copyright owner mentioned in 
the file's header.
In most cases, this person also maintains this file.

(or anything similar)

For people who don't know what "Main loop" (and the other topics) is and 
also
for automated parsing (matching of files to maintainers like it is done 
by the linux kernel)
additional path or filename entries would be helpful in a future version.

Regards
Stefan

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Anthony Liguori]

On 09/09/2010 03:12 PM, Stefan Weil wrote:
> Am 09.09.2010 21:54, schrieb Anthony Liguori:
>> The goal of this file is not to establish strict ownership of the 
>> code base but
>> rather to provide a place for people to direct questions to and for 
>> names of
>> people to CC on patches.
>
> This text should be added to MAINTAINERS.
>
>> I'm basing this update on my observations on how things currently 
>> work today.
>> If you think your name should be on something and it's not, don't 
>> read anything
>> into it and just correct me :-)
>>
>> Signed-off-by: Anthony Liguori<aliguori@us.ibm.com>
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index e5165fb..ee84f3f 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -4,13 +4,12 @@ QEMU Maintainers
>>   Project leaders:
>>   ----------------
>>
>> -Fabrice Bellard
>>   Paul Brook
>>
>> -CPU cores:
>> +TCG cores:
>>   ----------
>>
>> -x86                Fabrice Bellard
>> +x86                ?
>>   ARM                Paul Brook
>>   SPARC              Blue Swirl
>>   MIPS               ?
>> @@ -26,7 +25,7 @@ Machines (sorted by CPU):
>>   -------------------------
>>
>>   x86
>> -  pc.c                    Fabrice Bellard (new maintainer needed)
>> +  pc.c                    Anthony Liguori
>>   ARM
>>     integratorcp.c          Paul Brook
>>     versatilepb.c           Paul Brook
>> @@ -70,19 +69,20 @@ S390
>>   Generic Subsystems:
>>   -------------------
>>
>> -Dynamic translator        Fabrice Bellard
>> -Main loop                 Fabrice Bellard (new maintainer needed)
>> -TCG                       Fabrice Bellard
>> -IDE device                ?
>> -SCSI device               Paul Brook
>> +Main loop                 Anthony Liguori
>> +TCG                       ?
>> +KVM                       Marcelo Tosatti/Avi Kivity
>> +Virtio devices            Anthony Liguori
>> +IDE devices               ?
>> +SCSI devices              Paul Brook
>>   PCI layer                 Michael S. Tsirkin
>>   USB layer                 ?
>> -Block layer               ?
>> -Graphic layer             ?
>> +Block layer               Kevin Wolf
>> +Graphic layer             Anthony Liguori
>>   Audio device layer        Vassili Karpov (malc)
>> -Character device layer    ?
>> -Network device layer      ?
>> +Character device layer    Anthony Liguori
>> +Network device layer      Anthony Liguori/Mark McLoughlin
>>   GDB stub                  ?
>> -Linux user                ?
>> -Darwin user               ?
>> -SLIRP                     ?
>> +Linux user                Riku Voipio
>> +Darwin user               Unmaintained (likely to be removed)
>> +SLIRP                     Unmaintained
>
> Hello Anthony,
>
> I feel responsible for any file with "Copyright (c) 20xx Stefan Weil" 
> in the header.

Yeah, I'm thinking of converting to a Linux style maintainers that's 
more conducive to fine-grain info.  Again, my intention is for people 
who are looking to ask for help or patch review for a particular piece 
of code so having more granularity is a good thing.

> Maybe a general remark like this would be good:
>
> For individual files, there is usually a copyright owner mentioned in 
> the file's header.

It's very often that the original copyright holder is no longer involved 
in the project.  Hence the need for something beyond the copyright 
header in the first place.

> In most cases, this person also maintains this file.
>
> (or anything similar)
>
> For people who don't know what "Main loop" (and the other topics) is 
> and also
> for automated parsing (matching of files to maintainers like it is 
> done by the linux kernel)
> additional path or filename entries would be helpful in a future version.

Yes, hence the desire to move to a Linux style format.

Regards,

Anthony Liguori

>
> Regards
> Stefan
>
>

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Blue Swirl]

On Thu, Sep 9, 2010 at 7:54 PM, Anthony Liguori <aliguori@us.ibm.com> wrote:
> The goal of this file is not to establish strict ownership of the code base but
> rather to provide a place for people to direct questions to and for names of
> people to CC on patches.
>
> I'm basing this update on my observations on how things currently work today.
> If you think your name should be on something and it's not, don't read anything
> into it and just correct me :-)
>
> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e5165fb..ee84f3f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4,13 +4,12 @@ QEMU Maintainers
>  Project leaders:
>  ----------------
>
> -Fabrice Bellard
>  Paul Brook

+Anthony Liguori

I think that is how things work today.

>
> -CPU cores:
> +TCG cores:

That could be read as TCG host support. How about 'Guest CPU cores
(TCG)' with 'Guest CPU cores (KVM)' added later?

>  ----------
>
> -x86                Fabrice Bellard
> +x86                ?
>  ARM                Paul Brook
>  SPARC              Blue Swirl
>  MIPS               ?
> @@ -26,7 +25,7 @@ Machines (sorted by CPU):
>  -------------------------
>
>  x86
> -  pc.c                    Fabrice Bellard (new maintainer needed)
> +  pc.c                    Anthony Liguori
>  ARM
>   integratorcp.c          Paul Brook
>   versatilepb.c           Paul Brook
> @@ -70,19 +69,20 @@ S390
>  Generic Subsystems:
>  -------------------
>
> -Dynamic translator        Fabrice Bellard
> -Main loop                 Fabrice Bellard (new maintainer needed)
> -TCG                       Fabrice Bellard
> -IDE device                ?
> -SCSI device               Paul Brook
> +Main loop                 Anthony Liguori
> +TCG                       ?
> +KVM                       Marcelo Tosatti/Avi Kivity
> +Virtio devices            Anthony Liguori
> +IDE devices               ?
> +SCSI devices              Paul Brook
>  PCI layer                 Michael S. Tsirkin
>  USB layer                 ?
> -Block layer               ?
> -Graphic layer             ?
> +Block layer               Kevin Wolf
> +Graphic layer             Anthony Liguori
>  Audio device layer        Vassili Karpov (malc)
> -Character device layer    ?
> -Network device layer      ?
> +Character device layer    Anthony Liguori
> +Network device layer      Anthony Liguori/Mark McLoughlin
>  GDB stub                  ?
> -Linux user                ?
> -Darwin user               ?
> -SLIRP                     ?
> +Linux user                Riku Voipio

+BSD user                Blue Swirl

> +Darwin user               Unmaintained (likely to be removed)
> +SLIRP                     Unmaintained
> --
> 1.7.0.4
>
>
>

Re: [Qemu-devel] [PATCH] Update MAINTAINERS

[Anthony Liguori]

On 09/09/2010 03:12 PM, Blue Swirl wrote:
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index e5165fb..ee84f3f 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -4,13 +4,12 @@ QEMU Maintainers
>>   Project leaders:
>>   ----------------
>>
>> -Fabrice Bellard
>>   Paul Brook
>>      
> +Anthony Liguori
>
> I think that is how things work today.
>    

I don't object to that :-) But I was also considering just removing the 
section or maybe listing "core maintainers" which are the folks that 
have commit access.

>> -CPU cores:
>> +TCG cores:
>>      
> That could be read as TCG host support. How about 'Guest CPU cores
> (TCG)' with 'Guest CPU cores (KVM)' added later?
>    

That makes sense.

>>   ----------
>>
>> -x86                Fabrice Bellard
>> +x86                ?
>>   ARM                Paul Brook
>>   SPARC              Blue Swirl
>>   MIPS               ?
>> @@ -26,7 +25,7 @@ Machines (sorted by CPU):
>>   -------------------------
>>
>>   x86
>> -  pc.c                    Fabrice Bellard (new maintainer needed)
>> +  pc.c                    Anthony Liguori
>>   ARM
>>    integratorcp.c          Paul Brook
>>    versatilepb.c           Paul Brook
>> @@ -70,19 +69,20 @@ S390
>>   Generic Subsystems:
>>   -------------------
>>
>> -Dynamic translator        Fabrice Bellard
>> -Main loop                 Fabrice Bellard (new maintainer needed)
>> -TCG                       Fabrice Bellard
>> -IDE device                ?
>> -SCSI device               Paul Brook
>> +Main loop                 Anthony Liguori
>> +TCG                       ?
>> +KVM                       Marcelo Tosatti/Avi Kivity
>> +Virtio devices            Anthony Liguori
>> +IDE devices               ?
>> +SCSI devices              Paul Brook
>>   PCI layer                 Michael S. Tsirkin
>>   USB layer                 ?
>> -Block layer               ?
>> -Graphic layer             ?
>> +Block layer               Kevin Wolf
>> +Graphic layer             Anthony Liguori
>>   Audio device layer        Vassili Karpov (malc)
>> -Character device layer    ?
>> -Network device layer      ?
>> +Character device layer    Anthony Liguori
>> +Network device layer      Anthony Liguori/Mark McLoughlin
>>   GDB stub                  ?
>> -Linux user                ?
>> -Darwin user               ?
>> -SLIRP                     ?
>> +Linux user                Riku Voipio
>>      
> +BSD user                Blue Swirl
>    

Indeed.

Regards,

Anthony Liguori

>> +Darwin user               Unmaintained (likely to be removed)
>> +SLIRP                     Unmaintained
>> --
>> 1.7.0.4
>>
>>
>>
>>