[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 06/37] balloon: Improve use of qapi visitor
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v9 06/37] balloon: Improve use of qapi visitor |
Date: |
Wed, 20 Jan 2016 15:09:42 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eric Blake <address@hidden> writes:
> Rework the control flow of balloon_stats_get_all() to make it
> easier for a later patch to split visit_end_struct(). Also
> switch to the uint64 visitor to match the data type.
>
> Signed-off-by: Eric Blake <address@hidden>
> Reviewed-by: Marc-André Lureau <address@hidden>
>
> ---
> v9: no change
> v8: no change
> v7: place earlier in series
> v6: new patch, split from RFC on v5 7/46
> ---
> hw/virtio/virtio-balloon.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c
> index 9671635..1ce987a 100644
> --- a/hw/virtio/virtio-balloon.c
> +++ b/hw/virtio/virtio-balloon.c
> @@ -130,10 +130,13 @@ static void balloon_stats_get_all(Object *obj, struct
> Visitor *v,
> if (err) {
> goto out_end;
> }
> - for (i = 0; !err && i < VIRTIO_BALLOON_S_NR; i++) {
> - visit_type_int64(v, (int64_t *) &s->stats[i], balloon_stat_names[i],
> - &err);
> + for (i = 0; i < VIRTIO_BALLOON_S_NR; i++) {
> + visit_type_uint64(v, &s->stats[i], balloon_stat_names[i], &err);
> + if (err) {
> + goto out_nested;
> + }
> }
> +out_nested:
> error_propagate(errp, err);
> err = NULL;
> visit_end_struct(v, &err);
What's wrong with plain break?
Hmm, PATCH 33 inserts something between the loop and the label. I
would've used break here regardless of the extra churn, just to avoid
review hiccups like this one. Too late now :)