[PATCH 0/3] X86/eager-fpu: Switch to eager fpu save/restore

Fouad Hilly posted 3 patches 1 month, 4 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20240304091307.2295344-1-fouad.hilly@cloud.com
xen/arch/x86/cpu/common.c                    |   3 -
xen/arch/x86/domain.c                        |  10 +-
xen/arch/x86/domctl.c                        |   4 +-
xen/arch/x86/hvm/emulate.c                   |  38 +----
xen/arch/x86/hvm/hvm.c                       |   8 +-
xen/arch/x86/hvm/svm/nestedsvm.c             |  62 +------
xen/arch/x86/hvm/svm/svm.c                   |  81 +--------
xen/arch/x86/hvm/svm/vmcb.c                  |   4 +-
xen/arch/x86/hvm/vlapic.c                    |   4 -
xen/arch/x86/hvm/vmx/vmcs.c                  |   8 +-
xen/arch/x86/hvm/vmx/vmx.c                   |  70 +-------
xen/arch/x86/hvm/vmx/vvmx.c                  |  15 +-
xen/arch/x86/i387.c                          | 170 +++++--------------
xen/arch/x86/include/asm/domain.h            |   3 -
xen/arch/x86/include/asm/hvm/svm/nestedsvm.h |   3 -
xen/arch/x86/include/asm/hvm/vmx/vmcs.h      |   2 -
xen/arch/x86/include/asm/i387.h              |   3 +-
xen/arch/x86/include/asm/xstate.h            |  18 +-
xen/arch/x86/pv/misc-hypercalls.c            |   3 +-
xen/arch/x86/traps.c                         |  18 +-
xen/arch/x86/xstate.c                        |  65 ++++---
xen/common/domain.c                          |   2 -
xen/common/efi/runtime.c                     |   9 +-
xen/include/xen/sched.h                      |   4 -
24 files changed, 108 insertions(+), 499 deletions(-)
[PATCH 0/3] X86/eager-fpu: Switch to eager fpu save/restore
Posted by Fouad Hilly 1 month, 4 weeks ago
X86 Xen will only eagerly switch FPU context in the scheduler.
Xen itslef won't set CR0.TS other than for the purpose of servicing
a PV guset.

Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
Signed-off-by: Fouad Hilly <fouad.hilly@cloud.com>
---
CC: Jan Beulich <jbeulich@suse.com>
CC: Andrew Cooper <andrew.cooper3@citrix.com>
CC: "Roger Pau Monné" <roger.pau@citrix.com>
CC: Wei Liu <wl@xen.org>
CC: George Dunlap <george.dunlap@citrix.com>
CC: Julien Grall <julien@xen.org>
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: Paul Durrant <paul@xen.org>
CC: Jun Nakajima <jun.nakajima@intel.com>
CC: Kevin Tian <kevin.tian@intel.com>

Wei Liu (3):
  x86: i387.c cleanup
  x86: introduce xstate_zero
  x86: switch to eager fpu save / restore

 xen/arch/x86/cpu/common.c                    |   3 -
 xen/arch/x86/domain.c                        |  10 +-
 xen/arch/x86/domctl.c                        |   4 +-
 xen/arch/x86/hvm/emulate.c                   |  38 +----
 xen/arch/x86/hvm/hvm.c                       |   8 +-
 xen/arch/x86/hvm/svm/nestedsvm.c             |  62 +------
 xen/arch/x86/hvm/svm/svm.c                   |  81 +--------
 xen/arch/x86/hvm/svm/vmcb.c                  |   4 +-
 xen/arch/x86/hvm/vlapic.c                    |   4 -
 xen/arch/x86/hvm/vmx/vmcs.c                  |   8 +-
 xen/arch/x86/hvm/vmx/vmx.c                   |  70 +-------
 xen/arch/x86/hvm/vmx/vvmx.c                  |  15 +-
 xen/arch/x86/i387.c                          | 170 +++++--------------
 xen/arch/x86/include/asm/domain.h            |   3 -
 xen/arch/x86/include/asm/hvm/svm/nestedsvm.h |   3 -
 xen/arch/x86/include/asm/hvm/vmx/vmcs.h      |   2 -
 xen/arch/x86/include/asm/i387.h              |   3 +-
 xen/arch/x86/include/asm/xstate.h            |  18 +-
 xen/arch/x86/pv/misc-hypercalls.c            |   3 +-
 xen/arch/x86/traps.c                         |  18 +-
 xen/arch/x86/xstate.c                        |  65 ++++---
 xen/common/domain.c                          |   2 -
 xen/common/efi/runtime.c                     |   9 +-
 xen/include/xen/sched.h                      |   4 -
 24 files changed, 108 insertions(+), 499 deletions(-)

-- 
2.42.0


Re: [PATCH 0/3] X86/eager-fpu: Switch to eager fpu save/restore
Posted by Jan Beulich 1 month, 4 weeks ago
On 04.03.2024 10:13, Fouad Hilly wrote:
> X86 Xen will only eagerly switch FPU context in the scheduler.
> Xen itslef won't set CR0.TS other than for the purpose of servicing
> a PV guset.
> 
> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> Signed-off-by: Fouad Hilly <fouad.hilly@cloud.com>

Throughout replying to the series I got back "undeliverable" responses
for Wei's old address at Citrix (noticed only now when I looked at what
has accumulated again in my inbox in the meantime). Can you please avoid
having dead addresses in patch submissions?

Jan