Message ID | 20210723193444.133412-2-peterx@redhat.com |
---|---|
State | New |
Headers | show |
Series | memory: Sanity checks memory transaction when releasing BQL | expand |
On 23.07.21 21:34, Peter Xu wrote: > This patch has no functional change, but prepares for moving the function > do_run_on_cpu() into softmmu/cpus.c. It does: > > 1. Move qemu_work_item into hw/core/cpu.h. > 2. Export queue_work_on_cpu()/qemu_work_cond. > > All of them will be used by softmmu/cpus.c later. > > Signed-off-by: Peter Xu <peterx@redhat.com> > --- > cpus-common.c | 11 ++--------- > include/hw/core/cpu.h | 10 +++++++++- > 2 files changed, 11 insertions(+), 10 deletions(-) > > diff --git a/cpus-common.c b/cpus-common.c > index 6e73d3e58d..d814b2439a 100644 > --- a/cpus-common.c > +++ b/cpus-common.c > @@ -27,7 +27,7 @@ > static QemuMutex qemu_cpu_list_lock; > static QemuCond exclusive_cond; > static QemuCond exclusive_resume; > -static QemuCond qemu_work_cond; > +QemuCond qemu_work_cond; > > /* >= 1 if a thread is inside start_exclusive/end_exclusive. Written > * under qemu_cpu_list_lock, read with atomic operations. > @@ -114,14 +114,7 @@ CPUState *qemu_get_cpu(int index) > /* current CPU in the current thread. It is only valid inside cpu_exec() */ > __thread CPUState *current_cpu; > > -struct qemu_work_item { > - QSIMPLEQ_ENTRY(qemu_work_item) node; > - run_on_cpu_func func; > - run_on_cpu_data data; > - bool free, exclusive, done; > -}; > - > -static void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi) > +void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi) > { > qemu_mutex_lock(&cpu->work_mutex); > QSIMPLEQ_INSERT_TAIL(&cpu->work_list, wi, node); > diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h > index bc864564ce..f62ae88524 100644 > --- a/include/hw/core/cpu.h > +++ b/include/hw/core/cpu.h > @@ -243,7 +243,15 @@ typedef union { > > typedef void (*run_on_cpu_func)(CPUState *cpu, run_on_cpu_data data); > > -struct qemu_work_item; > +struct qemu_work_item { > + QSIMPLEQ_ENTRY(qemu_work_item) node; > + run_on_cpu_func func; > + run_on_cpu_data data; > + bool free, exclusive, done; > +}; > + > +void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi); > +extern QemuCond qemu_work_cond; > > #define CPU_UNSET_NUMA_NODE_ID -1 > #define CPU_TRACE_DSTATE_MAX_EVENTS 32 > Reviewed-by: David Hildenbrand <david@redhat.com>
diff --git a/cpus-common.c b/cpus-common.c index 6e73d3e58d..d814b2439a 100644 --- a/cpus-common.c +++ b/cpus-common.c @@ -27,7 +27,7 @@ static QemuMutex qemu_cpu_list_lock; static QemuCond exclusive_cond; static QemuCond exclusive_resume; -static QemuCond qemu_work_cond; +QemuCond qemu_work_cond; /* >= 1 if a thread is inside start_exclusive/end_exclusive. Written * under qemu_cpu_list_lock, read with atomic operations. @@ -114,14 +114,7 @@ CPUState *qemu_get_cpu(int index) /* current CPU in the current thread. It is only valid inside cpu_exec() */ __thread CPUState *current_cpu; -struct qemu_work_item { - QSIMPLEQ_ENTRY(qemu_work_item) node; - run_on_cpu_func func; - run_on_cpu_data data; - bool free, exclusive, done; -}; - -static void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi) +void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi) { qemu_mutex_lock(&cpu->work_mutex); QSIMPLEQ_INSERT_TAIL(&cpu->work_list, wi, node); diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h index bc864564ce..f62ae88524 100644 --- a/include/hw/core/cpu.h +++ b/include/hw/core/cpu.h @@ -243,7 +243,15 @@ typedef union { typedef void (*run_on_cpu_func)(CPUState *cpu, run_on_cpu_data data); -struct qemu_work_item; +struct qemu_work_item { + QSIMPLEQ_ENTRY(qemu_work_item) node; + run_on_cpu_func func; + run_on_cpu_data data; + bool free, exclusive, done; +}; + +void queue_work_on_cpu(CPUState *cpu, struct qemu_work_item *wi); +extern QemuCond qemu_work_cond; #define CPU_UNSET_NUMA_NODE_ID -1 #define CPU_TRACE_DSTATE_MAX_EVENTS 32
This patch has no functional change, but prepares for moving the function do_run_on_cpu() into softmmu/cpus.c. It does: 1. Move qemu_work_item into hw/core/cpu.h. 2. Export queue_work_on_cpu()/qemu_work_cond. All of them will be used by softmmu/cpus.c later. Signed-off-by: Peter Xu <peterx@redhat.com> --- cpus-common.c | 11 ++--------- include/hw/core/cpu.h | 10 +++++++++- 2 files changed, 11 insertions(+), 10 deletions(-)