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

Stefan Hajnoczi posted 1 patch 3 years, 7 months ago
Test docker-quick@centos7 failed
Test docker-mingw@fedora failed
Test checkpatch failed
Test FreeBSD failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200907111632.90499-1-stefanha@redhat.com
MAINTAINERS | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
[PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Stefan Hajnoczi 3 years, 7 months ago
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
 
 Bootdevice
 M: Gonglei <arei.gonglei@huawei.com>
-- 
2.26.2

Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Kevin Wolf 3 years, 7 months ago
Am 07.09.2020 um 13:16 hat Stefan Hajnoczi geschrieben:
> 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>

Acked-by: Kevin Wolf <kwolf@redhat.com>


Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Klaus Jensen 3 years, 7 months ago
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>
Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Fam Zheng 3 years, 7 months ago
On 2020-09-07 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>

Acked-by: Fam Zheng <fam@euphon.net>

Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Stefan Hajnoczi 3 years, 7 months ago
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
Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Philippe Mathieu-Daudé 3 years, 7 months ago
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.


Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Stefan Hajnoczi 3 years, 7 months ago
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?

Stefan
Re: [PATCH] MAINTAINERS: add Stefan Hajnoczi as block/nvme.c maintainer
Posted by Alex Williamson 3 years, 7 months ago
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