Re: [qemu-s390x] [PATCH v6 3/5] s390x/ap: base Adjunct Processor (AP) object model

[Cornelia Huck]

On Fri, 29 Jun 2018 18:48:01 -0400
Tony Krowiak <address@hidden> wrote:

> From: Tony Krowiak <address@hidden>

You may want to switch your sending address to this one as well.

> 
> Introduces the base object model for virtualizing AP devices.
> 
> Signed-off-by: Tony Krowiak <address@hidden>
> ---
>  MAINTAINERS                  |   12 +++++++
>  hw/s390x/Makefile.objs       |    2 +
>  hw/s390x/ap-bridge.c         |   76 
> ++++++++++++++++++++++++++++++++++++++++++
>  hw/s390x/ap-device.c         |   39 +++++++++++++++++++++
>  hw/s390x/s390-virtio-ccw.c   |    4 ++
>  include/hw/s390x/ap-bridge.h |   37 ++++++++++++++++++++
>  include/hw/s390x/ap-device.h |   38 +++++++++++++++++++++
>  7 files changed, 208 insertions(+), 0 deletions(-)
>  create mode 100644 hw/s390x/ap-bridge.c
>  create mode 100644 hw/s390x/ap-device.c
>  create mode 100644 include/hw/s390x/ap-bridge.h
>  create mode 100644 include/hw/s390x/ap-device.h
> 

> diff --git a/hw/s390x/ap-bridge.c b/hw/s390x/ap-bridge.c
> new file mode 100644
> index 0000000..4f10425
> --- /dev/null
> +++ b/hw/s390x/ap-bridge.c
> @@ -0,0 +1,76 @@
> +/*
> + * ap bridge
> + *
> + * Copyright 2018 IBM Corp.
> + * Author(s): Halil Pasic <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or (at
> + * your option) any later version. See the COPYING file in the top-level
> + * directory.
> + */
> +#include "qemu/osdep.h"
> +#include "qapi/error.h"
> +#include "hw/sysbus.h"
> +#include "qemu/bitops.h"
> +#include "hw/s390x/ap-bridge.h"
> +#include "cpu.h"
> +
> +static char *vfio_ap_bus_get_dev_path(DeviceState *dev)
> +{
> +    /* at most one */
> +    return g_strdup_printf("/1");

Looks a bit odd... add 'ap'? Start counting with 0?

> +}
> +

(...)

> diff --git a/include/hw/s390x/ap-bridge.h b/include/hw/s390x/ap-bridge.h
> new file mode 100644
> index 0000000..7841ec2
> --- /dev/null
> +++ b/include/hw/s390x/ap-bridge.h
> @@ -0,0 +1,37 @@
> +/*
> + * ap bridge
> + *
> + * Copyright 2018 IBM Corp.
> + * Author(s): Halil Pasic <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or (at
> + * your option) any later version. See the COPYING file in the top-level
> + * directory.
> + */
> +
> +#ifndef HW_S390X_AP_BRIDGE_H
> +#define HW_S390X_AP_BRIDGE_H
> +#include "qom/object.h"
> +#include "hw/qdev-core.h"
> +#include "hw/sysbus.h"
> +
> +typedef struct APBridge {
> +    SysBusDevice sysbus_dev;
> +    bool css_dev_path;

Probable copy-and-paste error?

> +} APBridge;
> +
> +#define TYPE_AP_BRIDGE "ap-bridge"
> +#define AP_BRIDGE(obj) \
> +    OBJECT_CHECK(APBridge, (obj), TYPE_AP_BRIDGE)
> +
> +typedef struct VFIOAPBus {
> +    BusState parent_obj;
> +} VFIOAPBus;
> +
> +#define TYPE_VFIO_AP_BUS "vfio-ap-bus"
> +#define VFIO_AP_BUS(obj) \
> +     OBJECT_CHECK(VFIOAPBus, (obj), TYPE_VFIO_AP_BUS)
> +
> +void s390_init_ap(void);
> +
> +#endif

Else, looks sane to me.