[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 04/11] s390x: get rid of s390-virtio.c
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH v1 04/11] s390x: get rid of s390-virtio.c |
Date: |
Thu, 31 Aug 2017 13:47:18 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
I have not reviewed this, but the whole idea is
Acked-by: Christian Borntraeger <address@hidden>
On 08/30/2017 07:05 PM, David Hildenbrand wrote:
> It is a leftover from the days where we had still the !ccw virtio
> machine. As this one is long gone, let's move everything to
> s390-virtio-ccw.c.
>
> Cornelia Huck <address@hidden>
> Signed-off-by: David Hildenbrand <address@hidden>
> ---
> hw/s390x/Makefile.objs | 1 -
> hw/s390x/s390-virtio-ccw.c | 120 ++++++++++++++++++++++++++++++++-
> hw/s390x/s390-virtio.c | 163
> ---------------------------------------------
> hw/s390x/s390-virtio.h | 14 ----
> 4 files changed, 119 insertions(+), 179 deletions(-)
> delete mode 100644 hw/s390x/s390-virtio.c
>
> diff --git a/hw/s390x/Makefile.objs b/hw/s390x/Makefile.objs
> index 7ee19d3abc..dc704b57d6 100644
> --- a/hw/s390x/Makefile.objs
> +++ b/hw/s390x/Makefile.objs
> @@ -1,4 +1,3 @@
> -obj-y += s390-virtio.o
> obj-y += s390-virtio-hcall.o
> obj-y += sclp.o
> obj-y += event-facility.o
> diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c
> index ffd56af834..41a9e976dc 100644
> --- a/hw/s390x/s390-virtio-ccw.c
> +++ b/hw/s390x/s390-virtio-ccw.c
> @@ -2,6 +2,7 @@
> * virtio ccw machine
> *
> * Copyright 2012 IBM Corp.
> + * Copyright (c) 2009 Alexander Graf <address@hidden>
> * Author(s): Cornelia Huck <address@hidden>
> *
> * This work is licensed under the terms of the GNU GPL, version 2 or (at
> @@ -31,6 +32,8 @@
> #include "hw/s390x/css-bridge.h"
> #include "migration/register.h"
> #include "cpu_models.h"
> +#include "qapi/qmp/qerror.h"
> +#include "hw/nmi.h"
>
> S390CPU *s390_cpu_addr2state(uint16_t cpu_addr)
> {
> @@ -133,7 +136,7 @@ static void virtio_ccw_register_hcalls(void)
> virtio_ccw_hcall_early_printk);
> }
>
> -void s390_memory_init(ram_addr_t mem_size)
> +static void s390_memory_init(ram_addr_t mem_size)
> {
> MemoryRegion *sysmem = get_system_memory();
> MemoryRegion *ram = g_new(MemoryRegion, 1);
> @@ -148,11 +151,104 @@ void s390_memory_init(ram_addr_t mem_size)
> s390_stattrib_init();
> }
>
> +#define S390_TOD_CLOCK_VALUE_MISSING 0x00
> +#define S390_TOD_CLOCK_VALUE_PRESENT 0x01
> +
> +static void gtod_save(QEMUFile *f, void *opaque)
> +{
> + uint64_t tod_low;
> + uint8_t tod_high;
> + int r;
> +
> + r = s390_get_clock(&tod_high, &tod_low);
> + if (r) {
> + fprintf(stderr, "WARNING: Unable to get guest clock for migration. "
> + "Error code %d. Guest clock will not be migrated "
> + "which could cause the guest to hang.\n", r);
> + qemu_put_byte(f, S390_TOD_CLOCK_VALUE_MISSING);
> + return;
> + }
> +
> + qemu_put_byte(f, S390_TOD_CLOCK_VALUE_PRESENT);
> + qemu_put_byte(f, tod_high);
> + qemu_put_be64(f, tod_low);
> +}
> +
> +static int gtod_load(QEMUFile *f, void *opaque, int version_id)
> +{
> + uint64_t tod_low;
> + uint8_t tod_high;
> + int r;
> +
> + if (qemu_get_byte(f) == S390_TOD_CLOCK_VALUE_MISSING) {
> + fprintf(stderr, "WARNING: Guest clock was not migrated. This could "
> + "cause the guest to hang.\n");
> + return 0;
> + }
> +
> + tod_high = qemu_get_byte(f);
> + tod_low = qemu_get_be64(f);
> +
> + r = s390_set_clock(&tod_high, &tod_low);
> + if (r) {
> + fprintf(stderr, "WARNING: Unable to set guest clock value. "
> + "s390_get_clock returned error %d. This could cause "
> + "the guest to hang.\n", r);
> + }
> +
> + return 0;
> +}
> +
> +
> static SaveVMHandlers savevm_gtod = {
> .save_state = gtod_save,
> .load_state = gtod_load,
> };
>
> +static void s390_init_ipl_dev(const char *kernel_filename,
> + const char *kernel_cmdline,
> + const char *initrd_filename, const char
> *firmware,
> + const char *netboot_fw, bool enforce_bios)
> +{
> + Object *new = object_new(TYPE_S390_IPL);
> + DeviceState *dev = DEVICE(new);
> +
> + if (kernel_filename) {
> + qdev_prop_set_string(dev, "kernel", kernel_filename);
> + }
> + if (initrd_filename) {
> + qdev_prop_set_string(dev, "initrd", initrd_filename);
> + }
> + qdev_prop_set_string(dev, "cmdline", kernel_cmdline);
> + qdev_prop_set_string(dev, "firmware", firmware);
> + qdev_prop_set_string(dev, "netboot_fw", netboot_fw);
> + qdev_prop_set_bit(dev, "enforce_bios", enforce_bios);
> + object_property_add_child(qdev_get_machine(), TYPE_S390_IPL,
> + new, NULL);
> + object_unref(new);
> + qdev_init_nofail(dev);
> +}
> +
> +static void s390_create_virtio_net(BusState *bus, const char *name)
> +{
> + int i;
> +
> + for (i = 0; i < nb_nics; i++) {
> + NICInfo *nd = &nd_table[i];
> + DeviceState *dev;
> +
> + if (!nd->model) {
> + nd->model = g_strdup("virtio");
> + }
> +
> + qemu_check_nic_model(nd, "virtio");
> +
> + dev = qdev_create(bus, name);
> + qdev_set_nic_properties(dev, nd);
> + qdev_init_nofail(dev);
> + }
> +}
> +
> static void ccw_init(MachineState *machine)
> {
> int ret;
> @@ -216,6 +312,19 @@ static void s390_cpu_plug(HotplugHandler *hotplug_dev,
> g_free(name);
> }
>
> +static void s390_machine_reset(void)
> +{
> + S390CPU *ipl_cpu = S390_CPU(qemu_get_cpu(0));
> +
> + s390_cmma_reset();
> + qemu_devices_reset();
> + s390_crypto_reset();
> +
> + /* all cpus are stopped - configure and start the ipl cpu only */
> + s390_ipl_prepare_cpu(ipl_cpu);
> + s390_cpu_set_state(CPU_STATE_OPERATING, ipl_cpu);
> +}
> +
> static void s390_machine_device_plug(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> @@ -240,6 +349,15 @@ static void s390_hot_add_cpu(const int64_t id, Error
> **errp)
> s390x_new_cpu(machine->cpu_model, id, errp);
> }
>
> +static void s390_nmi(NMIState *n, int cpu_index, Error **errp)
> +{
> + CPUState *cs = qemu_get_cpu(cpu_index);
> +
> + if (s390_cpu_restart(S390_CPU(cs))) {
> + error_setg(errp, QERR_UNSUPPORTED);
> + }
> +}
> +
> static void ccw_machine_class_init(ObjectClass *oc, void *data)
> {
> MachineClass *mc = MACHINE_CLASS(oc);
> diff --git a/hw/s390x/s390-virtio.c b/hw/s390x/s390-virtio.c
> deleted file mode 100644
> index 464b5c71f8..0000000000
> --- a/hw/s390x/s390-virtio.c
> +++ /dev/null
> @@ -1,163 +0,0 @@
> -/*
> - * QEMU S390 virtio target
> - *
> - * Copyright (c) 2009 Alexander Graf <address@hidden>
> - * Copyright IBM Corp 2012
> - *
> - * This library is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU Lesser General Public
> - * License as published by the Free Software Foundation; either
> - * version 2 of the License, or (at your option) any later version.
> - *
> - * This library is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> - * Lesser General Public License for more details.
> - *
> - * Contributions after 2012-10-29 are licensed under the terms of the
> - * GNU GPL, version 2 or (at your option) any later version.
> - *
> - * You should have received a copy of the GNU (Lesser) General Public
> - * License along with this library; if not, see
> <http://www.gnu.org/licenses/>.
> - */
> -
> -#include "qemu/osdep.h"
> -#include "qapi/error.h"
> -#include "hw/hw.h"
> -#include "qapi/qmp/qerror.h"
> -#include "qemu/error-report.h"
> -#include "sysemu/block-backend.h"
> -#include "sysemu/blockdev.h"
> -#include "sysemu/sysemu.h"
> -#include "net/net.h"
> -#include "hw/boards.h"
> -#include "hw/loader.h"
> -#include "hw/virtio/virtio.h"
> -#include "exec/address-spaces.h"
> -#include "sysemu/qtest.h"
> -
> -#include "hw/s390x/sclp.h"
> -#include "hw/s390x/s390_flic.h"
> -#include "hw/s390x/s390-virtio.h"
> -#include "hw/s390x/storage-keys.h"
> -#include "hw/s390x/ipl.h"
> -#include "cpu.h"
> -
> -#define MAX_BLK_DEVS 10
> -
> -#define S390_TOD_CLOCK_VALUE_MISSING 0x00
> -#define S390_TOD_CLOCK_VALUE_PRESENT 0x01
> -
> -void s390_init_ipl_dev(const char *kernel_filename,
> - const char *kernel_cmdline,
> - const char *initrd_filename,
> - const char *firmware,
> - const char *netboot_fw,
> - bool enforce_bios)
> -{
> - Object *new = object_new(TYPE_S390_IPL);
> - DeviceState *dev = DEVICE(new);
> -
> - if (kernel_filename) {
> - qdev_prop_set_string(dev, "kernel", kernel_filename);
> - }
> - if (initrd_filename) {
> - qdev_prop_set_string(dev, "initrd", initrd_filename);
> - }
> - qdev_prop_set_string(dev, "cmdline", kernel_cmdline);
> - qdev_prop_set_string(dev, "firmware", firmware);
> - qdev_prop_set_string(dev, "netboot_fw", netboot_fw);
> - qdev_prop_set_bit(dev, "enforce_bios", enforce_bios);
> - object_property_add_child(qdev_get_machine(), TYPE_S390_IPL,
> - new, NULL);
> - object_unref(new);
> - qdev_init_nofail(dev);
> -}
> -
> -
> -void s390_create_virtio_net(BusState *bus, const char *name)
> -{
> - int i;
> -
> - for (i = 0; i < nb_nics; i++) {
> - NICInfo *nd = &nd_table[i];
> - DeviceState *dev;
> -
> - if (!nd->model) {
> - nd->model = g_strdup("virtio");
> - }
> -
> - qemu_check_nic_model(nd, "virtio");
> -
> - dev = qdev_create(bus, name);
> - qdev_set_nic_properties(dev, nd);
> - qdev_init_nofail(dev);
> - }
> -}
> -
> -void gtod_save(QEMUFile *f, void *opaque)
> -{
> - uint64_t tod_low;
> - uint8_t tod_high;
> - int r;
> -
> - r = s390_get_clock(&tod_high, &tod_low);
> - if (r) {
> - fprintf(stderr, "WARNING: Unable to get guest clock for migration. "
> - "Error code %d. Guest clock will not be migrated "
> - "which could cause the guest to hang.\n", r);
> - qemu_put_byte(f, S390_TOD_CLOCK_VALUE_MISSING);
> - return;
> - }
> -
> - qemu_put_byte(f, S390_TOD_CLOCK_VALUE_PRESENT);
> - qemu_put_byte(f, tod_high);
> - qemu_put_be64(f, tod_low);
> -}
> -
> -int gtod_load(QEMUFile *f, void *opaque, int version_id)
> -{
> - uint64_t tod_low;
> - uint8_t tod_high;
> - int r;
> -
> - if (qemu_get_byte(f) == S390_TOD_CLOCK_VALUE_MISSING) {
> - fprintf(stderr, "WARNING: Guest clock was not migrated. This could "
> - "cause the guest to hang.\n");
> - return 0;
> - }
> -
> - tod_high = qemu_get_byte(f);
> - tod_low = qemu_get_be64(f);
> -
> - r = s390_set_clock(&tod_high, &tod_low);
> - if (r) {
> - fprintf(stderr, "WARNING: Unable to set guest clock value. "
> - "s390_get_clock returned error %d. This could cause "
> - "the guest to hang.\n", r);
> - }
> -
> - return 0;
> -}
> -
> -void s390_nmi(NMIState *n, int cpu_index, Error **errp)
> -{
> - CPUState *cs = qemu_get_cpu(cpu_index);
> -
> - if (s390_cpu_restart(S390_CPU(cs))) {
> - error_setg(errp, QERR_UNSUPPORTED);
> - }
> -}
> -
> -void s390_machine_reset(void)
> -{
> - S390CPU *ipl_cpu = S390_CPU(qemu_get_cpu(0));
> -
> - s390_cmma_reset();
> - qemu_devices_reset();
> - s390_crypto_reset();
> -
> - /* all cpus are stopped - configure and start the ipl cpu only */
> - s390_ipl_prepare_cpu(ipl_cpu);
> - s390_cpu_set_state(CPU_STATE_OPERATING, ipl_cpu);
> -}
> diff --git a/hw/s390x/s390-virtio.h b/hw/s390x/s390-virtio.h
> index b6660e3ae9..d984cd4115 100644
> --- a/hw/s390x/s390-virtio.h
> +++ b/hw/s390x/s390-virtio.h
> @@ -12,23 +12,9 @@
> #ifndef HW_S390_VIRTIO_H
> #define HW_S390_VIRTIO_H
>
> -#include "hw/nmi.h"
> #include "standard-headers/asm-s390/kvm_virtio.h"
> #include "standard-headers/asm-s390/virtio-ccw.h"
>
> typedef int (*s390_virtio_fn)(const uint64_t *args);
> void s390_register_virtio_hypercall(uint64_t code, s390_virtio_fn fn);
> -
> -void s390_init_ipl_dev(const char *kernel_filename,
> - const char *kernel_cmdline,
> - const char *initrd_filename,
> - const char *firmware,
> - const char *netboot_fw,
> - bool enforce_bios);
> -void s390_create_virtio_net(BusState *bus, const char *name);
> -void s390_nmi(NMIState *n, int cpu_index, Error **errp);
> -void s390_machine_reset(void);
> -void s390_memory_init(ram_addr_t mem_size);
> -void gtod_save(QEMUFile *f, void *opaque);
> -int gtod_load(QEMUFile *f, void *opaque, int version_id);
> #endif
>
[Qemu-devel] [PATCH v1 04/11] s390x: get rid of s390-virtio.c, David Hildenbrand, 2017/08/30
[Qemu-devel] [PATCH v1 06/11] target/s390x: cleanup cpu number/address handling, David Hildenbrand, 2017/08/30
[Qemu-devel] [PATCH v1 05/11] s390x: rename s390-virtio.h to s390-virtio-hcall.h, David Hildenbrand, 2017/08/30