On 2025-05-06 09:48, Thomas Huth wrote:
> On 06/05/2025 08.48, Nina Schoetterl-Glausch wrote:
> > On Mon, 2025-05-05 at 08:55 +0200, Shalini Chellathurai Saroja wrote:
> > > On 2025-04-28 11:22, Janis Schoetterl-Glausch wrote:
> > > > On Thu, 2025-04-10 at 17:09 +0200, Shalini Chellathurai
> > > > Saroja wrote:
> > > > > Implement the Service-Call Logical Processor (SCLP) event
> > > > > type Control-Program Identification (CPI) in QEMU. This
> > > > > event is used to send CPI identifiers from the guest to the
> > > > > host. The CPI identifiers are: system type, system name,
> > > > > system level and sysplex name.
> > > > >
> > > > > System type: operating system of the guest (e.g. "LINUX").
> > > > > System name: user configurable name of the guest (e.g. "TESTVM").
> > > > > System level: distribution and kernel version, if the
> > > > > system type is
> > > > > Linux
> > > > > (e.g. 0x50e00).
> > > > > Sysplex name: name of the cluster which the guest
> > > > > belongs to (if any)
> > > > > (e.g. "PLEX").
> > > > >
> > > > > Signed-off-by: Shalini Chellathurai Saroja <shal...@linux.ibm.com>
> > > > > Reviewed-by: Thomas Huth <th...@redhat.com>
> > > > > ---
> > > > > hw/s390x/event-facility.c | 2 +
> > > > > hw/s390x/meson.build | 1 +
> > > > > hw/s390x/s390-virtio-ccw.c | 14 +++++
> > > > > hw/s390x/sclpcpi.c | 92
> > > > > +++++++++++++++++++++++++++++++
> > > > > include/hw/s390x/event-facility.h | 13 +++++
> > > > > 5 files changed, 122 insertions(+)
> > > > > create mode 100644 hw/s390x/sclpcpi.c
> > > > >
> > > > > diff --git a/hw/s390x/event-facility.c b/hw/s390x/event-facility.c
> > > > > index 2b0332c20e..60237b8581 100644
> > > > > --- a/hw/s390x/event-facility.c
> > > > > +++ b/hw/s390x/event-facility.c
> >
> > [...]
> >
> > > It is possible to add the identifiers directly as the properties of
> > > sclpcpi (eg. system type as shown below) and remove the
> > > control-program-id.
> >
> > This is what I meant, drop it from qom.
Ok.
> > >
> > > virsh # qemu-monitor-command vm --pretty
> > > '{"execute":"qom-list","arguments": {"path":
> > > "/machine/sclp/s390-sclp-event-facility/sclpcpi"}}'
> > > {
> > > "return": [
> > > {
> > > "name": "type",
> > > "type": "string"
> > > },
> > > {
> > > "name": "parent_bus",
> > > "type": "link<bus>"
> > > },
> > > {
> > > "name": "realized",
> > > "type": "bool"
> > > },
> > > {
> > > "name": "hotplugged",
> > > "type": "bool"
> > > },
> > > {
> > > "name": "hotpluggable",
> > > "type": "bool"
> > > },
> > > {
> > > "name": "system_type",
> > > "type": "string"
> > > },
> > > {
> > > "name": "control-program-id",
> > > "type": "S390ControlProgramId"
> > > }
> > > ],
> > > "id": "libvirt-16"
> > > }
> > >
> > > virsh # qemu-monitor-command vm --pretty '{"execute":"qom-get",
> > > "arguments":{"path":"/machine/sclp/s390-sclp-event-facility/sclpcpi",
> > > "property":"system_type"}}'
> > > {
> > > "return": "LINUX ",
> > > "id": "libvirt-17"
> > > }
> > >
> > > However, Hendrik Brückner suggested to group the identifiers together
> > > during the initial discussion of this line item. So, I would
> > > prefer to
> >
> > They are grouped together under "sclpcpi", no?
Yes.
> >
> > > leave this as it is. Please let me know if you still think otherwise.
> >
> > I don't have a strong opinion on this, just wanted to mention it and
> > see what other people say.
>
> I guess it mostly depends on whether there are future plans for using
> the QOM type elsewhere. If this type is supposed to be used in other
> QAPI calls in the future, it makes sense to keep it separate now. If
> there are no such plans, it's maybe easier to integrate the values
> into sclpcpi directly. Hendrik, any opinion from your side?
>
> Thomas
Hello Hendrik,
I have provided the output of both the options below. Please let me
know
which option do you prefer.
I have incorporated the other comments. So I will be able to send V5
after
your response. Thank you very much!
Option 1: Control-program identifiers set as the properties of
sclpcpi:
virsh # qemu-monitor-command vm --pretty
'{"execute":"qom-list","arguments":
{"path": "/machine/sclp/s390-sclp-event-facility/sclpcpi"}}'
{
"return": [
{
[...]
},
{
"name": "system_level",
"type": "uint64"
},
{
"name": "system_name",
"type": "string"
},
{
"name": "system_type",
"type": "string"
},
{
"name": "timestamp",
"type": "uint64"
},
{
"name": "sysplex_name",
"type": "string"
}
],
"id": "libvirt-14"
}
