Re: [PATCH v2 00/11] x86/paravirt: several cleanups

[Juergen Gross]

On 13/08/18 09:37, Juergen Gross wrote:
> This series removes some no longer needed stuff from paravirt
> infrastructure and puts large quantities of paravirt ops under a new
> config option PARAVIRT_XXL which is selected by XEN_PV only.
> 
> A pvops kernel without XEN_PV being configured is about 2.5% smaller
> with this series applied.
> 
> tip commit 5800dc5c19f34e6e03b5adab1282535cb102fafd ("x86/paravirt:
> Fix spectre-v2 mitigations for paravirt guests") is a prerequisite
> for this series.
> 
> The last 4 patches of this series require my Xen cleanup series
> https://lore.kernel.org/lkml/20180717120113.12756-1-jgross@xxxxxxxx/
> which hides more Xen PV-only code behind CONFIG_XEN_PV.
> 
> Changes in V2:
> - patch 4: shorten pv_ops sub-structure names (Jan Beulich)
> - patch 11: new patch
> 
> Juergen Gross (11):
>   x86/paravirt: make paravirt_patch_call() and paravirt_patch_jmp()
>     static
>   x86/paravirt: remove clobbers parameter from paravirt patch functions
>   x86/paravirt: remove clobbers from struct paravirt_patch_site
>   x86/paravirt: use a single ops structure
>   x86/paravirt: remove unused paravirt bits
>   x86/paravirt: introduce new config option PARAVIRT_XXL
>   x86/paravirt: move items in pv_info under PARAVIRT_XXL umbrella
>   x86/paravirt: move the Xen-only pv_cpu_ops under the PARAVIRT_XXL
>     umbrella
>   x86/paravirt: move the Xen-only pv_irq_ops under the PARAVIRT_XXL
>     umbrella
>   x86/paravirt: move the Xen-only pv_mmu_ops under the PARAVIRT_XXL
>     umbrella
>   x86/paravirt: remove unneeded mmu related paravirt ops bits
> 
>  arch/x86/Kconfig                            |   3 +
>  arch/x86/hyperv/mmu.c                       |   4 +-
>  arch/x86/include/asm/debugreg.h             |   2 +-
>  arch/x86/include/asm/desc.h                 |   4 +-
>  arch/x86/include/asm/fixmap.h               |   2 +-
>  arch/x86/include/asm/irqflags.h             |  56 ++--
>  arch/x86/include/asm/mmu_context.h          |   4 +-
>  arch/x86/include/asm/msr.h                  |   4 +-
>  arch/x86/include/asm/paravirt.h             | 399 +++++++++++++---------------
>  arch/x86/include/asm/paravirt_types.h       |  77 +++---
>  arch/x86/include/asm/pgalloc.h              |   2 +-
>  arch/x86/include/asm/pgtable-3level_types.h |   2 +-
>  arch/x86/include/asm/pgtable.h              |   7 +-
>  arch/x86/include/asm/processor.h            |   4 +-
>  arch/x86/include/asm/ptrace.h               |   3 +-
>  arch/x86/include/asm/segment.h              |   2 +-
>  arch/x86/include/asm/special_insns.h        |   4 +-
>  arch/x86/kernel/alternative.c               |   2 +-
>  arch/x86/kernel/asm-offsets.c               |  13 +-
>  arch/x86/kernel/asm-offsets_64.c            |   9 +-
>  arch/x86/kernel/cpu/common.c                |   4 +-
>  arch/x86/kernel/cpu/vmware.c                |   4 +-
>  arch/x86/kernel/head_64.S                   |   2 +-
>  arch/x86/kernel/kvm.c                       |  17 +-
>  arch/x86/kernel/kvmclock.c                  |   4 +-
>  arch/x86/kernel/paravirt-spinlocks.c        |  15 +-
>  arch/x86/kernel/paravirt.c                  | 292 ++++++++++----------
>  arch/x86/kernel/paravirt_patch_32.c         |  57 ++--
>  arch/x86/kernel/paravirt_patch_64.c         |  65 ++---
>  arch/x86/kernel/tsc.c                       |   2 +-
>  arch/x86/kernel/vsmp_64.c                   |  24 +-
>  arch/x86/xen/Kconfig                        |   1 +
>  arch/x86/xen/enlighten_pv.c                 |  31 ++-
>  arch/x86/xen/irq.c                          |   2 +-
>  arch/x86/xen/mmu_hvm.c                      |   2 +-
>  arch/x86/xen/mmu_pv.c                       |  28 +-
>  arch/x86/xen/spinlock.c                     |  11 +-
>  arch/x86/xen/time.c                         |   4 +-
>  drivers/xen/time.c                          |   2 +-
>  39 files changed, 575 insertions(+), 595 deletions(-)
> 

Ping?


Juergen