From nobody Fri Dec 19 17:33:11 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1487783732229332.1753134186223; Wed, 22 Feb 2017 09:15:32 -0800 (PST) Received: from localhost ([::1]:54196 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cgaVj-0007vf-TH for importer@patchew.org; Wed, 22 Feb 2017 12:15:27 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35637) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cgaTt-0006kl-JE for qemu-devel@nongnu.org; Wed, 22 Feb 2017 12:13:34 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cgaTp-0000XW-Mv for qemu-devel@nongnu.org; Wed, 22 Feb 2017 12:13:33 -0500 Received: from mail-wm0-x236.google.com ([2a00:1450:400c:c09::236]:36814) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1cgaTp-0000XA-Gs for qemu-devel@nongnu.org; Wed, 22 Feb 2017 12:13:29 -0500 Received: by mail-wm0-x236.google.com with SMTP id v77so4522727wmv.1 for ; Wed, 22 Feb 2017 09:13:29 -0800 (PST) Received: from zen.linaro.local ([81.128.185.34]) by smtp.gmail.com with ESMTPSA id p49sm2491026wrb.10.2017.02.22.09.13.22 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 22 Feb 2017 09:13:23 -0800 (PST) Received: from zen.linaroharston (localhost [127.0.0.1]) by zen.linaro.local (Postfix) with ESMTP id 9D9293E04AE; Wed, 22 Feb 2017 17:13:28 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=MNMMpZ7CLH2YhoCMoa2yqR3KUTAfrtWXrDKURceoQMU=; b=ilL1SnAKKDmw62Dxapml6Xw/wMZXNvYM1P6Q98aUOWLS/J9INI2mi3cSkilBa2zqj9 2QBMo7vMlnIhkLCCvrTXpPPTA4td8H6XHcNnMgN1VZt3SmHcL85E5OTcE7g13fyxe5hE nJrhsa/eH6+0Yr5VMhp7MfWfm1GDhs2Rq85Nk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=MNMMpZ7CLH2YhoCMoa2yqR3KUTAfrtWXrDKURceoQMU=; b=fc8dlUxJSoSG1iB5HfDbeE8lo1u/+d8y4LSmGOWEI4FgczwMqrvOe+cPwmEEZKPW3G 5vClYAWbENdhE5MvuwIp/rzhHMLbjL3lINjWR4lYy2kgks79O0OvW8zilM8pROfU/kk3 iFj6fH0tPGUlOdaVQq8OLejscD0luztyIPPOE8J4OPE5uNqksVySRq06dNKLVLLLc10i O8Uuf+c/4Uy8hWrtW3ijh3vqlaS81oAQkSTCOBwz8tBD2pKb5YIrQMErRSqG7932Fe6A GO/VQErW+gy2r1iKfWaGF/iMVSyrUjtLMFjnMmlXIOgw8lCLAVlf8VXj8WYYGF96NFT2 jgBA== X-Gm-Message-State: AMke39ka+k9+s+XWi6kTz65uHF+K9MWYeudC6aH0BM/1J79Lw9g373hAu9QJF3bq1+eshF4v X-Received: by 10.28.229.193 with SMTP id c184mr3620988wmh.83.1487783608406; Wed, 22 Feb 2017 09:13:28 -0800 (PST) From: =?UTF-8?q?Alex=20Benn=C3=A9e?= To: rth@twiddle.net, peter.maydell@linaro.org Date: Wed, 22 Feb 2017 17:13:13 +0000 Message-Id: <20170222171327.26624-11-alex.bennee@linaro.org> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170222171327.26624-1-alex.bennee@linaro.org> References: <20170222171327.26624-1-alex.bennee@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2a00:1450:400c:c09::236 Subject: [Qemu-devel] [PATCH v13 10/24] tcg: enable tb_lock() for SoftMMU X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mttcg@listserver.greensocs.com, nikunj@linux.vnet.ibm.com, Peter Crosthwaite , jan.kiszka@siemens.com, mark.burton@greensocs.com, a.rigo@virtualopensystems.com, qemu-devel@nongnu.org, cota@braap.org, serge.fdrv@gmail.com, pbonzini@redhat.com, bobby.prani@gmail.com, =?UTF-8?q?Alex=20Benn=C3=A9e?= , bamvor.zhangjian@linaro.org, fred.konrad@greensocs.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 tb_lock() has long been used for linux-user mode to protect code generation. By enabling it now we prepare for MTTCG and ensure all code generation is serialised by this lock. The other major structure that needs protecting is the l1_map and its PageDesc structures. For the SoftMMU case we also use tb_lock() to protect these structures instead of linux-user mmap_lock() which as the name suggests serialises updates to the structure as a result of guest mmap operations. Signed-off-by: Alex Benn=C3=A9e Reviewed-by: Richard Henderson --- v4 - split from main tcg: enable thread-per-vCPU patch v7 - fixed up with Pranith's tcg_debug_assert() changes --- translate-all.c | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/translate-all.c b/translate-all.c index f810259c41..9bac061c9b 100644 --- a/translate-all.c +++ b/translate-all.c @@ -75,7 +75,7 @@ * mmap_lock. */ #ifdef CONFIG_SOFTMMU -#define assert_memory_lock() do { /* nothing */ } while (0) +#define assert_memory_lock() tcg_debug_assert(have_tb_lock) #else #define assert_memory_lock() tcg_debug_assert(have_mmap_lock()) #endif @@ -135,9 +135,7 @@ TCGContext tcg_ctx; bool parallel_cpus; =20 /* translation block context */ -#ifdef CONFIG_USER_ONLY __thread int have_tb_lock; -#endif =20 static void page_table_config_init(void) { @@ -159,40 +157,29 @@ static void page_table_config_init(void) assert(v_l2_levels >=3D 0); } =20 -#ifdef CONFIG_USER_ONLY #define assert_tb_locked() tcg_debug_assert(have_tb_lock) #define assert_tb_unlocked() tcg_debug_assert(!have_tb_lock) -#else -#define assert_tb_locked() do { /* nothing */ } while (0) -#define assert_tb_unlocked() do { /* nothing */ } while (0) -#endif =20 void tb_lock(void) { -#ifdef CONFIG_USER_ONLY assert_tb_unlocked(); qemu_mutex_lock(&tcg_ctx.tb_ctx.tb_lock); have_tb_lock++; -#endif } =20 void tb_unlock(void) { -#ifdef CONFIG_USER_ONLY assert_tb_locked(); have_tb_lock--; qemu_mutex_unlock(&tcg_ctx.tb_ctx.tb_lock); -#endif } =20 void tb_lock_reset(void) { -#ifdef CONFIG_USER_ONLY if (have_tb_lock) { qemu_mutex_unlock(&tcg_ctx.tb_ctx.tb_lock); have_tb_lock =3D 0; } -#endif } =20 static TranslationBlock *tb_find_pc(uintptr_t tc_ptr); --=20 2.11.0