Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Luck, Tony" <tony.luck@intel.com>
To:     "Raj, Ashok" <ashok.raj@intel.com>, Borislav Petkov <bp@alien8.de>
CC:     "Hansen, Dave" <dave.hansen@intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        X86-kernel <x86@kernel.org>,
        LKML Mailing List <linux-kernel@vger.kernel.org>,
        Ingo Molnar <mingo@kernel.org>,
        "Schofield, Alison" <alison.schofield@intel.com>,
        "Chatre, Reinette" <reinette.chatre@intel.com>,
        Tom Lendacky <thomas.lendacky@amd.com>
Subject: RE: [PATCH v4 6/6] x86/microcode/intel: Print when early microcode
 loading fails
Thread-Topic: [PATCH v4 6/6] x86/microcode/intel: Print when early microcode
 loading fails
Thread-Index: AQHZJEAjPJqPA6/3BUKKuoCqIYf3wK6f4BgAgAL0VgCAAATOAIAAHzEAgAADRwCAAAImgIAAIYUAgAAD5/A=
Date:   Tue, 17 Jan 2023 20:58:12 +0000
Message-ID: <IA1PR11MB6076660471E9C688E79A2B20FCC69@IA1PR11MB6076.namprd11.prod.outlook.com>
References: <20230109153555.4986-1-ashok.raj@intel.com>
 <20230109153555.4986-7-ashok.raj@intel.com> <Y8ROaoJtUtj5bPcx@zn.tnic>
 <Y8bI4BBst78qrApD@a4bf019067fa.jf.intel.com>
 <e491fe87-2c2a-e9d2-3cd9-dfc7535f7c27@intel.com> <Y8bnEgP6iXCL+QmX@zn.tnic>
 <832b5c6c-cafa-e9e6-6e52-b741315d0865@intel.com> <Y8brnyIltcgtUvPn@zn.tnic>
 <Y8cHvmU6pa/rNw8n@a4bf019067fa.jf.intel.com>
