Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
Date:   Thu, 30 Nov 2023 10:46:29 -0800
From:   Ashok Raj <ashok.raj@intel.com>
To:     Borislav Petkov <bp@alien8.de>
CC:     X86 ML <x86@kernel.org>, LKML <linux-kernel@vger.kernel.org>,
        "Linus Torvalds" <torvalds@linux-foundation.org>,
        Ashok Raj <ashok.raj@intel.com>
Subject: Re: [PATCH 2/2] x86/microcode: Rework early revisions reporting
Message-ID: <ZWjYhedNfhAUmt0k@a4bf019067fa.jf.intel.com>
References: <20231115210212.9981-1-bp@alien8.de>
 <20231115210212.9981-3-bp@alien8.de>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20231115210212.9981-3-bp@alien8.de>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?bTTDEc9xr0FhEsaNdS6LfshE3qMgDDaNWS3Bgs+MUmpsE+6xoEFf/aId/GyU?=
 =?us-ascii?Q?k/j/oIGFowouLEDT56waJJk5D11QbbgQ+wGHjSt6xgCd3LeIhLOmSALheugN?=
 =?us-ascii?Q?RUuUpv6115odAcqK7NdTb0IiMWtsBYF1noEFPply1SyBdKl+GClQeVAGirdo?=
 =?us-ascii?Q?yUWfW13n6l+nXnDzbSYXzAnqJjmhFCggeL//fWqk9AlqeYkyWBr9sAHOBjGM?=
 =?us-ascii?Q?h1qsy8rkk8j+1B4Vuvb6gEFg7eV654QwuiZYwn5EFQTztWuexweRlOB5Tot/?=
 =?us-ascii?Q?AkmUmKlmb7IoC9Ks6IqU4FHWE+obtSC/cr7KXKoqBu60FH9gOfRr/UlXVzeB?=
 =?us-ascii?Q?31FnuEH7yGryCGD+/u+7DBdWFGwr8yk4N/K7mFxasuB3E9ozamE5aADJGtBo?=
 =?us-ascii?Q?1pyqoe9XgwcZmPNl3JwNJdFo+8q1VDue92Iq3ex1YtccTeJLlo6guihiH3lS?=
 =?us-ascii?Q?wo16c1xgfucINMvBj59WZEhKK6SATa+r508YZqWmKFBaqU3y01/dxqgm/AUM?=
 =?us-ascii?Q?5MJnfm3BEmqVjHP4g+gw4E4JIySOL77dJCLh3IaiBrCaqpSz54vNniRdIB4z?=
 =?us-ascii?Q?LehdsHNoexZEI2ANK6c1EkxjqbnCnrAu17JrKqENoVGb9S6dRtDfnrLtS6u/?=
 =?us-ascii?Q?pd+YN7xvBQpgkFlVjE0UOhQ006gecBAplW3HBxvBjoTtmo3wP/SFXB90EAR1?=
 =?us-ascii?Q?DOEA6sc7dyvMYMSaLZ0DTBFbW1ZuYx+e0IdSP8FOVPJLa6doV0DsiNnDyab5?=
 =?us-ascii?Q?A5PzyCtUct4u7hCZyU94DABonMxoo1rQB9UXU4jAt2CZnCKfMRddPYO2hw2q?=
 =?us-ascii?Q?5nR7KKjSaR1o6BfO2pgiPrbmjHYwJ7FTAifZ8LX/C0c1wJKlAShPP/ucuSEB?=
 =?us-ascii?Q?zFpIX1YTPY3SjX+CbTNLatIx69xiORXndcR+jIirU8W10hYhsyJRByB54K+V?=
 =?us-ascii?Q?d1oDltGnnmvxOa8aQtHcI33hYwBAeTfMmfCaSPr9PmtH4v8XJDBWfB1sKhQd?=
 =?us-ascii?Q?JPGO/3OKA4S5JbJ60aecGgPy84eDjJOX9VP+r4zmcPZnGaRmbXnYRqFjIT/S?=
 =?us-ascii?Q?GYVphN+AIgdunzBoDo1X5HuAcEsfz/BP3xruq8b2UoYmlVxQAh6EdTZ+mFIn?=
 =?us-ascii?Q?Q/uxLy1zGHG78Eg8GONQv6uRJF8wC6RjTlUSY/YWoMeIfIoTlOj4Nm0vwnL1?=
 =?us-ascii?Q?JzPogj3NEUvgCnEzES+z+K+n7zZCnfsd/mC/GhuskPuyoF+872kpVF3fF0bl?=
 =?us-ascii?Q?a1ef4MB1WvM/qTkOJUYT1TYUumEe0B4AU/CeoCzoRBY7+fXwbavWyQOl7YQR?=
 =?us-ascii?Q?tITZbcCmGd6/dyuUo47WMXWTxX9GIQd+SUjO+0o2M4ZPlbmMinK923psEC7o?=
 =?us-ascii?Q?4SUxuHsbr5E6LZ6PY7OzJstWZUggpMHE2Ro18G0+6PVCxEnlkqbhDWl/RaOH?=
 =?us-ascii?Q?iAbI3OvEvkyjoE46PAFMAlMVE3htUiXgqg3gbmsN5XZ4OJYxRfzOvthjtwVB?=
 =?us-ascii?Q?/2SALCJOkmDxfMNm2+Y/oeZ3kE+bWKjW+7HYS8MnbpzfpmAnQFKBfq+keWtU?=
 =?us-ascii?Q?LlKmzDUZDT+oLzfaLgAsWUX2n2dtl8CfnX/EyCqB?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 17d3d8a9-703a-4645-b599-08dbf1d4ad60
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR11MB6201.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Nov 2023 18:46:35.1164
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: yVj3X2ZP1JbDH5ohOvHt9enCG4m1lp6L47rKW0yV0e/dwNJh1XoLTH4mqAr8BRmQvVycbI92NxJBt2g8NmCmCQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB6727
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_PDS_OTHER_BAD_TLD,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Thu, 30 Nov 2023 10:46:54 -0800 (PST)

