From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=59902 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Pu5b5-0003Ut-AU for qemu-devel@nongnu.org; Mon, 28 Feb 2011 11:05:20 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Pu5b1-0001UM-CI for qemu-devel@nongnu.org; Mon, 28 Feb 2011 11:05:19 -0500 Received: from thoth.sbs.de ([192.35.17.2]:30254) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Pu5b1-0001UH-1m for qemu-devel@nongnu.org; Mon, 28 Feb 2011 11:05:15 -0500 Message-ID: <4D6BC7B9.5090400@siemens.com> Date: Mon, 28 Feb 2011 17:05:13 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <688de146287dc589a2e0fcce4cdec85d4f2c1127.1296594961.git.jan.kiszka@web.de> <4D6BC55B.9030805@redhat.com> <4D6BC70E.3000500@siemens.com> In-Reply-To: <4D6BC70E.3000500@siemens.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH v2 13/24] kvm: Set up signal mask also for !CONFIG_IOTHREAD List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: Marcelo Tosatti , qemu-devel@nongnu.org, kvm@vger.kernel.org On 2011-02-28 17:02, Jan Kiszka wrote: > On 2011-02-28 16:55, Avi Kivity wrote: >> On 02/01/2011 11:15 PM, Jan Kiszka wrote: >>> From: Jan Kiszka >>> >>> Block SIG_IPI, unblock it during KVM_RUN, just like in io-thread mode. >>> It's unused so far, but this infrastructure will be required for >>> self-IPIs and to process SIGBUS plus, in KVM mode, SIGIO and SIGALRM. As >>> Windows doesn't support signal services, we need to provide a stub for >>> the init function. >>> >> >> This patch breaks qemu-kvm after merging. The symptoms are that Windows >> XP x64 does not respond when netcat connects to some server in it, via >> -net user,hostfwd. The vcpu thread loops indefinitely on KVM_EXIT_INTR, >> which is consistent with signals being messed up. > > Does the same test case work with qemu, iothread on and off? Just to Err, "iothread on" makes no sense here, of course. > ensure we are not hunting an issue with the patch itself but of the merge. > > Will have a look as well. > Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux