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. >
signature.asc
Description: OpenPGP digital signature
