diff mbox series

MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer

Message ID 20200907111632.90499-1-stefanha@redhat.com
State New
Headers show
Series MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer | expand

Commit Message

Stefan Hajnoczi Sept. 7, 2020, 11:16 a.m. UTC
Development of the userspace NVMe block driver picked up again recently.
After talking with Fam I am stepping up as block/nvme.c maintainer.
Patches will be merged through my 'block' tree.

Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Klaus Jensen <k.jensen@samsung.com>
Cc: Fam Zheng <fam@euphon.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
 MAINTAINERS | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Klaus Jensen Sept. 7, 2020, 11:46 a.m. UTC | #1
On Sep  7 12:16, Stefan Hajnoczi wrote:
> Development of the userspace NVMe block driver picked up again recently.
> After talking with Fam I am stepping up as block/nvme.c maintainer.
> Patches will be merged through my 'block' tree.
> 
> Cc: Kevin Wolf <kwolf@redhat.com>
> Cc: Klaus Jensen <k.jensen@samsung.com>
> Cc: Fam Zheng <fam@euphon.net>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

I'm only superficially involved in block/nvme, so my Ack doesn't carry
that much weight, but

Acked-by: Klaus Jensen <k.jensen@samsung.com>
Stefan Hajnoczi Sept. 8, 2020, 2:10 p.m. UTC | #2
On Mon, Sep 07, 2020 at 12:16:32PM +0100, Stefan Hajnoczi wrote:
> Development of the userspace NVMe block driver picked up again recently.
> After talking with Fam I am stepping up as block/nvme.c maintainer.
> Patches will be merged through my 'block' tree.
> 
> Cc: Kevin Wolf <kwolf@redhat.com>
> Cc: Klaus Jensen <k.jensen@samsung.com>
> Cc: Fam Zheng <fam@euphon.net>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
>  MAINTAINERS | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block

Stefan
Philippe Mathieu-Daudé Sept. 9, 2020, 12:51 p.m. UTC | #3
Hi Stefan,

+Alex

On 9/7/20 1:16 PM, Stefan Hajnoczi wrote:
> Development of the userspace NVMe block driver picked up again recently.
> After talking with Fam I am stepping up as block/nvme.c maintainer.
> Patches will be merged through my 'block' tree.
> 
> Cc: Kevin Wolf <kwolf@redhat.com>
> Cc: Klaus Jensen <k.jensen@samsung.com>
> Cc: Fam Zheng <fam@euphon.net>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
>  MAINTAINERS | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b233da2a73..a143941551 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2895,10 +2895,12 @@ S: Supported
>  F: block/null.c
>  
>  NVMe Block Driver
> -M: Fam Zheng <fam@euphon.net>
> +M: Stefan Hajnoczi <stefanha@redhat.com>
> +R: Fam Zheng <fam@euphon.net>
>  L: qemu-block@nongnu.org
>  S: Supported
>  F: block/nvme*
> +T: git https://github.com/stefanha/qemu.git block

As this driver is the only consumer of the 'VFIO helpers',
maybe we can:

- maintains them in the same bucket
- add another entry (eventually with R: tag for Alex)

The 'VFIO helpers' files are:
- util/vfio-helpers.c
- include/qemu/vfio-helpers.h

What do you think?

Regards,

Phil.
Alex Williamson Sept. 14, 2020, 3:32 p.m. UTC | #4
On Mon, 14 Sep 2020 11:37:03 +0100
Stefan Hajnoczi <stefanha@gmail.com> wrote:

> On Wed, Sep 09, 2020 at 02:51:50PM +0200, Philippe Mathieu-Daudé wrote:

> > On 9/7/20 1:16 PM, Stefan Hajnoczi wrote:  

> > > Development of the userspace NVMe block driver picked up again recently.

> > > After talking with Fam I am stepping up as block/nvme.c maintainer.

> > > Patches will be merged through my 'block' tree.

> > > 

> > > Cc: Kevin Wolf <kwolf@redhat.com>

> > > Cc: Klaus Jensen <k.jensen@samsung.com>

> > > Cc: Fam Zheng <fam@euphon.net>

> > > Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

> > > ---

> > >  MAINTAINERS | 4 +++-

> > >  1 file changed, 3 insertions(+), 1 deletion(-)

> > > 

> > > diff --git a/MAINTAINERS b/MAINTAINERS

> > > index b233da2a73..a143941551 100644

> > > --- a/MAINTAINERS

> > > +++ b/MAINTAINERS

> > > @@ -2895,10 +2895,12 @@ S: Supported

> > >  F: block/null.c

> > >  

> > >  NVMe Block Driver

> > > -M: Fam Zheng <fam@euphon.net>

> > > +M: Stefan Hajnoczi <stefanha@redhat.com>

> > > +R: Fam Zheng <fam@euphon.net>

> > >  L: qemu-block@nongnu.org

> > >  S: Supported

> > >  F: block/nvme*

> > > +T: git https://github.com/stefanha/qemu.git block  

> > 

> > As this driver is the only consumer of the 'VFIO helpers',

> > maybe we can:

> > 

> > - maintains them in the same bucket

> > - add another entry (eventually with R: tag for Alex)

> > 

> > The 'VFIO helpers' files are:

> > - util/vfio-helpers.c

> > - include/qemu/vfio-helpers.h

> > 

> > What do you think?  

> 

> I'm happy to include vfio-helpers with the goal of eventually switching

> to vfio-common.

> 

> Alex: does this sound good?


Yep, ok by me.  Thanks,

Alex
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index b233da2a73..a143941551 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2895,10 +2895,12 @@  S: Supported
 F: block/null.c
 
 NVMe Block Driver
-M: Fam Zheng <fam@euphon.net>
+M: Stefan Hajnoczi <stefanha@redhat.com>
+R: Fam Zheng <fam@euphon.net>
 L: qemu-block@nongnu.org
 S: Supported
 F: block/nvme*
+T: git https://github.com/stefanha/qemu.git block
 
 Bootdevice
 M: Gonglei <arei.gonglei@huawei.com>