x86/ucode: Add extra rows to the entrysign model table (again)

[PATCH for-4.21] x86/ucode: Add extra rows to the entrysign model table (again)

Posted by Andrew Cooper 4 days, 9 hours ago

Link: https://git.kernel.org/tip/dd14022a7ce96963aa923e35cf4bcc8c32f95840
Fixes: ff8228ab4658 ("x86/ucode: Relax digest check when Entrysign is fixed in firmware")
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Jan Beulich <JBeulich@suse.com>
CC: Roger Pau Monné <roger.pau@citrix.com>
CC: Oleksii Kurochko <oleksii.kurochko@gmail.com>

Found because AMD have released ucode to fix RDSEED on this CPU, and I
cross-referenced the tables.

Again, for 4.21 even at this juncture.  It wants backporting everywhere in due
course.
---
 xen/arch/x86/cpu/microcode/amd.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/xen/arch/x86/cpu/microcode/amd.c b/xen/arch/x86/cpu/microcode/amd.c
index 550b8c1e57ec..4a7573c885e5 100644
--- a/xen/arch/x86/cpu/microcode/amd.c
+++ b/xen/arch/x86/cpu/microcode/amd.c
@@ -659,6 +659,7 @@ void __init amd_check_entrysign(void)
     case 0x0b1010: fixed_rev = 0x47; break;
     case 0x0b2040: fixed_rev = 0x32; break;
     case 0x0b4040: fixed_rev = 0x32; break;
+    case 0x0b4041: fixed_rev = 0x02; break;
     case 0x0b6000: fixed_rev = 0x32; break;
     case 0x0b6080: fixed_rev = 0x32; break;
     case 0x0b7000: fixed_rev = 0x32; break;
-- 
2.39.5

Re: [PATCH for-4.21] x86/ucode: Add extra rows to the entrysign model table (again)

Posted by Jan Beulich 1 day, 14 hours ago

On 14.11.2025 16:24, Andrew Cooper wrote:
> Link: https://git.kernel.org/tip/dd14022a7ce96963aa923e35cf4bcc8c32f95840
> Fixes: ff8228ab4658 ("x86/ucode: Relax digest check when Entrysign is fixed in firmware")
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Acked-by: Jan Beulich <jbeulich@suse.com>

> ---
> CC: Jan Beulich <JBeulich@suse.com>
> CC: Roger Pau Monné <roger.pau@citrix.com>
> CC: Oleksii Kurochko <oleksii.kurochko@gmail.com>
> 
> Found because AMD have released ucode to fix RDSEED on this CPU, and I
> cross-referenced the tables.

I wonder how many more we're going to see.

Jan

Re: [PATCH for-4.21] x86/ucode: Add extra rows to the entrysign model table (again)

Posted by Andrew Cooper 1 day, 12 hours ago

On 17/11/2025 9:43 am, Jan Beulich wrote:
> On 14.11.2025 16:24, Andrew Cooper wrote:
>> Link: https://git.kernel.org/tip/dd14022a7ce96963aa923e35cf4bcc8c32f95840
>> Fixes: ff8228ab4658 ("x86/ucode: Relax digest check when Entrysign is fixed in firmware")
>> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> Acked-by: Jan Beulich <jbeulich@suse.com>

Thanks.

>> ---
>> CC: Jan Beulich <JBeulich@suse.com>
>> CC: Roger Pau Monné <roger.pau@citrix.com>
>> CC: Oleksii Kurochko <oleksii.kurochko@gmail.com>
>>
>> Found because AMD have released ucode to fix RDSEED on this CPU, and I
>> cross-referenced the tables.
> I wonder how many more we're going to see.

I doubt this will be the last.

~Andrew

Re: [PATCH for-4.21] x86/ucode: Add extra rows to the entrysign model table (again)

Posted by Oleksii Kurochko 4 days, 7 hours ago

On 11/14/25 4:24 PM, Andrew Cooper wrote:
> Link:https://git.kernel.org/tip/dd14022a7ce96963aa923e35cf4bcc8c32f95840
> Fixes: ff8228ab4658 ("x86/ucode: Relax digest check when Entrysign is fixed in firmware")
> Signed-off-by: Andrew Cooper<andrew.cooper3@citrix.com>
> ---
> CC: Jan Beulich<JBeulich@suse.com>
> CC: Roger Pau Monné<roger.pau@citrix.com>
> CC: Oleksii Kurochko<oleksii.kurochko@gmail.com>
>
> Found because AMD have released ucode to fix RDSEED on this CPU, and I
> cross-referenced the tables.
>
> Again, for 4.21 even at this juncture.  It wants backporting everywhere in due
> course.

The same as for previous/another patch:
   Release-Acked-By: Oleksii Kurochko<oleksii.kurochko@gmail.com>

Thanks.

~ Oleksii

> ---
>   xen/arch/x86/cpu/microcode/amd.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/xen/arch/x86/cpu/microcode/amd.c b/xen/arch/x86/cpu/microcode/amd.c
> index 550b8c1e57ec..4a7573c885e5 100644
> --- a/xen/arch/x86/cpu/microcode/amd.c
> +++ b/xen/arch/x86/cpu/microcode/amd.c
> @@ -659,6 +659,7 @@ void __init amd_check_entrysign(void)
>       case 0x0b1010: fixed_rev = 0x47; break;
>       case 0x0b2040: fixed_rev = 0x32; break;
>       case 0x0b4040: fixed_rev = 0x32; break;
> +    case 0x0b4041: fixed_rev = 0x02; break;
>       case 0x0b6000: fixed_rev = 0x32; break;
>       case 0x0b6080: fixed_rev = 0x32; break;
>       case 0x0b7000: fixed_rev = 0x32; break;