Re: [Qemu-devel] [PATCH 0/4] chardev: support non-default gcontext

2017-09-20 Thread Peter Xu
On Wed, Sep 20, 2017 at 11:21:47PM -0700, no-re...@patchew.org wrote: > Hi, > > This series failed automatic build test. Please find the testing commands and > their output below. If you have docker installed, you can probably reproduce > it > locally. > Oops. This should not be out yet. Pleas

Re: [Qemu-devel] [PATCH 0/4] chardev: support non-default gcontext

2017-09-20 Thread no-reply
Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH 0/4] chardev: support non-default gcontext Message-id: 1505974414-6033-1-git-send-email-pet

Re: [Qemu-devel] [PATCH 0/4] chardev: support non-default gcontext

2017-09-20 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Subject: [Qemu-devel] [PATCH 0/4] chardev: support non-default gcontext Type: series Message-id: 1505974414-6033-1-git-send-email-pet...@redhat.com === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be

[Qemu-devel] [PATCH 0/4] chardev: support non-default gcontext

2017-09-20 Thread Peter Xu
The old chardev may not fully support non-default GMainContext. One direct clue is that when we call io_add_watch_poll() sometimes we are still passing in the NULL context pointer. IIUC we are fine during setup since the context will be passed correctly during setup via chr_update_read_handler().