[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v9 02/74] cpu: rename cpu->work_mutex to cpu->lock
From: |
Robert Foley |
Subject: |
[PATCH v9 02/74] cpu: rename cpu->work_mutex to cpu->lock |
Date: |
Thu, 21 May 2020 12:38:59 -0400 |
From: "Emilio G. Cota" <address@hidden>
This lock will soon protect more fields of the struct. Give
it a more appropriate name.
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 | 14 +++++++-------
cpus.c | 4 ++--
hw/core/cpu.c | 4 ++--
include/hw/core/cpu.h | 6 ++++--
4 files changed, 15 insertions(+), 13 deletions(-)
diff --git a/cpus-common.c b/cpus-common.c
index 210fc7fc39..34025923e3 100644
--- a/cpus-common.c
+++ b/cpus-common.c
@@ -105,10 +105,10 @@ struct qemu_work_item {
static void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi)
{
- qemu_mutex_lock(&cpu->work_mutex);
+ qemu_mutex_lock(&cpu->lock);
QSIMPLEQ_INSERT_TAIL(&cpu->work_list, wi, node);
wi->done = false;
- qemu_mutex_unlock(&cpu->work_mutex);
+ qemu_mutex_unlock(&cpu->lock);
qemu_cpu_kick(cpu);
}
@@ -300,15 +300,15 @@ void process_queued_cpu_work(CPUState *cpu)
{
struct qemu_work_item *wi;
- qemu_mutex_lock(&cpu->work_mutex);
+ qemu_mutex_lock(&cpu->lock);
if (QSIMPLEQ_EMPTY(&cpu->work_list)) {
- qemu_mutex_unlock(&cpu->work_mutex);
+ qemu_mutex_unlock(&cpu->lock);
return;
}
while (!QSIMPLEQ_EMPTY(&cpu->work_list)) {
wi = QSIMPLEQ_FIRST(&cpu->work_list);
QSIMPLEQ_REMOVE_HEAD(&cpu->work_list, node);
- qemu_mutex_unlock(&cpu->work_mutex);
+ qemu_mutex_unlock(&cpu->lock);
if (wi->exclusive) {
/* Running work items outside the BQL avoids the following
deadlock:
* 1) start_exclusive() is called with the BQL taken while another
@@ -324,13 +324,13 @@ void process_queued_cpu_work(CPUState *cpu)
} else {
wi->func(cpu, wi->data);
}
- qemu_mutex_lock(&cpu->work_mutex);
+ qemu_mutex_lock(&cpu->lock);
if (wi->free) {
g_free(wi);
} else {
atomic_mb_set(&wi->done, true);
}
}
- qemu_mutex_unlock(&cpu->work_mutex);
+ qemu_mutex_unlock(&cpu->lock);
qemu_cond_broadcast(&qemu_work_cond);
}
diff --git a/cpus.c b/cpus.c
index af44027549..77a2d7b7cf 100644
--- a/cpus.c
+++ b/cpus.c
@@ -101,9 +101,9 @@ static inline bool cpu_work_list_empty(CPUState *cpu)
{
bool ret;
- qemu_mutex_lock(&cpu->work_mutex);
+ qemu_mutex_lock(&cpu->lock);
ret = QSIMPLEQ_EMPTY(&cpu->work_list);
- qemu_mutex_unlock(&cpu->work_mutex);
+ qemu_mutex_unlock(&cpu->lock);
return ret;
}
diff --git a/hw/core/cpu.c b/hw/core/cpu.c
index 77703d62b7..4d58399cfc 100644
--- a/hw/core/cpu.c
+++ b/hw/core/cpu.c
@@ -367,7 +367,7 @@ static void cpu_common_initfn(Object *obj)
cpu->nr_cores = 1;
cpu->nr_threads = 1;
- qemu_mutex_init(&cpu->work_mutex);
+ qemu_mutex_init(&cpu->lock);
QSIMPLEQ_INIT(&cpu->work_list);
QTAILQ_INIT(&cpu->breakpoints);
QTAILQ_INIT(&cpu->watchpoints);
@@ -379,7 +379,7 @@ static void cpu_common_finalize(Object *obj)
{
CPUState *cpu = CPU(obj);
- qemu_mutex_destroy(&cpu->work_mutex);
+ qemu_mutex_destroy(&cpu->lock);
}
static int64_t cpu_common_get_arch_id(CPUState *cpu)
diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h
index d78ff1d165..873bf23a39 100644
--- a/include/hw/core/cpu.h
+++ b/include/hw/core/cpu.h
@@ -331,7 +331,8 @@ struct qemu_work_item;
* @opaque: User data.
* @mem_io_pc: Host Program Counter at which the memory was accessed.
* @kvm_fd: vCPU file descriptor for KVM.
- * @work_mutex: Lock to prevent multiple access to @work_list.
+ * @lock: Lock to prevent multiple access to per-CPU fields. Must be acquired
+ * after the BQL.
* @work_list: List of pending asynchronous work.
* @trace_dstate_delayed: Delayed changes to trace_dstate (includes all changes
* to @trace_dstate).
@@ -375,7 +376,8 @@ struct CPUState {
uint64_t random_seed;
sigjmp_buf jmp_env;
- QemuMutex work_mutex;
+ QemuMutex lock;
+ /* fields below protected by @lock */
QSIMPLEQ_HEAD(, qemu_work_item) work_list;
CPUAddressSpace *cpu_ases;
--
2.17.1
- [PATCH v9 00/74] per-CPU locks, Robert Foley, 2020/05/21
- [PATCH v9 01/74] cpu: convert queued work to a QSIMPLEQ, Robert Foley, 2020/05/21
- [PATCH v9 02/74] cpu: rename cpu->work_mutex to cpu->lock,
Robert Foley <=
- [PATCH v9 03/74] cpu: introduce cpu_mutex_lock/unlock, Robert Foley, 2020/05/21
- [PATCH v9 04/74] cpu: make qemu_work_cond per-cpu, Robert Foley, 2020/05/21
- [PATCH v9 05/74] cpu: move run_on_cpu to cpus-common, Robert Foley, 2020/05/21
- [PATCH v9 06/74] cpu: introduce process_queued_cpu_work_locked, Robert Foley, 2020/05/21
- [PATCH v9 07/74] cpu: make per-CPU locks an alias of the BQL in TCG rr mode, Robert Foley, 2020/05/21
- [PATCH v9 08/74] tcg-runtime: define helper_cpu_halted_set, Robert Foley, 2020/05/21
- [PATCH v9 09/74] ppc: convert to helper_cpu_halted_set, Robert Foley, 2020/05/21
- [PATCH v9 10/74] cris: convert to helper_cpu_halted_set, Robert Foley, 2020/05/21
- [PATCH v9 12/74] m68k: convert to helper_cpu_halted_set, Robert Foley, 2020/05/21