Re: [PATCH v13 0/7] s390x: CPU Topology

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/8/22 10:44, Pierre Morel wrote:
Hi,

Implementation discussions
==========================

CPU models
----------

Since the S390_FEAT_CONFIGURATION_TOPOLOGY is already in the CPU model
for old QEMU we could not activate it as usual from KVM but needed
a KVM capability: KVM_CAP_S390_CPU_TOPOLOGY.
Checking and enabling this capability enables
S390_FEAT_CONFIGURATION_TOPOLOGY.

Migration
---------

Once the S390_FEAT_CONFIGURATION_TOPOLOGY is enabled in the source
host the STFL(11) is provided to the guest.
Since the feature is already in the CPU model of older QEMU,
a migration from a new QEMU enabling the topology to an old QEMU
will keep STFL(11) enabled making the guest get an exception for
illegal operation as soon as it uses the PTF instruction.

A VMState keeping track of the S390_FEAT_CONFIGURATION_TOPOLOGY
allows to forbid the migration in such a case.

Note that the VMState will be used to hold information on the
topology once we implement topology change for a running guest.

Topology
--------

Until we introduce bookss and drawers, polarization and dedication
the topology is kept very simple and is specified uniquely by
the core_id of the vCPU which is also the vCPU address.

Testing
=======

To use the QEMU patches, you will need Linux V6-rc1 or newer,
or use the following Linux mainline patches:

f5ecfee94493 2022-07-20 KVM: s390: resetting the Topology-Change-Report
24fe0195bc19 2022-07-20 KVM: s390: guest support for topology function
0130337ec45b 2022-07-20 KVM: s390: Cleanup ipte lock access and SIIF fac..

Currently this code is for KVM only, I have no idea if it is interesting
to provide a TCG patch. If ever it will be done in another series.

Documentation
=============

To have a better understanding of the S390x CPU Topology and its
implementation in QEMU you can have a look at the documentation in the
last patch of this series.

The admin will want to match the host and the guest topology, taking
into account that the guest does not recognize multithreading.
Consequently, two vCPU assigned to threads of the same real CPU should
preferably be assigned to the same socket of the guest machine.

Future developments
===================

Two series are actively prepared:
- Adding drawers, book, polarization and dedication to the vCPU.
- changing the topology with a running guest

Since we have time before the next QEMU 8.0 release, could you please
send the whole patchset ? Having the full picture would help in taking
decisions for downstream also.

I am still uncertain about the usefulness of S390Topology object because,
as for now, the state can be computed on the fly, the reset can be
handled at the machine level directly under s390_machine_reset() and so
could migration if the machine had a vmstate (not the case today but
quite easy to add). So before merging anything that could be difficult
to maintain and/or backport, I would prefer to see it all !

Thanks,

C.



Regards,
Pierre

Pierre Morel (7):
   s390x/cpu topology: Creating CPU topology device
   s390x/cpu topology: reporting the CPU topology to the guest
   s390x/cpu_topology: resetting the Topology-Change-Report
   s390x/cpu_topology: CPU topology migration
   s390x/cpu_topology: interception of PTF instruction
   s390x/cpu_topology: activating CPU topology
   docs/s390x: document s390x cpu topology

  docs/system/s390x/cpu-topology.rst |  87 ++++++++++
  docs/system/target-s390x.rst       |   1 +
  include/hw/s390x/cpu-topology.h    |  52 ++++++
  include/hw/s390x/s390-virtio-ccw.h |   6 +
  target/s390x/cpu.h                 |  78 +++++++++
  target/s390x/kvm/kvm_s390x.h       |   1 +
  hw/s390x/cpu-topology.c            | 261 +++++++++++++++++++++++++++++
  hw/s390x/s390-virtio-ccw.c         |   7 +
  target/s390x/cpu-sysemu.c          |  21 +++
  target/s390x/cpu_models.c          |   1 +
  target/s390x/kvm/cpu_topology.c    | 186 ++++++++++++++++++++
  target/s390x/kvm/kvm.c             |  46 ++++-
  hw/s390x/meson.build               |   1 +
  target/s390x/kvm/meson.build       |   3 +-
  14 files changed, 749 insertions(+), 2 deletions(-)
  create mode 100644 docs/system/s390x/cpu-topology.rst
  create mode 100644 include/hw/s390x/cpu-topology.h
  create mode 100644 hw/s390x/cpu-topology.c
  create mode 100644 target/s390x/kvm/cpu_topology.c





[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux