[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] balloon: Fix device name in error message.
From: |
Hani Benhabiles |
Subject: |
Re: [Qemu-devel] [PATCH] balloon: Fix device name in error message. |
Date: |
Mon, 24 Feb 2014 19:59:14 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Feb 24, 2014 at 10:04:05AM +0100, Markus Armbruster wrote:
> I agree the error message is bad, but I'm afraid your patch makes it
> worse :)
>
> qemu_balloon() returns zero when balloon_event_fn has not been set with
> qemu_add_balloon_handler().
>
> Right now, the only device that calls qemu_add_balloon_handler() is
> virtio-balloon-device. In theory, there could be others in the future.
>
> virtio-balloon-device is a virtio-bus device. The bus can be provided
> by virtio-balloon-pci or virtio-balloon-ccw.
>
> Your error message is misleading when it's provided by the latter.
>
I see. Sorry for not throughly understanding the whole chain yet!
I somehow missed virtio-balloon-ccw, thus my false belief that
virtio-balloon-pci would be the only one to be ever needed/used.
> Moreover, you missed qmp_query_balloon().
>
> QERR_DEVICE_NOT_ACTIVE is also used with argument "spice", and the
> resulting error message is similarly bad. Perhaps something like this
> would do:
>
> #define QERR_DEVICE_NOT_ACTIVE \
> ERROR_CLASS_DEVICE_NOT_ACTIVE, "No %s device has been activated"
Sounds good to me. Will send a patch shortly.