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

2018-08-24 Thread Peter Zijlstra
On Fri, Aug 24, 2018 at 03:52:55PM +0200, Juergen Gross wrote:
> Ping?

Looking good; although I messed it up a little bit by adding a new
paravirt function.

Thanks for doing this!

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

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

2018-08-24 Thread 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-jgr...@suse.com/
> 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

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

[Xen-devel] [PATCH v2 00/11] x86/paravirt: several cleanups

2018-08-13 Thread Juergen Gross
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-jgr...@suse.com/
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(-)

-- 
2.13.7


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