Re: [PATCH 0/4] Implements the NetBSD Virtual Machine Monitor accelerator

[Sergio Lopez]

On Tue, Jan 28, 2020 at 12:11:51PM +0100, Kamil Rytarowski wrote:
> On 28.01.2020 10:10, Sergio Lopez wrote:
> > On Tue, Jan 07, 2020 at 01:53:57PM +0100, Kamil Rytarowski wrote:
> >> Hello QEMU Community!
> >>
> >> Over the past year the NetBSD team has been working hard on a new 
> >> user-mode API
> >> for our hypervisor that will be released as part of the upcoming NetBSD 
> >> 9.0.
> >> This new API adds user-mode capabilities to create and manage virtual 
> >> machines,
> >> configure memory mappings for guest machines, and create and control 
> >> execution
> >> of virtual processors.
> >>
> >> With this new API we are now able to bring our hypervisor to the QEMU
> >> community! The following patches implement the NetBSD Virtual Machine 
> >> Monitor
> >> accelerator (NVMM) for QEMU on NetBSD 9.0 and newer hosts.
> >>
> >> When compiling QEMU for x86_64 passing the --enable-nvmm flag will compile 
> >> the
> >> accelerator for use. At runtime using the '-accel nvmm' should see a
> >> significant performance improvement over emulation, much like when using 
> >> 'hax'
> >> on NetBSD.
> >>
> >> The documentation for this new API is visible at https://man.netbsd.org 
> >> under
> >> the libnvmm(3) and nvmm(4) pages.
> >>
> >> NVMM was designed and implemented by Maxime Villard.
> >>
> >> Thank you for your feedback.
> > 
> > Thank you for working on this, it's nice to see that QEMU will gain
> > HW-assisted acceleration on NetBSD 9.0. A couple things:
> > 
> 
> Thank you for the review!
> 
> >  - Are you and/or Maxime willing to step up as maintainers for the
> >    NVMM support in QEMU?
> > 
> 
> NVMM (as of today) is part of the NetBSD support and I am the maintainer
> for the NetBSD code (noted in MAINTAINERS).
> 
> In case of non-trivial changes I will reach Maxime for his feedback.

OK, sounds good to me.

> >  - In the next version of the patch series, please use
> >    "scripts/get_maintainer.pl" to get the list of people you need to
> >    CC for the patch series.
> > 
> 
> I've submitted a fixup patch '[PATCH v2 2/4] Add the NetBSD Virtual
> Machine Monitor accelerator.' instead of the full series.
> 
> I have rechecked the maintainers with the patch and I don't see anybody
> else as a candidate for review.

$ ./scripts/get_maintainer.pl /tmp/0004.patch 
Paolo Bonzini <address@hidden> (maintainer:Main loop)
Richard Henderson <address@hidden> (maintainer:Overall TCG CPUs)
Eduardo Habkost <address@hidden> (maintainer:X86 TCG CPUs)
address@hidden (open list:All patches CC here)

$ ./scripts/get_maintainer.pl /tmp/0002.patch 
get_maintainer.pl: No maintainers found, printing recent contributors.
get_maintainer.pl: Do not blindly cc: them on patches!  Use common sense.

"Philippe Mathieu-Daudé" <address@hidden> (commit_signer:31/112=28%)
Paolo Bonzini <address@hidden> (commit_signer:31/112=28%)
Thomas Huth <address@hidden> (commit_signer:25/112=22%,commit_signer:14/53=26%)
"Daniel P. Berrangé" <address@hidden> (commit_signer:24/112=21%)
Laurent Vivier <address@hidden> 
(commit_signer:22/112=20%,commit_signer:10/53=19%)
Gerd Hoffmann <address@hidden> (commit_signer:13/53=25%)
"Michael S. Tsirkin" <address@hidden> (commit_signer:9/53=17%)
Markus Armbruster <address@hidden> (commit_signer:9/53=17%)
address@hidden (open list:All patches CC here)

I think you should resping the whole series as v2, adding Richard,
Eduardo and Philippe to the CC list. That should help with the review
process.

Thanks,
Sergio.

> I don't maintain a merge queue on my own. Please put the reviewed
> patches on a merge queue yourself.
> 
> 
> Thank you in advance,
> 
> > Thanks,
> > Sergio.
> > 
> 
>

signature.asc
Description: PGP signature