From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40536) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ciMRY-0007bB-LS for qemu-devel@nongnu.org; Mon, 27 Feb 2017 09:38:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ciMRU-0001Xu-M7 for qemu-devel@nongnu.org; Mon, 27 Feb 2017 09:38:28 -0500 Received: from mail-wr0-x22c.google.com ([2a00:1450:400c:c0c::22c]:34546) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ciMRU-0001X7-El for qemu-devel@nongnu.org; Mon, 27 Feb 2017 09:38:24 -0500 Received: by mail-wr0-x22c.google.com with SMTP id l37so9544642wrc.1 for ; Mon, 27 Feb 2017 06:38:24 -0800 (PST) References: <20170224112109.3147-1-alex.bennee@linaro.org> <20170224112109.3147-12-alex.bennee@linaro.org> From: Alex =?utf-8?Q?Benn=C3=A9e?= In-reply-to: Date: Mon, 27 Feb 2017 14:38:21 +0000 Message-ID: <87y3wr8zs2.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PULL 11/24] tcg: enable thread-per-vCPU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Laurent Vivier Cc: peter.maydell@linaro.org, Peter Crosthwaite , qemu-devel@nongnu.org, Paolo Bonzini , KONRAD Frederic , Richard Henderson Laurent Vivier writes: > Le 24/02/2017 à 12:20, Alex Bennée a écrit : >> There are a couple of changes that occur at the same time here: >> >> - introduce a single vCPU qemu_tcg_cpu_thread_fn >> >> One of these is spawned per vCPU with its own Thread and Condition >> variables. qemu_tcg_rr_cpu_thread_fn is the new name for the old >> single threaded function. >> >> - the TLS current_cpu variable is now live for the lifetime of MTTCG >> vCPU threads. This is for future work where async jobs need to know >> the vCPU context they are operating in. >> >> The user to switch on multi-thread behaviour and spawn a thread >> per-vCPU. For a simple test kvm-unit-test like: >> >> ./arm/run ./arm/locking-test.flat -smp 4 -accel tcg,thread=multi >> >> Will now use 4 vCPU threads and have an expected FAIL (instead of the >> unexpected PASS) as the default mode of the test has no protection when >> incrementing a shared variable. >> >> We enable the parallel_cpus flag to ensure we generate correct barrier >> and atomic code if supported by the front and backends. This doesn't >> automatically enable MTTCG until default_mttcg_enabled() is updated to >> check the configuration is supported. > > This commit breaks linux-user mode: > > debian-8 with qemu-ppc on x86_64 with ltp-full-20170116 > > cd /opt/ltp > ./runltp -p -l "qemu-$(date +%FT%T).log" -f /opt/ltp/runtest/syscalls -s > setgroups03 > > setgroups03 1 TPASS : setgroups(65537) fails, Size is > > sysconf(_SC_NGROUPS_MAX), errno=22 > qemu-ppc: /home/laurent/Projects/qemu/include/qemu/rcu.h:89: > rcu_read_unlock: Assertion `p_rcu_reader->depth != 0' failed. > qemu-ppc: /home/laurent/Projects/qemu/include/qemu/rcu.h:89: > rcu_read_unlock: Assertion `p_rcu_reader->depth != 0' failed. > qemu-ppc: /home/laurent/Projects/qemu/include/qemu/rcu.h:89: > rcu_read_unlock: Assertion `p_rcu_reader->depth != 0' failed. > ... Interesting. I can only think the current_cpu change has broken it because most of the changes in this commit affect softmmu targets only (linux-user has its own run loop). Thanks for the report - I'll look into it. -- Alex Bennée