Re: [Qemu-devel] We need more reviewers/maintainers!!

[Stefano Stabellini]

On Mon, 12 Mar 2012, Anthony Liguori wrote:
> > OK, so the actual problem seems to be that not all the source files that
> > are supposed to be Supported are actually supported.
> > And of course some key files, like savevm.c are not even Maintained!!
> > For example if I am not mistaken we are missing an entry for vga/cirrus,
> > so that would also be Orphan, correct?
> 
> More likely Odd Fixes.  I think most of the things not listed are Odd Fixes 
> which for something like cirrus is fine.  I doubt it's ever going to see a 
> big 
> invasive change as the code hasn't changed fundamentally in years.
> 
> But I agree, migration is an area that we need a proper maintainer for and 
> hopefully this thread will help make that happen.

Michael Roth demonstrated some interest in the role, maybe we should add
his email address in relation to savevm.c in the MAINTAINERS file.
If not as a maintainer yet, maybe just as a person of interest, I am not
sure whether a tag already exists for that. "M:" in theory just means
"Mail patches to" but in practice are all maintainers. If we want to
keep the separation clear we could introduce a new tag, maybe "I:" ?

It is true that most of the changes to vga.c are simple nowadays but it
would be nice to have somebody in charge of it, even if he would offer
only "Odd Fixes" as level of commitment.
You cannot be in charge of everything, unless you finally manage to make
that cloning machine down in basement work once and for all ;-)
Until then, we need more names in the MAINTAINERS file.
Is there anybody that feel like maintaining vga/cirrus?

Now that I look you are still the only one in charge of pc.c, pc_piix.c
and vl.c.  I think there is room for another x86 machine emulator
maintainer and another vl.c maintainer to help you out with the reviews.
Do you agree?  Who feels up for some x86 love? Anybody that feels up for
taking care of vl.c?