[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v8 72/74] cpus-common: release BQL earlier in run_on_cpu
From: |
Robert Foley |
Subject: |
[PATCH v8 72/74] cpus-common: release BQL earlier in run_on_cpu |
Date: |
Thu, 26 Mar 2020 15:31:54 -0400 |
From: "Emilio G. Cota" <address@hidden>
After completing the conversion to per-CPU locks, there is no need
to release the BQL after having called cpu_kick.
Reviewed-by: Richard Henderson <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Emilio G. Cota <address@hidden>
Signed-off-by: Robert Foley <address@hidden>
---
cpus-common.c | 20 +++++---------------
1 file changed, 5 insertions(+), 15 deletions(-)
diff --git a/cpus-common.c b/cpus-common.c
index 9ca025149e..f5daf2d518 100644
--- a/cpus-common.c
+++ b/cpus-common.c
@@ -137,6 +137,11 @@ void run_on_cpu(CPUState *cpu, run_on_cpu_func func,
run_on_cpu_data data)
return;
}
+ /* We are going to sleep on the CPU lock, so release the BQL */
+ if (has_bql) {
+ qemu_mutex_unlock_iothread();
+ }
+
wi.func = func;
wi.data = data;
wi.done = false;
@@ -145,21 +150,6 @@ void run_on_cpu(CPUState *cpu, run_on_cpu_func func,
run_on_cpu_data data)
cpu_mutex_lock(cpu);
queue_work_on_cpu_locked(cpu, &wi);
-
- /*
- * We are going to sleep on the CPU lock, so release the BQL.
- *
- * During the transition to per-CPU locks, we release the BQL _after_
- * having kicked the destination CPU (from queue_work_on_cpu_locked above).
- * This makes sure that the enqueued work will be seen by the CPU
- * after being woken up from the kick, since the CPU sleeps on the BQL.
- * Once we complete the transition to per-CPU locks, we will release
- * the BQL earlier in this function.
- */
- if (has_bql) {
- qemu_mutex_unlock_iothread();
- }
-
while (!atomic_mb_read(&wi.done)) {
CPUState *self_cpu = current_cpu;
--
2.17.1
- [PATCH v8 61/74] cpu: convert to interrupt_request, (continued)
- [PATCH v8 61/74] cpu: convert to interrupt_request, Robert Foley, 2020/03/26
- [PATCH v8 62/74] cpu: call .cpu_has_work with the CPU lock held, Robert Foley, 2020/03/26
- [PATCH v8 63/74] cpu: introduce cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 64/74] ppc: convert to cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 65/74] mips: convert to cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 66/74] s390x: convert to cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 67/74] riscv: convert to cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 68/74] sparc: convert to cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 69/74] xtensa: convert to cpu_has_work_with_iothread_lock, Robert Foley, 2020/03/26
- [PATCH v8 70/74] cpu: rename all_cpu_threads_idle to qemu_tcg_rr_all_cpu_threads_idle, Robert Foley, 2020/03/26
- [PATCH v8 72/74] cpus-common: release BQL earlier in run_on_cpu,
Robert Foley <=
- [PATCH v8 71/74] cpu: protect CPU state with cpu->lock instead of the BQL, Robert Foley, 2020/03/26
- [PATCH v8 73/74] cpu: add async_run_on_cpu_no_bql, Robert Foley, 2020/03/26
- [PATCH v8 74/74] cputlb: queue async flush jobs without the BQL, Robert Foley, 2020/03/26
- Re: [PATCH v8 00/74] per-CPU locks, Aleksandar Markovic, 2020/03/26
Re: [PATCH v8 00/74] per-CPU locks, Emilio G. Cota, 2020/03/27