In-Reply-To: <Y8cHvmU6pa/rNw8n@a4bf019067fa.jf.intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?+rTXQlImd9Z2oKCOnXaXmMTg36usR2VOzu8K230IX+grVQ9uoXqPwmxiBuzt?=
 =?us-ascii?Q?VDTd9Qq5sn5Ht150iPgHcjaKqwXCUWkQjJL3n4xzN4kr6orwyGIk8M1H73d8?=
 =?us-ascii?Q?NAW1NOmoB0Isp19AvOcaPKjp6/jnceo5EL8KzjXC4TzFMYQb5oqPU4sKovCc?=
 =?us-ascii?Q?7aF6ahOOCUQl7GqpCcJe4K9W3QVA4bHbnr/oCazJESrm7xNPE0W/VcjAD0pY?=
 =?us-ascii?Q?NeU7CeQ1vlB8Lu+sp8K6K8Wk5A29git6TNv6ft4mojxsEx0tbf72Tckv1aV+?=
 =?us-ascii?Q?jYmzN030XWgPtziv3Kzmfki2x5raPIkbpUGwyFEJXrSPQKucoTcDIlhWCBiV?=
 =?us-ascii?Q?o2USKdo87t3OP5u9id468vY3zNGHLm/TUmS7e+GkKr66k+5zCAoMFXMnZOkD?=
 =?us-ascii?Q?eH1TAsX9rgOnesG9Uqze8oYiRJ+0cyiTdsILGUOwu6+1/WTyWIjniCIqv+1h?=
 =?us-ascii?Q?Zsv1gB2NwYdz55GGJJIqri1w2wbCsnx99KwGT8IoaG4eDKxmtokYclg/5kbL?=
 =?us-ascii?Q?F3OQS+VbLMwhH1CQ2iiIfCarPIXsdM/lak49Wa3qRNfpF7te/0DfcD1zXd3Z?=
 =?us-ascii?Q?n8wyKUbd94oSBMNZ63TMAZzS6509TICiol1OXKYziO/98xk3cSHpUPp7OYdJ?=
 =?us-ascii?Q?GBgSRAzHN/DkEz+s2W+qRF2ZX+aDP3qrT6XZjYjDiwS0WcyxODXo0Pv05JH2?=
 =?us-ascii?Q?h+Cs+y/WX8TBHOU/rvWPM64t3SDJx3c4g07jOTNs8fMGQVhrWOeJZ49FInch?=
 =?us-ascii?Q?qm8X1Ym+EEsrWCu7H7zj5fDu3s3x4YTnFkvPh+TVmx/CpDk6yVymvQaKHIhO?=
 =?us-ascii?Q?06Bwv5tN/JW1cX+Baryy9nIgYou9s4o+qA1x1fNqd0gdJaSDjHfJ1rwEqcus?=
 =?us-ascii?Q?yTOSpxSTSpDytK5IStsYtmZO4KeucEHXoF7qZQ8o1PpbM/OGZTAImiuJOcGh?=
 =?us-ascii?Q?DcSq0OKSEw148nk9d32d4ObgIp3NmT5ntn0MgeLElguCGJ0Ld/4xJ8Xr1vsN?=
 =?us-ascii?Q?+D2RhIMRU4iS3y4wNigva5UFJtNzMBFb1xqpOnfTyZWWriq8Gpja6u520MMh?=
 =?us-ascii?Q?AIi7rgWWiMqYooG/5PBg3oqh2TlF96W/vEOsOjdTCSilwvFKpfpVuh4GtINt?=
 =?us-ascii?Q?0hk2O4jaHJG568Ks+GYCa2+cjhoLs6bKZ8Qfx8wvx5jvpEER3hrgOwTTx4dQ?=
 =?us-ascii?Q?XiV4aMEV8FKx0KkKAsWnnLOvU6FN/WdIqSZMb1e7EjkDEq3tC4tsxv+ZF+85?=
 =?us-ascii?Q?PeVWB0deJlmbcmihNLBuc7R28XzvlMEZCpx1fLAMbSk3qJRxWUWyCmxy3R1x?=
 =?us-ascii?Q?WIGbN+QFT/6+BzwpAsWook85Jpc9JIZBQ8ai3ZRa8f5IxmWcGvnRCa2No7Z+?=
 =?us-ascii?Q?7BhYB7ya/q7auyltprLX8Dyo7lEptlkj6GYVSZ6JmLrqSEq479vLIr7ZhlwJ?=
 =?us-ascii?Q?IYZ+i0vS9FCnZWS3Owyd1jtc05y5vPMlROM/5XlFjM2qMKHdyCK3/PPyZYLO?=
 =?us-ascii?Q?vwNgaVTFcrvkoW05Rv58fPM0zBsGiGuKJT6M6ZGd45Of3eC2jAjG1UkuhVbC?=
 =?us-ascii?Q?pZZFixizWyNuZBi0i7gc6IYK1k66+a2R+IYx2Ocd?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: IA1PR11MB6076.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 110c5543-a511-440f-b494-08daf8cd8c2a
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Jan 2023 20:58:12.9314
 (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: 00tID4MksaV5aYr0N5bgTi+vvzaWhuf6BIafkfIQeWR+QlkI4ADmshVaS4m/wB4Ky65wxHb6RtuvxyH6FJscPw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB8173
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_PASS,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> If there is no microcode, we don't print anything. So what's loaded in th=
e
> CPU is the latest version. When we have something we can always tell if i=
ts
> successful or not.
>
> Its not a microcode file in initrd, but a matching microcode to load. If
> none is found, nothing to worry about.
>
> We just agreed to show both failed and success for late-load. Doing this =
is
> consistent with that isn't it?
>
> https://lore.kernel.org/lkml/Y7iYLbEJSYnVn+dW@zn.tnic/
>
> Ingo's:
> https://lore.kernel.org/lkml/Y7k9DNz%2F%2FvqBAvZK@gmail.com/
>
> Should we treat early loading differently?

Getting a better set of messages from early microcode update would
be a "nice-to-have" feature. But if there is no agreement on what those
messages should look like, perhaps just skip this part for now.

Then Ashok can move on to the real issue of allowing LATE_LOAD for a
microcode that supports the new "minrev" header field.

-Tony