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;
Date:   Fri, 2 Dec 2022 17:47:05 -0800
From:   Ashok Raj <ashok.raj@intel.com>
To:     Sohil Mehta <sohil.mehta@intel.com>
CC:     Borislav Petkov <bp@alien8.de>, X86-kernel <x86@kernel.org>,
        "LKML Mailing List" <linux-kernel@vger.kernel.org>,
        Dave Hansen <dave.hansen@intel.com>,
        Tony Luck <tony.luck@intel.com>, <alison.schofield@intel.com>,
        <reinette.chatre@intel.com>, Ashok Raj <ashok.raj@intel.com>
Subject: Re: [Patch V1 2/7] x86/microcode/intel: Remove retries on early
 microcode load
Message-ID: <Y4qqmcM8iijFaemO@a4bf019067fa.jf.intel.com>
References: <20221129210832.107850-1-ashok.raj@intel.com>
 <20221129210832.107850-3-ashok.raj@intel.com>
 <baf496c4-afb5-bb89-02eb-17a319618833@intel.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <baf496c4-afb5-bb89-02eb-17a319618833@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?E3FhL9dzzhdT9bwZceSmNzcLU1fGsByoG0um6ZgfU9mxnNaMVIzodutGHVn1?=
 =?us-ascii?Q?EBCY+eJISWirvWokF3osfvALSR3zJw6pxnTqQoK5nQLfRN3Ame+FZI+m+rwh?=
 =?us-ascii?Q?Dh1KtLi1eG0g96WOGDA405TV1K0TOxP2L0OlayjLi79roGFpdbqo9P2pNlPq?=
 =?us-ascii?Q?lIHVujPQUTlMTIpK3r6uoU3kHxwdlhW3/uRIQZ2IuE3PwaEQn6uh7mMNcQuj?=
 =?us-ascii?Q?zqynMuYXJzGJQz9r92pmL7y5njrdrD8lmoZt8a4gNZwgFTtVrT4y0KsapKG2?=
 =?us-ascii?Q?gGszd1m+wr3XbWB2wgC981hWLim1eGMT3g8sZ/ZRq8IRn21DO0hbgUCneoCl?=
 =?us-ascii?Q?fSfyeaDDc7nLGe9QbMaPuCs4K5VgjiGTbX05knOolej1VviCS9maLUvTipfd?=
 =?us-ascii?Q?Tae9eo9iwMQgvwdMQVRrAq3W/bMc3JAP8M6+eVeP2EfmEIMS0E5CttEWO23w?=
 =?us-ascii?Q?HvybjEz79Ht/N9twqEvP2mfef1QzZMgF2cuKJvyn1B6lTPHgUCJCrO6BfzL4?=
 =?us-ascii?Q?72BQYHBDhWgaefK6hicAHhsFUxT8vDTMZ4FM/GX2ieL/VU60/hRMSl5AQ3ic?=
 =?us-ascii?Q?4578IIiwgY678leQ8TVzkFfTppFvL7dgvgg9W4hdArJPjubUN+zwtaviq513?=
 =?us-ascii?Q?C4xWJxlHIhJMnDx5OXZ+SdNOFMgBvHFEnNPuynxZ1m9Y/wcfXrE2Sc+UCwRQ?=
 =?us-ascii?Q?8Y2Lfr1qJX1M6efKegnDxv1jlRYsfQsYCTqgg7MK3ngMk4xT/EfSs2wWr08J?=
 =?us-ascii?Q?o2xBDYM9tg416pgGqV7HNIcv0CXJCd6oEm5YDTRrDv4ydCDEhFYGw8mzANv3?=
 =?us-ascii?Q?+ye+dVWJQdszqzSDPIzyUpalIZXij0CxQoR//MrfDPfIahbrrnV6IFlzrl6d?=
 =?us-ascii?Q?WAP20KFtD3Ip59mWiTj0V4qO+p9DBDwZ8iXPuTiHhx+tx8XJcil8ftmtFaw/?=
 =?us-ascii?Q?VA3hnBiP0huqGhm85qDFxOC9qdxHW6LkaUWBu6ATLjJxEaGC5q7X+iN68YFM?=
 =?us-ascii?Q?XpzXkCiucgEK5eeR8Sw5q3Z+vHm27toJrhLtzpv5dlbIQRqbD23UJZu5F7DU?=
 =?us-ascii?Q?xFBrwk8rd+DZflcSLwGyyHrjU8JzMwGpp9MwrrhruJPuSuI1cedR7cJX2+JZ?=
 =?us-ascii?Q?38DZrSEseeWjIBF+pOa7wqyLUCmkytRlYKPRxLsK8fGJOg9i8D2ZVqtD/7Mg?=
 =?us-ascii?Q?GTZqK9I6Ckfr4j3Zdy4FEQpfe277vu+658GpzgHcuRw8Ej5wr9moCDvrZOEy?=
 =?us-ascii?Q?EA2SfwIOpWOJuhzZXrxBmSDrRoxZyGZJyLRKkEsbUYzTtc8bl4pOA1wLugpz?=
 =?us-ascii?Q?pPfgo13foDst2/C7iBYyNeZbewvaVoI0kH3DQYE9+aU6zOnNuUv3sJQCHYzI?=
 =?us-ascii?Q?O0dK0S9bytnQ5zLxuFGs2BVOg1Ce0uWnhdyjmDwrLceiaZlsFGPSUZj/UTk8?=
 =?us-ascii?Q?sQnW/Emd7LW5WcEa0aRb6o98F18af1S498rHAZmIllsSOefaQA7pElzH/dtH?=
 =?us-ascii?Q?zkGKAeT0sVuzcq2TJryPBk6SP3+xgPDqhz77PMVAYkgWpTUbGN0DOBcEPQYS?=
 =?us-ascii?Q?YVnOqepD2qCXjWP3dUOt72Z82ykzd/uxYNxvtKnyq5Hbfu0u4OkgwMc/wxYp?=
 =?us-ascii?Q?pQ=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: b5ba5746-2cc2-4a6c-d252-08dad4d049fd
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR11MB6201.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Dec 2022 01:47:08.6941
 (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: yZ1oLQTnj1X1hr1JQ6Z4iJF7DCh6CzVhKpgNKcTZ3SkhVTUvC2rLFEc5nDRbNyv0CvzVUFdomoALvcUu4F5x5Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7058
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,
        SPF_HELO_NONE,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

On Fri, Dec 02, 2022 at 03:53:52PM -0800, Mehta, Sohil wrote:
> On 11/29/2022 1:08 PM, Ashok Raj wrote:
> > -
> > -	if (apply_microcode_early(&uci, true)) {
> > -		/* Mixed-silicon system? Try to refetch the proper patch: */
> > -		*iup = NULL;
> > -
> > -		goto reget;
> > -	}
> > +	apply_microcode_early(&uci, true);
> 
> After this change, none of the callers of apply_microcode_early() check the
> return code.
> 
> In future, do we expect callers to care about the return code? The rest
> patches in this series don't seem to suggest so. Also, the expected error
> printing happens in the function itself.
> 
> Should the return type for apply_microcode_early() be changed to void (in a
> follow-up patch)?

Good idea.. But I think its early, the return code could be used for
something useful. I have some additional cleanup patches that I need to
fixup and we could use this for real.

For e.g. early loading failures are now reported by each vendor, if we can
consolidate this, we could do it more at a core level, but I'm worried it
might be too much change right now, and this can wait its turn. 

Cheers,
Ashok