x86/ucode: Minor cleanup

[PATCH 0/5] x86/ucode: Minor cleanup

Andrew Cooper posted 5 patches 7 months, 3 weeks ago

Download series mbox

Failed in applying to current master (apply log)

xen/arch/x86/cpu/microcode/amd.c     | 11 ++---
xen/arch/x86/cpu/microcode/core.c    | 72 +++++++++++++++-------------
xen/arch/x86/cpu/microcode/intel.c   | 12 ++---
xen/arch/x86/cpu/microcode/private.h |  9 ++--
xen/arch/x86/include/asm/microcode.h |  4 +-
xen/arch/x86/platform_hypercall.c    |  6 +--
6 files changed, 59 insertions(+), 55 deletions(-)

Expand all Fold all

[PATCH 0/5] x86/ucode: Minor cleanup

Posted by Andrew Cooper 7 months, 3 weeks ago

Misc cleanup without functional change.  Mostly to improve clarity.

Andrew Cooper (5):
  x86/ucode: Rename hypercall-context functions
  x86/ucode: Drop the parse_blob() wrapper
  x86/ucode: Rename the cpu_request_microcode() hook to parse()
  x86/ucode: Rename the apply_microcode() hook to load()
  x86/ucode: Drop the match_reg[] field from AMD's microcode_patch

 xen/arch/x86/cpu/microcode/amd.c     | 11 ++---
 xen/arch/x86/cpu/microcode/core.c    | 72 +++++++++++++++-------------
 xen/arch/x86/cpu/microcode/intel.c   | 12 ++---
 xen/arch/x86/cpu/microcode/private.h |  9 ++--
 xen/arch/x86/include/asm/microcode.h |  4 +-
 xen/arch/x86/platform_hypercall.c    |  6 +--
 6 files changed, 59 insertions(+), 55 deletions(-)


base-commit: a974725a87a1afc8056b41c56dfe7fe272a7169c
-- 
2.39.5

Re: [PATCH 0/5] x86/ucode: Minor cleanup

Posted by Alejandro Vallejo 7 months, 3 weeks ago

On Thu Oct 24, 2024 at 2:22 PM BST, Andrew Cooper wrote:
> Misc cleanup without functional change.  Mostly to improve clarity.
>
> Andrew Cooper (5):
>   x86/ucode: Rename hypercall-context functions
>   x86/ucode: Drop the parse_blob() wrapper
>   x86/ucode: Rename the cpu_request_microcode() hook to parse()
>   x86/ucode: Rename the apply_microcode() hook to load()
>   x86/ucode: Drop the match_reg[] field from AMD's microcode_patch
>
>  xen/arch/x86/cpu/microcode/amd.c     | 11 ++---
>  xen/arch/x86/cpu/microcode/core.c    | 72 +++++++++++++++-------------
>  xen/arch/x86/cpu/microcode/intel.c   | 12 ++---
>  xen/arch/x86/cpu/microcode/private.h |  9 ++--
>  xen/arch/x86/include/asm/microcode.h |  4 +-
>  xen/arch/x86/platform_hypercall.c    |  6 +--
>  6 files changed, 59 insertions(+), 55 deletions(-)
>
>
> base-commit: a974725a87a1afc8056b41c56dfe7fe272a7169c

IMO, this change warrants renaming the parent directory to ucode and
microcode.h to ucode.h.

Cheers,
Alejandro