From nobody Tue Feb 10 12:39:20 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass header.i=@intel.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=linux.intel.com ARC-Seal: i=1; a=rsa-sha256; t=1711535287; cv=none; d=zohomail.com; s=zohoarc; b=AEhFujtWvDijJpVJGClprM5isphBOwzQ2n35boi6LhJdApv5TwmNqOdRwB7/2Nz5xDW+wrPwIbHEKx+KumwlFtQdKNwGwqQP72hNHJRMijwrPiPKo4m6r9sxzUO4n5MFZmEzKS+UE3BeYRZRd+BtYaRv8GUWr2nD/AGHS76lGlM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1711535287; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=5vVRfS1DoFbPpYkCTmuhBDmwX3XuWeLI6O2swrmE2rg=; b=TWyx2xK2Ky7oveX3ODu3S9ecGq9/8/OETYZVv/WijPy7dgbr+z2nhvqpeYsKTFyHhNWO8QclFbZtPaF3kUc2G0TW56dvq9mpbkt8l/Jq2dPN5GClGN76IkN3U+Mor7XYBs/0d42qlwJRBIDHXLWh33gs5iGwz77HhadIuIJUQSI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=@intel.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1711535287351266.39513126204497; Wed, 27 Mar 2024 03:28:07 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rpQVG-0008H6-0S; Wed, 27 Mar 2024 06:27:30 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rpQUC-0007NH-Fw for qemu-devel@nongnu.org; Wed, 27 Mar 2024 06:26:24 -0400 Received: from mgamail.intel.com ([192.198.163.19]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rpQU4-0007aP-J9 for qemu-devel@nongnu.org; Wed, 27 Mar 2024 06:26:23 -0400 Received: from fmviesa008.fm.intel.com ([10.60.135.148]) by fmvoesa113.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Mar 2024 03:26:00 -0700 Received: from liuzhao-optiplex-7080.sh.intel.com ([10.239.160.36]) by fmviesa008.fm.intel.com with ESMTP; 27 Mar 2024 03:25:59 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1711535176; x=1743071176; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=NN7bJ9kcE9ymRjA80j4SFQrPkSHGU1tCOZouHjnJcSA=; b=EvoE/PCEAXhJGBZL0VFp4K6GHzLBNi0fZmyNMDcG4XAwFfQc88RwIr/B shFNLebGtlnuHfYPSzpi6tiDmpGyGdEuFmykmL2+rVUiHEevw7u6nkPll rpDpK4pqcHzHC8uSt25ehTcA0LvlDOQsEJyuvG1Mb7sFcVo6qhNbG2Qrs n+NEhW4IAnlN0ChUbFmJyoc/j0T+vJ1LGQJ+fudad30rasnOiejD+QKb1 Xg2VKkF827+mQ6pMoBTJig2oOh5QRFgttD9sqyOKNH3a3SKtFwQ0dyCDi gMkPdm+CFDyqFv7dnZvDWpzCudDNkAKvbwjIErJevZNN3OxNFIx14tMPX w==; X-CSE-ConnectionGUID: 5zLAERswTACTsWLPcoRbTA== X-CSE-MsgGUID: ucRV/GsCT8adfBTFBnTxxQ== X-IronPort-AV: E=McAfee;i="6600,9927,11025"; a="6489758" X-IronPort-AV: E=Sophos;i="6.07,158,1708416000"; d="scan'208";a="6489758" X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,158,1708416000"; d="scan'208";a="16354475" From: Zhao Liu To: Paolo Bonzini Cc: qemu-devel@nongnu.org, Zhao Liu Subject: [PATCH 2/3] target/i386/cpu: Consolidate the use of warn_report_once() Date: Wed, 27 Mar 2024 18:39:50 +0800 Message-Id: <20240327103951.3853425-3-zhao1.liu@linux.intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240327103951.3853425-1-zhao1.liu@linux.intel.com> References: <20240327103951.3853425-1-zhao1.liu@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: none client-ip=192.198.163.19; envelope-from=zhao1.liu@linux.intel.com; helo=mgamail.intel.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.088, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @intel.com) X-ZM-MESSAGEID: 1711535287680100001 Content-Type: text/plain; charset="utf-8" From: Zhao Liu The difference between error_printf() and error_report() is the latter may contain more information, such as the name of the program ("qemu-system-x86_64"). Thus its variant error_report_once() and warn_report()'s variant warn_report_once() can be used here to print the information only once without a static local variable "ht_warned". Signed-off-by: Zhao Liu --- target/i386/cpu.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index 33760a2ee163..0487469d75f3 100644 --- a/target/i386/cpu.c +++ b/target/i386/cpu.c @@ -7230,7 +7230,6 @@ static void x86_cpu_realizefn(DeviceState *dev, Error= **errp) X86CPUClass *xcc =3D X86_CPU_GET_CLASS(dev); CPUX86State *env =3D &cpu->env; Error *local_err =3D NULL; - static bool ht_warned; unsigned requested_lbr_fmt; =20 #if defined(CONFIG_TCG) && !defined(CONFIG_USER_ONLY) @@ -7453,13 +7452,11 @@ static void x86_cpu_realizefn(DeviceState *dev, Err= or **errp) */ if (IS_AMD_CPU(env) && !(env->features[FEAT_8000_0001_ECX] & CPUID_EXT3_TOPOEXT) && - cs->nr_threads > 1 && !ht_warned) { - warn_report("This family of AMD CPU doesn't support " - "hyperthreading(%d)", - cs->nr_threads); - error_printf("Please configure -smp options properly" - " or try enabling topoext feature.\n"); - ht_warned =3D true; + cs->nr_threads > 1) { + warn_report_once("This family of AMD CPU doesn't support " + "hyperthreading(%d).", cs->nr_threads); + error_report_once("Please configure -smp options properly" + " or try enabling topoext feature."); } =20 #ifndef CONFIG_USER_ONLY --=20 2.34.1