在 2019/1/9 上午1:29, Markus Armbruster 写道:
fei <lifei1...@126.com> writes:
在 2019年1月8日，01:55，Markus Armbruster <arm...@redhat.com> 写道：
Fei Li <f...@suse.com> writes:
To avoid the segmentation fault in qemu_thread_join(), just directly
return when the QemuThread *thread failed to be created in either
qemu-thread-posix.c or qemu-thread-win32.c.
Cc: Stefan Weil <s...@weilnetz.de>
Signed-off-by: Fei Li <f...@suse.com>
Reviewed-by: Fam Zheng <f...@redhat.com>
---
util/qemu-thread-posix.c | 3 +++
util/qemu-thread-win32.c | 2 +-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/util/qemu-thread-posix.c b/util/qemu-thread-posix.c
index 39834b0551..3548935dac 100644
--- a/util/qemu-thread-posix.c
+++ b/util/qemu-thread-posix.c
@@ -571,6 +571,9 @@ void *qemu_thread_join(QemuThread *thread)
int err;
void *ret;
+ if (!thread->thread) {
+ return NULL;
+ }
How can this happen?
I think I have answered this earlier, please check the following link to see
whether it helps:
http://lists.nongnu.org/archive/html/qemu-devel/2018-11/msg06554.html
Thanks for the pointer. Unfortunately, I don't understand your
explanation. You also wrote there "I will remove this patch in next
version"; looks like you've since changed your mind.
Emm, issues left over from history.. The background is I was hurry to
make those five
Reviewed-by patches be merged, including this v9 16/16 patch but not the
real
qemu_thread_create() modification. But actually this patch is to fix the
segmentation
fault after we modified qemu_thread_create() related functions although
it has got a
Reviewed-by earlier. :) Thus to not make troube, I wrote the "remove..."
sentence
to separate it from those 5 Reviewed-by patches, and were plan to send
only four patches.
But later I got a message that these five patches are not that urgent to
catch qemu v3.1,
thus I joined the earlier 5 R-b patches into the later v8 & v9 to have a
better review.
Sorry for the trouble, I need to explain it without involving too much
background..
Back at the farm: in our current qemu code, some cleanups use a loop to
join()
the total number of threads if caller fails. This is not a problem until
applying the
qemu_thread_create() modification. E.g. when compress_threads_save_setup()
fails while trying to create the last do_data_compress thread,
segmentation fault
will occur when join() is called (sadly there's not enough condition to
filter this
unsuccessful created thread) as this thread is actually not be created.
Hope the above makes it clear. :)
Have a nice day
Fei
What exactly breaks if we omit this patch? Assuming something does
break: imagine we did omit this patch, then forgot we ever saw it, and
now you've discovered the breakage. Write us the bug report, complete
with reproducer.
[...]