Hi

Since the late load already announces the old and new revision in core.c

Now that early loading doesn't print the 'date' for Intel, does it make
sense to remove those for late-load as well?

Feel free to squash this into any patch you have in progress if that makes
sense.

On Wed, Nov 15, 2023 at 10:02:12PM +0100, Borislav Petkov wrote:

[snip]

> diff --git a/arch/x86/kernel/cpu/microcode/intel.c b/arch/x86/kernel/cpu/microcode/intel.c
> index 6024feb98d29..070426b9895f 100644
> --- a/arch/x86/kernel/cpu/microcode/intel.c
> +++ b/arch/x86/kernel/cpu/microcode/intel.c
> @@ -339,16 +339,9 @@ static enum ucode_state __apply_microcode(struct ucode_cpu_info *uci,
>  static enum ucode_state apply_microcode_early(struct ucode_cpu_info *uci)
>  {
>  	struct microcode_intel *mc = uci->mc;
> -	enum ucode_state ret;
> -	u32 cur_rev, date;
> +	u32 cur_rev;
>  
> -	ret = __apply_microcode(uci, mc, &cur_rev);
> -	if (ret == UCODE_UPDATED) {
> -		date = mc->hdr.date;
> -		pr_info_once("updated early: 0x%x -> 0x%x, date = %04x-%02x-%02x\n",
> -			     cur_rev, mc->hdr.rev, date & 0xffff, date >> 24, (date >> 16) & 0xff);
> -	}
> -	return ret;
> +	return __apply_microcode(uci, mc, &cur_rev);
>  }


From: Ashok Raj <ashok.raj@intel.com>
Date: Wed, 29 Nov 2023 13:56:43 -0800
Subject: [PATCH 2/2] x86/microcode/intel: Remove the redundant microcode
 updated message

After successful update, core.c/late_load_stop_cpus() prints a message as
shown below.

[  215.386472] microcode: load: updated on 128 primary CPUs with 128 siblings
[  215.394370] microcode: revision: 0x21000170 -> 0x21000190

Remove the redundant message.

Signed-off-by: Ashok Raj <ashok.raj@intel.com>
---
 arch/x86/kernel/cpu/microcode/intel.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/x86/kernel/cpu/microcode/intel.c b/arch/x86/kernel/cpu/microcode/intel.c
index 070426b9895f..5d6ea875b81b 100644
--- a/arch/x86/kernel/cpu/microcode/intel.c
+++ b/arch/x86/kernel/cpu/microcode/intel.c
@@ -457,12 +457,6 @@ static enum ucode_state apply_microcode_late(int cpu)
 	if (ret != UCODE_UPDATED && ret != UCODE_OK)
 		return ret;
 
-	if (!cpu && uci->cpu_sig.rev != cur_rev) {
-		pr_info("Updated to revision 0x%x, date = %04x-%02x-%02x\n",
-			uci->cpu_sig.rev, mc->hdr.date & 0xffff, mc->hdr.date >> 24,
-			(mc->hdr.date >> 16) & 0xff);
-	}
-
 	cpu_data(cpu).microcode	 = uci->cpu_sig.rev;
 	if (!cpu)
 		boot_cpu_data.microcode = uci->cpu_sig.rev;
-- 
2.39.2