[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/4] x86/ucode: Use altcall, and __initdata_cf_clobber
- To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Wed, 1 Dec 2021 09:23:21 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=r8WoZLuvw6Be1Wvo2zfIpdJ59RRLJlaWGE7mfQefSdU=; b=TPt89Fp2Y8cEx5SYA7PzN2vAMDUx6YHGBgWnpd6F+NwOexob0M42ktvpjn/p0rTBWe2HPi1Kli3kUrqWymhPUWJjYavFiSVoXMiY7ghlmQthg5RV7VH7kloz9iAjvzj1WP3kIOm20p+ST8sF2tCT/PmVYZtnNdlUGy6RLsDBiRiX8kyG8g0seOpaUnEAzTxutc1h7LgO62nFjIZltgDO4p8POTXiXHH0C4s6N+kiBf39ZBF3GqKmFs6O55FdAFHGD+z0QLOiLYtIK/PeaIz2bKrCD80UUkwuzN/TyAMAwKVa0DAlhMUsUuusH5htEng/xbgF6ypJmX6mch1K7TF81A==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FiEsWaSLxqwcgDx79wP+621u3jlXCGyMo3hTbVlGSzG4JbEfo1LkWG4G5XOYiEbAZqsx8i41nOX2q0OxS+ZD7hB77N3uI7pYQEh/glZQRMS1qbdAVdvOVKFQUBHe0P6o9D43ojIDIkSg6YCeG0xzkw7rlT9h3phI3+3BYNHMxDycTAGZi+ydguE9a9JqmoDEo+1D3Ll9Ok8A7LM/dLvE7e+uVvIVGwSvPkonhv2r/aw6oippERolRfYxX0+g1051Im/PqMlFmTXJRmzzFr68/A+nZtEOHL92z3T4CQUebw8j6j0hia7Ul+ZBmwaFZ0xC+z5Ge982dNko8Ah3KCJz/A==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Wed, 01 Dec 2021 08:23:38 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 26.11.2021 22:22, Andrew Cooper wrote:
> Microcode loading is not a fastpath, but there are control flow security
> benefits from using altcall()'s hardening side effect.
>
> Convert the existing microcode_ops pointer into a __read_mostly structure, and
> move {amd,intel}_ucode_ops into __initdata_cf_clobber.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Again subject to the resolution of the const aspect and perhaps
with __read_mostly converted to __ro_after_init (assuming its
introduction goes in first)
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
Jan
|
