From: Marcelo Tosatti <mtosatti@redhat.com> To: Jan Kiszka <jan.kiszka@siemens.com> Cc: Avi Kivity <avi@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>, "kvm@vger.kernel.org" <kvm@vger.kernel.org>, "aurelien@aurel32.net" <aurelien@aurel32.net>, "blauwirbel@gmail.com" <blauwirbel@gmail.com> Subject: Re: [PATCH v3 uq/master 00/22] Win32 iothread support Date: Wed, 2 Mar 2011 16:42:23 -0300 [thread overview] Message-ID: <20110302194223.GA23050@amt.cnet> (raw) In-Reply-To: <20110302193634.GB22743@amt.cnet> On Wed, Mar 02, 2011 at 04:36:34PM -0300, Marcelo Tosatti wrote: > On Wed, Mar 02, 2011 at 08:03:42PM +0100, Jan Kiszka wrote: > > On 2011-03-02 19:43, Marcelo Tosatti wrote: > > > On Tue, Mar 01, 2011 at 02:35:56PM +0200, Avi Kivity wrote: > > >> On 02/28/2011 04:05 PM, Paolo Bonzini wrote: > > >>> On 02/28/2011 01:13 PM, Avi Kivity wrote: > > >>>>> > > >>>> > > >>>> If there's a git tree of this I'll be happy to do an autotest run. > > >>> > > >>> Sure, it's branch iothread-win32 of git://github.com/bonzini/qemu.git > > >> > > >> Fails on Fedora 9 i386 install, hangs right after "Performing post > > >> install configuration...". The guest is processing interrupts but > > >> the mouse won't move, and it doesn't make progress. > > >> > > >> Configured with --enable-io-thread. Perhaps the problem exists even > > >> before the patchset. > > > > > > Probably unrelated, looks similar to the regression seen with qemu-kvm. > > > > > > > Do these patches change some behavior or not? > > Yes, they change some behaviour. Autotest fails. Sorry, i misunderstood. I don't think these patches change any behaviour. Avi's failure case is similar to what i've seen earlier upon qemu->qemu-kvm merge. Conclusion is there is no new regression introduced by these patches. > > > Is it this same effect you saw with my qemu-kvm queue? > > Yes. >
WARNING: multiple messages have this Message-ID (diff)
From: Marcelo Tosatti <mtosatti@redhat.com> To: Jan Kiszka <jan.kiszka@siemens.com> Cc: "kvm@vger.kernel.org" <kvm@vger.kernel.org>, "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>, "blauwirbel@gmail.com" <blauwirbel@gmail.com>, Avi Kivity <avi@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "aurelien@aurel32.net" <aurelien@aurel32.net> Subject: [Qemu-devel] Re: [PATCH v3 uq/master 00/22] Win32 iothread support Date: Wed, 2 Mar 2011 16:42:23 -0300 [thread overview] Message-ID: <20110302194223.GA23050@amt.cnet> (raw) In-Reply-To: <20110302193634.GB22743@amt.cnet> On Wed, Mar 02, 2011 at 04:36:34PM -0300, Marcelo Tosatti wrote: > On Wed, Mar 02, 2011 at 08:03:42PM +0100, Jan Kiszka wrote: > > On 2011-03-02 19:43, Marcelo Tosatti wrote: > > > On Tue, Mar 01, 2011 at 02:35:56PM +0200, Avi Kivity wrote: > > >> On 02/28/2011 04:05 PM, Paolo Bonzini wrote: > > >>> On 02/28/2011 01:13 PM, Avi Kivity wrote: > > >>>>> > > >>>> > > >>>> If there's a git tree of this I'll be happy to do an autotest run. > > >>> > > >>> Sure, it's branch iothread-win32 of git://github.com/bonzini/qemu.git > > >> > > >> Fails on Fedora 9 i386 install, hangs right after "Performing post > > >> install configuration...". The guest is processing interrupts but > > >> the mouse won't move, and it doesn't make progress. > > >> > > >> Configured with --enable-io-thread. Perhaps the problem exists even > > >> before the patchset. > > > > > > Probably unrelated, looks similar to the regression seen with qemu-kvm. > > > > > > > Do these patches change some behavior or not? > > Yes, they change some behaviour. Autotest fails. Sorry, i misunderstood. I don't think these patches change any behaviour. Avi's failure case is similar to what i've seen earlier upon qemu->qemu-kvm merge. Conclusion is there is no new regression introduced by these patches. > > > Is it this same effect you saw with my qemu-kvm queue? > > Yes. >
next prev parent reply other threads:[~2011-03-02 19:45 UTC|newest] Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-02-28 9:10 [PATCH v3 uq/master 00/22] Win32 iothread support Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 01/22] unlock iothread during WaitForMultipleObjects Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 02/22] implement win32 dynticks timer Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 03/22] use win32 timer queues Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 04/22] Refactor thread retrieval and check Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 05/22] add win32 qemu-thread implementation Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:33 ` Stefan Hajnoczi 2011-02-28 9:33 ` [Qemu-devel] " Stefan Hajnoczi 2011-02-28 10:14 ` [PATCH v4 uq/master] " Paolo Bonzini 2011-02-28 10:14 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 06/22] include qemu-thread.h early Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 07/22] add assertions on the owner of a QemuMutex Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 08/22] remove CONFIG_THREAD Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 09/22] target-sh4: move intr_at_halt out of cpu_halted() Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 10/22] inline cpu_halted into sole caller Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 11/22] always qemu_cpu_kick after unhalting a cpu Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 12/22] exit round-robin vcpu loop if cpu->stopped is true Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 13/22] always signal pause_cond after stopping a VCPU Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 14/22] do not use timedwait on qemu_halt_cond Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 15/22] do not use timedwait on qemu_system_cond Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 16/22] do not use timedwait on qemu_pause_cond Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 17/22] do not use timedwait on qemu_cpu_cond Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 18/22] iothread stops the vcpu thread via IPI Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 19/22] merge all signal initialization with qemu_signalfd_init, rename Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 20/22] provide dummy signal init functions for win32 Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 21/22] protect qemu_cpu_kick_self for Win32 Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:10 ` [PATCH v3 uq/master 22/22] add Win32 IPI service Paolo Bonzini 2011-02-28 9:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 9:16 ` [PATCH v3 uq/master 00/22] Win32 iothread support Avi Kivity 2011-02-28 9:16 ` [Qemu-devel] " Avi Kivity 2011-02-28 10:10 ` Paolo Bonzini 2011-02-28 10:10 ` [Qemu-devel] " Paolo Bonzini 2011-02-28 11:57 ` Jan Kiszka 2011-02-28 11:57 ` [Qemu-devel] " Jan Kiszka 2011-02-28 12:13 ` Avi Kivity 2011-02-28 12:13 ` [Qemu-devel] " Avi Kivity 2011-02-28 14:05 ` Paolo Bonzini 2011-02-28 14:05 ` [Qemu-devel] " Paolo Bonzini 2011-03-01 12:35 ` Avi Kivity 2011-03-01 12:35 ` [Qemu-devel] " Avi Kivity 2011-03-02 18:43 ` Marcelo Tosatti 2011-03-02 18:43 ` [Qemu-devel] " Marcelo Tosatti 2011-03-02 19:03 ` Jan Kiszka 2011-03-02 19:03 ` [Qemu-devel] " Jan Kiszka 2011-03-02 19:36 ` Marcelo Tosatti 2011-03-02 19:36 ` [Qemu-devel] " Marcelo Tosatti 2011-03-02 19:42 ` Marcelo Tosatti [this message] 2011-03-02 19:42 ` Marcelo Tosatti 2011-03-03 8:48 ` Avi Kivity 2011-03-03 8:48 ` [Qemu-devel] " Avi Kivity 2011-03-03 9:07 ` Jan Kiszka 2011-03-03 9:07 ` [Qemu-devel] " Jan Kiszka 2011-03-03 9:12 ` Avi Kivity 2011-03-03 9:12 ` [Qemu-devel] " Avi Kivity 2011-03-03 9:13 ` Paolo Bonzini 2011-03-03 9:13 ` [Qemu-devel] " Paolo Bonzini 2011-03-03 9:28 ` Avi Kivity 2011-03-03 9:28 ` [Qemu-devel] " Avi Kivity
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20110302194223.GA23050@amt.cnet \ --to=mtosatti@redhat.com \ --cc=aurelien@aurel32.net \ --cc=avi@redhat.com \ --cc=blauwirbel@gmail.com \ --cc=jan.kiszka@siemens.com \ --cc=kvm@vger.kernel.org \ --cc=pbonzini@redhat.com \ --cc=qemu-devel@nongnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.