Re: [Xen-devel] [PATCH 1/2] x86/microcode: Indicate "not found" in rc of microcode_resume_cpu()

[Jan Beulich]

>>> On 11.04.18 at 22:14,  wrote:
> Make it possible to distinguish between a failure to load a microcode
> update and a failure to find any matching microcode update by returning
> -ENOENT (instead of -EIO) in the later case.
> 
> Signed-off-by: Simon Gaiser 
> ---
>  xen/arch/x86/microcode.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/xen/arch/x86/microcode.c b/xen/arch/x86/microcode.c
> index 77c1efc97f..e9fafb1d14 100644
> --- a/xen/arch/x86/microcode.c
> +++ b/xen/arch/x86/microcode.c
> @@ -248,7 +248,7 @@ int microcode_resume_cpu(unsigned int cpu)
>  __microcode_fini_cpu(cpu);
>  uci->cpu_sig = nsig;
>  
> -err = -EIO;
> +err = -ENOENT;
>  for_each_online_cpu ( cpu2 )
>  {
>  uci = &per_cpu(ucode_cpu_info, cpu2);

I think this should be part of the other patch, but I'll reply there
in a minute.

Jan


___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

[Xen-devel] [PATCH 1/2] x86/microcode: Indicate "not found" in rc of microcode_resume_cpu()

[Simon Gaiser]

Make it possible to distinguish between a failure to load a microcode
update and a failure to find any matching microcode update by returning
-ENOENT (instead of -EIO) in the later case.

Signed-off-by: Simon Gaiser 
---
 xen/arch/x86/microcode.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/microcode.c b/xen/arch/x86/microcode.c
index 77c1efc97f..e9fafb1d14 100644
--- a/xen/arch/x86/microcode.c
+++ b/xen/arch/x86/microcode.c
@@ -248,7 +248,7 @@ int microcode_resume_cpu(unsigned int cpu)
 __microcode_fini_cpu(cpu);
 uci->cpu_sig = nsig;
 
-err = -EIO;
+err = -ENOENT;
 for_each_online_cpu ( cpu2 )
 {
 uci = &per_cpu(ucode_cpu_info, cpu2);
-- 
2.16.2


___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel