PR/55406 CVS commit: [netbsd-9] src/sys

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR kern/55406; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/55406 CVS commit: [netbsd-9] src/sys
Date: Sun, 2 Aug 2020 08:49:08 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Sun Aug  2 08:49:08 UTC 2020
 
 Modified Files:
 	src/sys/dev/nvmm [netbsd-9]: files.nvmm nvmm.c nvmm_internal.h
 	src/sys/dev/nvmm/x86 [netbsd-9]: nvmm_x86_svm.c nvmm_x86_vmx.c
 	src/sys/modules/nvmm [netbsd-9]: nvmm.ioconf
 
 Log Message:
 Pull up following revision(s) (requested by maxv in ticket #1032):
 
 	sys/dev/nvmm/x86/nvmm_x86_vmx.c: revision 1.60 (patch)
 	sys/dev/nvmm/x86/nvmm_x86_vmx.c: revision 1.61 (patch)
 	sys/dev/nvmm/nvmm.c: revision 1.30
 	sys/dev/nvmm/nvmm.c: revision 1.31
 	sys/dev/nvmm/nvmm.c: revision 1.32
 	sys/dev/nvmm/nvmm_internal.h: revision 1.15
 	sys/dev/nvmm/nvmm_internal.h: revision 1.16
 	sys/dev/nvmm/files.nvmm: revision 1.3
 	sys/dev/nvmm/x86/nvmm_x86_svm.c: revision 1.62 (patch)
 	sys/dev/nvmm/x86/nvmm_x86_svm.c: revision 1.63 (patch)
 	sys/dev/nvmm/x86/nvmm_x86_vmx.c: revision 1.59 (patch)
 	sys/modules/nvmm/nvmm.ioconf: revision 1.2
 
 Gather the conditions to return from the VCPU loops in nvmm_return_needed(),
 and use it in nvmm_do_vcpu_run() as well. This fixes two undesired behaviors:
 
  - When a VM initializes, the many nested page faults that need processing
    could cause the calling thread to occupy the CPU too much if we're unlucky
    and are only getting repeated nested page faults thousands of times in a
    row.
 
  - When the emulator calls nvmm_vcpu_run() and immediately sends a signal to
    stop the VCPU, it's better to check signals earlier and leave right away,
    rather than doing a round of VCPU run that could increase the time spent
    by the emulator waiting for the return.
 
 style
 
 Register NVMM as an actual pseudo-device. Without PMF handler, to
 explicitly disallow ACPI suspend if NVMM is running.
 
 Should fix PR/55406.
 
 Print the backend name when attaching.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.2 -r1.2.6.1 src/sys/dev/nvmm/files.nvmm
 cvs rdiff -u -r1.22.2.4 -r1.22.2.5 src/sys/dev/nvmm/nvmm.c
 cvs rdiff -u -r1.12.2.2 -r1.12.2.3 src/sys/dev/nvmm/nvmm_internal.h
 cvs rdiff -u -r1.46.4.5 -r1.46.4.6 src/sys/dev/nvmm/x86/nvmm_x86_svm.c
 cvs rdiff -u -r1.36.2.7 -r1.36.2.8 src/sys/dev/nvmm/x86/nvmm_x86_vmx.c
 cvs rdiff -u -r1.1 -r1.1.8.1 src/sys/modules/nvmm/nvmm.ioconf
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.