[Qemu-devel] [PATCH v6 0/4] qemu-thread: support --enable-debug-mutex

Peter Xu posted 4 patches 6 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180425025459.5258-1-peterx@redhat.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test s390x passed
configure                   | 11 +++++++++
include/qemu/thread-posix.h |  4 ++++
include/qemu/thread-win32.h |  4 ++++
util/qemu-thread-common.h   | 55 +++++++++++++++++++++++++++++++++++++++++++++
tests/atomic_add-bench.c    | 19 ++++++++++++++--
util/qemu-thread-posix.c    | 19 +++++++---------
util/qemu-thread-win32.c    | 17 +++++++-------
7 files changed, 107 insertions(+), 22 deletions(-)
create mode 100644 util/qemu-thread-common.h
[Qemu-devel] [PATCH v6 0/4] qemu-thread: support --enable-debug-mutex
Posted by Peter Xu 6 years ago
v6:
- rename __QEMU_THREAD_COMMON_H__ to QEMU_THREAD_COMMON_H
- collect r-bs for Emilio

v5:
- init file/line [Emilio]

v4:
- cherrypick Emilio's patch to add "-m" parameter to atomic_add-bench
  tool.
- use inline functions [Emilio]

Micro benchmark result on "taskset -c 0 atomic_add-bench -m":

|--------+---------------------+------------------------|
|        | with --enable-debug | without --enable-debug |
|--------+---------------------+------------------------|
| Before | 20 Mops/s           | 34 Mops/s              |
| After  | 18 Mops/s           | 34 Mops/s              |
|--------+---------------------+------------------------|

Please review.  Thanks.

Emilio G. Cota (1):
  tests/atomic_add-bench: add -m option to use mutexes

Peter Xu (3):
  qemu-thread: introduce qemu-thread-common.h
  QemuMutex: support --enable-debug-mutex
  configure: enable debug-mutex if debug enabled

 configure                   | 11 +++++++++
 include/qemu/thread-posix.h |  4 ++++
 include/qemu/thread-win32.h |  4 ++++
 util/qemu-thread-common.h   | 55 +++++++++++++++++++++++++++++++++++++++++++++
 tests/atomic_add-bench.c    | 19 ++++++++++++++--
 util/qemu-thread-posix.c    | 19 +++++++---------
 util/qemu-thread-win32.c    | 17 +++++++-------
 7 files changed, 107 insertions(+), 22 deletions(-)
 create mode 100644 util/qemu-thread-common.h

-- 
2.14.3


Re: [Qemu-devel] [PATCH v6 0/4] qemu-thread: support --enable-debug-mutex
Posted by Peter Xu 5 years, 11 months ago
On Wed, Apr 25, 2018 at 10:54:55AM +0800, Peter Xu wrote:
> v6:
> - rename __QEMU_THREAD_COMMON_H__ to QEMU_THREAD_COMMON_H
> - collect r-bs for Emilio

Ping.

It's fine if we don't really want this, but in case if this falls
through the cracks...

Regards,

-- 
Peter Xu