[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 2/4] Add NVMM accelerator: x86 CPU support
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v6 2/4] Add NVMM accelerator: x86 CPU support |
Date: |
Thu, 1 Apr 2021 10:35:40 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.0 |
On 31/03/21 22:07, Reinoud Zandijk wrote:
+void nvmm_vcpu_kick(CPUState *cpu);
Not defined anywhere.
+{
+#if NVMM_USER_VERSION == 1
+ struct sigaction sigact;
+ sigset_t set;
+
+ /* Install the IPI handler. */
+ memset(&sigact, 0, sizeof(sigact));
+ sigact.sa_handler = nvmm_ipi_signal;
+ sigaction(SIG_IPI, &sigact, NULL);
+
+ /* Allow IPIs on the current thread. */
+ sigprocmask(SIG_BLOCK, NULL, &set);
+ sigdelset(&set, SIG_IPI);
+ pthread_sigmask(SIG_SETMASK, &set, NULL);
+#else
+ /*
+ * We use the nvmm_vcpu_stop() mechanism, and don't use signals.
+ * Nothing to do.
+ */
+#endif
Since nvmm_vcpu_stop is very similar to KVM's immediate_exit mechanism,
I think you still need to have a dummy signal handler to kick the VM out
of the run loop *if it is in the kernel*. The signal handler however
can just do nothing.
Also, can you just drop support for NVMM_USER_VERSION == 1?
diff --git a/target/i386/nvmm/meson.build b/target/i386/nvmm/meson.build
new file mode 100644
index 0000000000..c154e78014
--- /dev/null
+++ b/target/i386/nvmm/meson.build
@@ -0,0 +1,4 @@
+i386_softmmu_ss.add(when: 'CONFIG_NVMM', if_true: files(
+ 'nvmm-all.c',
+ 'nvmm-accel-ops.c',
+))
The nvmm library should be added here.
Paolo
- Re: [PATCH v6 2/4] Add NVMM accelerator: x86 CPU support,
Paolo Bonzini <=