[FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi

[Eoff, Ullysses A]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Michael Niedermayer
> Sent: Tuesday, December 14, 2021 8:30 AM
> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi
> 
> On Tue, Dec 14, 2021 at 04:04:39PM +0000, Eoff, Ullysses A wrote:
> > > -----Original Message-----
> > > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Soft Works
> > > Sent: Monday, December 13, 2021 10:56 PM
> > > To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> > > Subject: Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of U. Artie
> > > > Eoff
> > > > Sent: Tuesday, December 14, 2021 5:08 AM
> > > > To: ffmpeg-devel at ffmpeg.org
> > > > Cc: U. Artie Eoff <ullysses.a.eoff at intel.com>
> > > > Subject: [FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi
> > > >
> > > > Current listed maintainers for vaapi plugin are
> > > > not reponsive and/or currently active in the
> > > > ffmpeg community.  Thus, vaapi plugin patches
> > > > (and qsv plugin) have generally gone ignored or
> > > > lost in the ether for too long.
> > > >
> > > > Remove Gwenole Beauchesne from vaapi maintainer
> > > > who has not been active since 2016.
> > > >
> > > > Current alternative maintainer for vaapi is Mark
> > > > Thompson whom has not been active since
> > > > March/April 2021.
> > > >
> > > > Therefore, add Haihao Xiang to vaapi maintainer
> > > > who's primary role is FFmpeg development with a
> > > > focus on the vaapi and qsv plugins.  Haihao has
> > > > over a decade of media experience and many years
> > > > of FFmpeg development experience, amongst other
> > > > media frameworks.
> > > >
> > > > The additional patch for adding Haihao as qsv
> > > > plugin maintainer has been submitted previously:
> > > >
> > > > https://patchwork.ffmpeg.org/project/ffmpeg/patch/20210608141134.27448-1-
> > > > zhongli_dev at 126.com/
> > > >
> > > > This will help FFmpeg to continue to be the leading
> > > > multimedia framework by allowing these plugins to be
> > > > actively improved, enhanced, and maintained for existing
> > > > and future HW platforms.
> > > >
> > > > Signed-off-by: U. Artie Eoff <ullysses.a.eoff at intel.com>
> > > > ---
> > > >  MAINTAINERS | 4 ++--
> > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index dcac46003ecd..fddf065a3abb 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -274,8 +274,8 @@ Hardware acceleration:
> > > >    dxva2*                                Hendrik Leppkes, Laurent Aimar,
> > > > Steve Lhomme
> > > >    d3d11va*                              Steve Lhomme
> > > >    mediacodec*                           Matthieu Bouron, Aman Gupta
> > > > -  vaapi*                                Gwenole Beauchesne
> > > > -  vaapi_encode*                         Mark Thompson
> > > > +  vaapi*                                Haihao Xiang
> > > > +  vaapi_encode*                         Mark Thompson, Haihao Xiang
> > > >    vdpau*                                Philip Langdale, Carl Eugen Hoyos
> > > >    videotoolbox*                         Rick Kern, Aman Gupta
> > > >
> > > > --
> > >
> > > Whatever it might count: This has my support. Haihao is the
> > > most suitable candidate from all who I've been working with on
> > > QSV issues (on and off-list) in the past two years.
> > >
> > > The current situation is far from ideal. In other areas, commits are
> > > pushed timely and frequently, while there's hardly any progress made
> > > in the Intel area, despite the amount of developer resources they
> > > are providing.
> > > Even the most trivial and straightforward patches are lying around
> > > for months without action. It really shouldn't be like that.
> > >
> > > @Artie - you might want to update the patch and also add him for QSV.
> > >
> >
> > @softworkz, thank you for your kind support.  There is already another
> > patch submitted to add Haihao for QSV... see link in my commit
> > message.  I have also just bumped that patch again.
> >
> > We're looking forward to seeing some progress here 😉.
> 
> Has anyone approved it, especially the people previously listed
> Has anyone objected?
> Where the previous sole maintainers CC-ed on this change ?

I have not seen any objections.
I just added Mark to CC using their email found on this ML.
Unfortunately, we don't have current email contact for
Gwenole (whom has not worked on ffmpeg for ~6 years).
Who else can make the approval, aside from the inactive people
previously listed?

> 
> thx
> 
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> The educated differ from the uneducated as much as the living from the
> dead. -- Aristotle