Am 08.01.2019 um 18:18 hat Markus Armbruster geschrieben: > This patch series got stuck. > > Markus Armbruster <[email protected]> writes: > > > Fam Zheng <[email protected]> writes: > > > >> On Tue, 09/25 07:00, Markus Armbruster wrote: > >>> Jeff Cody <[email protected]> writes: > >>> > >>> > I'll not be involved in day-to-day qemu development. Remove > >>> > myself as maintainer from the remainder of the network block drivers > >>> > (and vhdx), and revert them to the general block layer maintainership. > >>> > > >>> > Signed-off-by: Jeff Cody <[email protected]> > >>> > --- > >>> > MAINTAINERS | 14 -------------- > >>> > 1 file changed, 14 deletions(-) > >>> > > >>> > diff --git a/MAINTAINERS b/MAINTAINERS > >>> > index e93f79672f..6ef6932628 100644 > >>> > --- a/MAINTAINERS > >>> > +++ b/MAINTAINERS > >>> > @@ -1982,28 +1982,22 @@ F: block/vmdk.c > >>> > > >>> > RBD > >>> > M: Josh Durgin <[email protected]> > >>> > -M: Jeff Cody <[email protected]> > >>> > L: [email protected] > >>> > S: Supported > >>> > F: block/rbd.c > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > > >>> > Sheepdog > >>> > M: Hitoshi Mitake <[email protected]> > >>> > M: Liu Yuan <[email protected]> > >>> > -M: Jeff Cody <[email protected]> > >>> > L: [email protected] > >>> > L: [email protected] > >>> > S: Supported > >>> > F: block/sheepdog.c > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > > >>> > VHDX > >>> > -M: Jeff Cody <[email protected]> > >>> > L: [email protected] > >>> > S: Supported > >>> > F: block/vhdx* > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > >>> Does "S: Supported" make sense without an M:? > >>> > >>> > > >>> > VDI > >>> > M: Stefan Weil <[email protected]> > >>> > @@ -2034,34 +2028,26 @@ F: docs/interop/nbd.txt > >>> > T: git git://repo.or.cz/qemu/ericb.git nbd > >>> > > >>> > NFS > >>> > -M: Jeff Cody <[email protected]> > >>> > M: Peter Lieven <[email protected]> > >>> > L: [email protected] > >>> > S: Maintained > >>> > F: block/nfs.c > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > > >>> > SSH > >>> > M: Richard W.M. Jones <[email protected]> > >>> > -M: Jeff Cody <[email protected]> > >>> > L: [email protected] > >>> > S: Supported > >>> > F: block/ssh.c > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > > >>> > CURL > >>> > -M: Jeff Cody <[email protected]> > >>> > L: [email protected] > >>> > S: Supported > >>> > F: block/curl.c > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > >>> Likewise. > >>> > >>> > GLUSTER > >>> > -M: Jeff Cody <[email protected]> > >>> > L: [email protected] > >>> > S: Supported > >>> > F: block/gluster.c > >>> > -T: git git://github.com/codyprime/qemu-kvm-jtc.git block > >>> > >>> Likewise. > >>> > >>> > Null Block Driver > >>> > M: Fam Zheng <[email protected]> > >>> > >> > >> Block drivers without an M: naturally fall under the overall > >> maintainership of > >> block layer (Kevin&Max), so IMO keeping the statuses is fine. Maybe CURL > >> can be > >> degraded to Maintained, though. > > > > Yes, get_maintainer.pl combines all applicable sections. Human readers > > are left to wonder, unless they know to look for other matches. > > > > Do we want to have a dedicated network block driver submaintainer again, > > if we can find one? > > > > Do we want to have a dedicated VHDX driver submaintainer again? Fam, > > you're maintaining VMDK, could you cover VHDX as well? > > Fam can't. > > I can see two sane options for the three sections that lose their sole > maintainer (VHDX, CURL, GLUSTER): > > * Downgrade to S: Orphan > > This reflects the fact that we'd like to have dedicated maintainers > for them. It camouflages the fact that the "Block layer core" > maintainers pick up the slack. > > * Delete > > The opposite. > > Kevin, Max, please pick your poison, or suggest one you find tastier.
Orphan is probably by far closer to the truth than Supported. But I think what we really do is Odd Fixes. Not sure if that's a status that works in any meaningful way without having a maintainer for the specific thing assigned? Kevin
