[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] hw/virtio/virtio-balloon: move adding prope
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] hw/virtio/virtio-balloon: move adding property to virtio_balloon_instance_init |
Date: |
Mon, 11 May 2015 12:13:35 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 11/05/2015 11:34, address@hidden wrote:
> From: Shannon Zhao <address@hidden>
>
> This is prepared for using alias property in virtio-balloon-pci and
> virtio-balloon-ccw.
English note: "This prepares for..."
Can be fixed by the committer.
Paolo
> Signed-off-by: Shannon Zhao <address@hidden>
> Signed-off-by: Shannon Zhao <address@hidden>
> ---
> hw/virtio/virtio-balloon.c | 22 ++++++++++++++--------
> 1 file changed, 14 insertions(+), 8 deletions(-)
>
> diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c
> index 484c3c3..cfff542 100644
> --- a/hw/virtio/virtio-balloon.c
> +++ b/hw/virtio/virtio-balloon.c
> @@ -396,14 +396,6 @@ static void virtio_balloon_device_realize(DeviceState
> *dev, Error **errp)
>
> register_savevm(dev, "virtio-balloon", -1, 1,
> virtio_balloon_save, virtio_balloon_load, s);
> -
> - object_property_add(OBJECT(dev), "guest-stats", "guest statistics",
> - balloon_stats_get_all, NULL, NULL, s, NULL);
> -
> - object_property_add(OBJECT(dev), "guest-stats-polling-interval", "int",
> - balloon_stats_get_poll_interval,
> - balloon_stats_set_poll_interval,
> - NULL, s, NULL);
> }
>
> static void virtio_balloon_device_unrealize(DeviceState *dev, Error **errp)
> @@ -417,6 +409,19 @@ static void virtio_balloon_device_unrealize(DeviceState
> *dev, Error **errp)
> virtio_cleanup(vdev);
> }
>
> +static void virtio_balloon_instance_init(Object *obj)
> +{
> + VirtIOBalloon *s = VIRTIO_BALLOON(obj);
> +
> + object_property_add(obj, "guest-stats", "guest statistics",
> + balloon_stats_get_all, NULL, NULL, s, NULL);
> +
> + object_property_add(obj, "guest-stats-polling-interval", "int",
> + balloon_stats_get_poll_interval,
> + balloon_stats_set_poll_interval,
> + NULL, s, NULL);
> +}
> +
> static Property virtio_balloon_properties[] = {
> DEFINE_PROP_END_OF_LIST(),
> };
> @@ -441,6 +446,7 @@ static const TypeInfo virtio_balloon_info = {
> .name = TYPE_VIRTIO_BALLOON,
> .parent = TYPE_VIRTIO_DEVICE,
> .instance_size = sizeof(VirtIOBalloon),
> + .instance_init = virtio_balloon_instance_init,
> .class_init = virtio_balloon_class_init,
> };
>
>
- [Qemu-devel] [PATCH 0/3] virtio-balloon: use alias property, shannon . zhao, 2015/05/11
- [Qemu-devel] [PATCH 1/3] hw/virtio/virtio-balloon: move adding property to virtio_balloon_instance_init, shannon . zhao, 2015/05/11
- Re: [Qemu-devel] [PATCH 1/3] hw/virtio/virtio-balloon: move adding property to virtio_balloon_instance_init,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 2/3] hw/virtio/virtio-pci: use alias property for virtio-balloon-pci, shannon . zhao, 2015/05/11
- [Qemu-devel] [PATCH 3/3] hw/s390x/virtio-ccw: use alias property for virtio-balloon-ccw, shannon . zhao, 2015/05/11
- Re: [Qemu-devel] [PATCH 0/3] virtio-balloon: use alias property, Paolo Bonzini, 2015/05/11
- Re: [Qemu-devel] [PATCH 0/3] virtio-balloon: use alias property, Cornelia Huck, 2015/05/11
- Re: [Qemu-devel] [PATCH 0/3] virtio-balloon: use alias property, Shannon Zhao, 2015/05/19