Re: [Qemu-devel] [PATCH] hw/qxl: fir build break with newer spice

[Alon Levy]

On Tue, Aug 09, 2011 at 03:21:15PM +0300, Alon Levy wrote:
> ioport_write's val was changed from uint32_t to uint64_t, this
> broke two printfs. Use PRId64 instead of %d.

Nack, missed a 'd' on the second fix. And the subject has a typo.

> 
> Signed-off-by: Alon Levy <address@hidden>
> ---
>  hw/qxl.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/hw/qxl.c b/hw/qxl.c
> index db7ae7a..2335e4b 100644
> --- a/hw/qxl.c
> +++ b/hw/qxl.c
> @@ -1189,7 +1189,7 @@ async_common:
>          }
>          d->current_async = orig_io_port;
>          qemu_mutex_unlock(&d->async_lock);
> -        dprint(d, 2, "start async %d (%d)\n", io_port, val);
> +        dprint(d, 2, "start async %d (%"PRId64")\n", io_port, val);
>          break;
>      default:
>          break;
> @@ -1305,7 +1305,8 @@ async_common:
>          break;
>      }
>      case QXL_IO_FLUSH_SURFACES_ASYNC:
> -        dprint(d, 1, "QXL_IO_FLUSH_SURFACES_ASYNC (%d) (%s, s#=%d, 
> res#=%d)\n",
> +        dprint(d, 1, "QXL_IO_FLUSH_SURFACES_ASYNC"
> +                     " (%"PRId64"d) (%s, s#=%d, res#=%d)\n",
>                 val, qxl_mode_to_string(d->mode), d->guest_surfaces.count,
>                 d->num_free_res);
>          qxl_spice_flush_surfaces_async(d);
> -- 
> 1.7.6
> 
>