[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH RFC v7 0/9] qemu_thread_create: propagate errors to
From: |
Fei Li |
Subject: |
[Qemu-devel] [PATCH RFC v7 0/9] qemu_thread_create: propagate errors to callers to check |
Date: |
Thu, 1 Nov 2018 18:17:06 +0800 |
Hi,
This idea comes from BiteSizedTasks, and this patch series implement
the error checking of qemu_thread_create: make qemu_thread_create
return a flag to indicate if it succeeded rather than failing with an
error; make all callers check it.
The first and the third patch fixes some segmentation faults occured
during the debugging. The second patch paves the way for the last
patch as that patch is too long. The last patch modifies the
qemu_thread_create() and makes it return a bool to all direct callers
to indicate if it succeeds. The middle five fix some migration
issues.
Please help to review, thanks. :)
v7:
- Split the previous multifd-migration into two patches: the src and
the dst. For the dst, only dump the error instead of quitting.
- Safely do the cleanup for postcopy_ram_enable_notify().
- Split the previous migration-error-handling patch into two patches.
v6:
- Add a new migration-multifd related patch. BTW, delete the previous
vnc related patch as it has been upstreamed.
- Use error_setg_errno() to set the errno when qemu_thread_create()
fails for both Linux and Windows implementation.
- Optimize the first patch, less codes are needed
v5:
- Remove `errno = err` in qemu_thread_create() for Linux, and change
`return errno` to `return -1` in qemu_signal_init() to indicate
the error in case qemu_thread_create() fails.
- Delete the v4-added qemu_cond/mutex_destroy() in iothread_complete()
as the destroy() will be done by its callers' object_unref().
v4:
- Separate the migration compression patch from this series
- Add one more error handling patch related with migration
- Add more cleaning up code for touched functions
v3:
- Add two migration related patches to fix the segmentaion fault
- Extract the segmentation fault fix from v2's last patch to be a
separate patch
- Add cleaning up code for touched functions
- Update some error messages
v2:
- Pass errp straightly instead of using a local_err & error_propagate
- Return a bool: false/true to indicate if one function succeeds
- Merge v1's last two patches into one to avoid the compile error
- Fix one omitted error in patch1 and update some error messages
Fei Li (9):
Fix segmentation fault when qemu_signal_init fails
qemu_init_vcpu: add a new Error parameter to propagate
qemu_thread_join: fix segmentation fault
migration: fix some segmentation faults when using multifd
migration: fix the multifd code when sending less channels
migration: fix the multifd code when receiving less channels
migration: remove unused &local_err parameter in migrate_set_error
migration: add more error handling for postcopy_ram_enable_notify
qemu_thread_create: propagate the error to callers to handle
accel/tcg/user-exec-stub.c | 3 +-
cpus.c | 79 ++++++++++++++++++++++++--------------
dump.c | 6 ++-
hw/misc/edu.c | 6 ++-
hw/ppc/spapr_hcall.c | 10 ++++-
hw/rdma/rdma_backend.c | 4 +-
hw/usb/ccid-card-emulated.c | 15 ++++++--
include/qemu/thread.h | 4 +-
include/qom/cpu.h | 2 +-
io/task.c | 3 +-
iothread.c | 16 +++++---
migration/channel.c | 11 +++---
migration/migration.c | 68 +++++++++++++++++++++------------
migration/migration.h | 2 +-
migration/postcopy-ram.c | 15 +++++++-
migration/ram.c | 84 ++++++++++++++++++++++++++++++-----------
migration/ram.h | 4 +-
migration/savevm.c | 12 ++++--
target/alpha/cpu.c | 4 +-
target/arm/cpu.c | 4 +-
target/cris/cpu.c | 4 +-
target/hppa/cpu.c | 4 +-
target/i386/cpu.c | 4 +-
target/lm32/cpu.c | 4 +-
target/m68k/cpu.c | 4 +-
target/microblaze/cpu.c | 4 +-
target/mips/cpu.c | 4 +-
target/moxie/cpu.c | 4 +-
target/nios2/cpu.c | 4 +-
target/openrisc/cpu.c | 4 +-
target/ppc/translate_init.inc.c | 4 +-
target/riscv/cpu.c | 4 +-
target/s390x/cpu.c | 4 +-
target/sh4/cpu.c | 4 +-
target/sparc/cpu.c | 4 +-
target/tilegx/cpu.c | 4 +-
target/tricore/cpu.c | 4 +-
target/unicore32/cpu.c | 4 +-
target/xtensa/cpu.c | 4 +-
tests/atomic_add-bench.c | 3 +-
tests/iothread.c | 2 +-
tests/qht-bench.c | 3 +-
tests/rcutorture.c | 3 +-
tests/test-aio.c | 2 +-
tests/test-rcu-list.c | 3 +-
ui/vnc-jobs.c | 17 ++++++---
ui/vnc-jobs.h | 2 +-
ui/vnc.c | 4 +-
util/compatfd.c | 12 +++++-
util/main-loop.c | 8 ++--
util/oslib-posix.c | 17 +++++++--
util/qemu-thread-posix.c | 27 +++++++++----
util/qemu-thread-win32.c | 18 ++++++---
util/rcu.c | 3 +-
util/thread-pool.c | 4 +-
55 files changed, 388 insertions(+), 168 deletions(-)
--
2.13.7
- [Qemu-devel] [PATCH RFC v7 0/9] qemu_thread_create: propagate errors to callers to check,
Fei Li <=
- [Qemu-devel] [PATCH RFC v7 3/9] qemu_thread_join: fix segmentation fault, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 2/9] qemu_init_vcpu: add a new Error parameter to propagate, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 1/9] Fix segmentation fault when qemu_signal_init fails, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 4/9] migration: fix some segmentation faults when using multifd, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 5/9] migration: fix the multifd code when sending less channels, Fei Li, 2018/11/01