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

From: Kamil Rytarowski <n54@gmx.com>
To: Sergio Lopez <slp@redhat.com>
Cc: pbonzini@redhat.com, Maxime Villard <maxv@netbsd.org>,
	qemu-devel@nongnu.org, peter.maydell@linaro.org
Subject: Re: [PATCH 0/4] Implements the NetBSD Virtual Machine Monitor accelerator
Date: Tue, 28 Jan 2020 12:11:51 +0100	[thread overview]
Message-ID: <d6078248-b6ca-1c9b-16a4-1e9eaccafa53@gmx.com> (raw)
In-Reply-To: <20200128091007.lh5re4neqhhqmr7x@dritchie>

[-- Attachment #1.1: Type: text/plain, Size: 2237 bytes --]

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.

>  - 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.

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.
> 

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]