On Thu, 13 Jul 2017 13:02:32 +0200
Ladi Prosek <lpro...@redhat.com> wrote:

> Most virtio error output pertains to a specific virtqueue so it makes
> sense to include the queue index in error messages. This commit makes
> all error output in virtio.c use the newly introduced virtqueue_error.

Split out the change introducing virtqueue_error(), as this patch is
quite large?

> 
> Suggested-by: Stefan Hajnoczi <stefa...@redhat.com>
> Signed-off-by: Ladi Prosek <lpro...@redhat.com>
> ---
>  hw/virtio/virtio.c         | 98 
> ++++++++++++++++++++++++++++------------------
>  include/hw/virtio/virtio.h |  1 +
>  2 files changed, 60 insertions(+), 39 deletions(-)

I like the change. Feel free to add

Reviewed-by: Cornelia Huck <coh...@redhat.com>

Reply via email to