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

2013-10-03 Thread Paolo Bonzini
Il 02/10/2013 19:53, Andreas Färber ha scritto:
>> >  LSI53C895A
>> > -M: Paul Brook 
>> > -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

2013-10-02 Thread 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


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

2013-10-02 Thread 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 
-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

2013-10-02 Thread 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 
-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

2013-10-02 Thread 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 
> ---
>  MAINTAINERS | 21 +
>  1 file changed, 5 insertions(+), 16 deletions(-)

$ echo -n p...@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 

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 
> -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

2010-09-09 Thread 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.


  --

-x86Fabrice Bellard
+x86?
  ARMPaul Brook
  SPARC  Blue Swirl
  MIPS   ?
@@ -26,7 +25,7 @@ Machines (sorted by CPU):
  -

  x86
-  pc.cFabrice Bellard (new maintainer needed)
+  pc.cAnthony Liguori
  ARM
   integratorcp.c  Paul Brook
   versatilepb.c   Paul Brook
@@ -70,19 +69,20 @@ S390
  Generic Subsystems:
  ---

-Dynamic translatorFabrice 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 devicesAnthony 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 layerVassili Karpov (malc)
-Character device layer?
-Network device layer  ?
+Character device layerAnthony Liguori
+Network device layer  Anthony Liguori/Mark McLoughlin
  GDB stub  ?
-Linux user?
-Darwin user   ?
-SLIRP ?
+Linux userRiku Voipio
 

+BSD userBlue Swirl
   


Indeed.

Regards,

Anthony Liguori


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



 





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

2010-09-09 Thread 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

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:
  --

-x86Fabrice Bellard
+x86?
  ARMPaul Brook
  SPARC  Blue Swirl
  MIPS   ?
@@ -26,7 +25,7 @@ Machines (sorted by CPU):
  -

  x86
-  pc.cFabrice Bellard (new maintainer needed)
+  pc.cAnthony Liguori
  ARM
integratorcp.c  Paul Brook
versatilepb.c   Paul Brook
@@ -70,19 +69,20 @@ S390
  Generic Subsystems:
  ---

-Dynamic translatorFabrice 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 devicesAnthony 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 layerVassili Karpov (malc)
-Character device layer?
-Network device layer  ?
+Character device layerAnthony Liguori
+Network device layer  Anthony Liguori/Mark McLoughlin
  GDB stub  ?
-Linux user?
-Darwin user   ?
-SLIRP ?
+Linux userRiku 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

2010-09-09 Thread Blue Swirl
On Thu, Sep 9, 2010 at 7:54 PM, Anthony Liguori  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 
>
> 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 userBlue Swirl

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


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

2010-09-09 Thread 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

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:
  --

-x86Fabrice Bellard
+x86?
  ARMPaul Brook
  SPARC  Blue Swirl
  MIPS   ?
@@ -26,7 +25,7 @@ Machines (sorted by CPU):
  -

  x86
-  pc.cFabrice Bellard (new maintainer needed)
+  pc.cAnthony Liguori
  ARM
integratorcp.c  Paul Brook
versatilepb.c   Paul Brook
@@ -70,19 +69,20 @@ S390
  Generic Subsystems:
  ---

-Dynamic translatorFabrice 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 devicesAnthony 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 layerVassili Karpov (malc)
-Character device layer?
-Network device layer  ?
+Character device layerAnthony Liguori
+Network device layer  Anthony Liguori/Mark McLoughlin
  GDB stub  ?
-Linux user?
-Darwin user   ?
-SLIRP ?
+Linux userRiku 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