[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/4] Implements the NetBSD Virtual Machine Monitor accelerato
From: |
Kamil Rytarowski |
Subject: |
Re: [PATCH 0/4] Implements the NetBSD Virtual Machine Monitor accelerator |
Date: |
Tue, 28 Jan 2020 13:12:56 +0100 |
User-agent: |
Mozilla/5.0 (X11; NetBSD amd64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 28.01.2020 13:08, Sergio Lopez wrote:
> 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.
>
OK. I will do it.
Please have a look at v2 2/4, submit reviewed-by and I will submit the
whole series again.
> 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: OpenPGP digital signature
- [PATCH 2/4] Add the NetBSD Virtual Machine Monitor accelerator., (continued)