[Qemu-devel] [PATCH] spice: fix 32bit build

[Qemu-devel] [PATCH] spice: fix 32bit build

[Gerd Hoffmann]

New 32bit warnings sneaked in, this time in
ui/spice-display.c, fix them.

This gets annonying, /me sets up a ubuntu buildbot
slave for 32bit spice testbuilds.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 roms/seabios       |    2 +-
 ui/spice-display.c |   12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/roms/seabios b/roms/seabios
index 2e8bd61..80d11e8 160000
--- a/roms/seabios
+++ b/roms/seabios
@@ -1 +1 @@
-Subproject commit 2e8bd611ce4e1e36b5a80c9ca6e256e23802f095
+Subproject commit 80d11e8577bf03e98f2eb1b0cb3a281ab2879c9e
diff --git a/ui/spice-display.c b/ui/spice-display.c
index 28d6d4a..6d7563f 100644
--- a/ui/spice-display.c
+++ b/ui/spice-display.c
@@ -80,8 +80,8 @@ void qemu_spice_add_memslot(SimpleSpiceDisplay *ssd, QXLDevMemSlot *memslot,
 
     if (async != QXL_SYNC) {
         spice_qxl_add_memslot_async(&ssd->qxl, memslot,
-                (uint64_t)qxl_cookie_new(QXL_COOKIE_TYPE_IO,
-                                         QXL_IO_MEMSLOT_ADD_ASYNC));
+                (uintptr_t)qxl_cookie_new(QXL_COOKIE_TYPE_IO,
+                                          QXL_IO_MEMSLOT_ADD_ASYNC));
     } else {
         ssd->worker->add_memslot(ssd->worker, memslot);
     }
@@ -100,8 +100,8 @@ void qemu_spice_create_primary_surface(SimpleSpiceDisplay *ssd, uint32_t id,
     trace_qemu_spice_create_primary_surface(ssd->qxl.id, id, surface, async);
     if (async != QXL_SYNC) {
         spice_qxl_create_primary_surface_async(&ssd->qxl, id, surface,
-                (uint64_t)qxl_cookie_new(QXL_COOKIE_TYPE_IO,
-                                         QXL_IO_CREATE_PRIMARY_ASYNC));
+                (uintptr_t)qxl_cookie_new(QXL_COOKIE_TYPE_IO,
+                                          QXL_IO_CREATE_PRIMARY_ASYNC));
     } else {
         ssd->worker->create_primary_surface(ssd->worker, id, surface);
     }
@@ -113,8 +113,8 @@ void qemu_spice_destroy_primary_surface(SimpleSpiceDisplay *ssd,
     trace_qemu_spice_destroy_primary_surface(ssd->qxl.id, id, async);
     if (async != QXL_SYNC) {
         spice_qxl_destroy_primary_surface_async(&ssd->qxl, id,
-                (uint64_t)qxl_cookie_new(QXL_COOKIE_TYPE_IO,
-                                         QXL_IO_DESTROY_PRIMARY_ASYNC));
+                (uintptr_t)qxl_cookie_new(QXL_COOKIE_TYPE_IO,
+                                          QXL_IO_DESTROY_PRIMARY_ASYNC));
     } else {
         ssd->worker->destroy_primary_surface(ssd->worker, id);
     }
-- 
1.7.1

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Peter Maydell]

On 20 March 2012 11:26, Gerd Hoffmann <kraxel@redhat.com> wrote:
> New 32bit warnings sneaked in, this time in
> ui/spice-display.c, fix them.
>
> This gets annonying, /me sets up a ubuntu buildbot
> slave for 32bit spice testbuilds.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  roms/seabios       |    2 +-
>  ui/spice-display.c |   12 ++++++------

What's the roms/seabios change doing here?

Also, isn't the ui/spice-display.c patch the same as this one from
me from a couple of weeks back?
http://patchwork.ozlabs.org/patch/145269/

-- PMM

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Gerd Hoffmann]

  Hi,

> What's the roms/seabios change doing here?

Just a missing "git submodule update" and the diff from that sneaking in.

> Also, isn't the ui/spice-display.c patch the same as this one from
> me from a couple of weeks back?
> http://patchwork.ozlabs.org/patch/145269/

It is.  Picked that one up instead.

/me is confused, I through I had that one already and we just got new
warnings, sorry.

cheers,
  Gerd

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Peter Maydell]

On 20 March 2012 12:13, Gerd Hoffmann <kraxel@redhat.com> wrote:
>> Also, isn't the ui/spice-display.c patch the same as this one from
>> me from a couple of weeks back?
>> http://patchwork.ozlabs.org/patch/145269/
>
> It is.  Picked that one up instead.
>
> /me is confused, I through I had that one already and we just got new
> warnings, sorry.

No problem. (I was under the impression that had been committed
already as well for some reason...)

-- PMM

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Michael Tokarev]

20.03.2012 15:26, Gerd Hoffmann wrote:
> New 32bit warnings sneaked in, this time in
> ui/spice-display.c, fix them.
>
> This gets annonying, /me sets up a ubuntu buildbot
> slave for 32bit spice testbuilds.

Um, is it worth to watch/fix?  Note that spice does
not work on 32bits anyway, qemu segfaults at startup...
http://bugs.debian.org/640139 (the bug was still valid
when 1.0 was released).

Thanks,

/mjt

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Alon Levy]

On Tue, Mar 20, 2012 at 04:45:26PM +0400, Michael Tokarev wrote:
> 20.03.2012 15:26, Gerd Hoffmann wrote:
> >New 32bit warnings sneaked in, this time in
> >ui/spice-display.c, fix them.
> >
> >This gets annonying, /me sets up a ubuntu buildbot
> >slave for 32bit spice testbuilds.
> 
> Um, is it worth to watch/fix?  Note that spice does
> not work on 32bits anyway, qemu segfaults at startup...
> http://bugs.debian.org/640139 (the bug was still valid
> when 1.0 was released).

Hmm, my bad - I only tested with Xspice, that doesn't go through the
slots. So at least it means most of it works fine for 32 bit. I'll try
to fix this part. Thanks for the link.

> 
> Thanks,
> 
> /mjt
> 

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Peter Maydell]

On 20 March 2012 12:45, Michael Tokarev <mjt@tls.msk.ru> wrote:
> Um, is it worth to watch/fix?  Note that spice does
> not work on 32bits anyway, qemu segfaults at startup...

I've had conflicting answers about this -- the Spice FAQ says
it doesn't work on 32 bits but in this message:
http://lists.gnu.org/archive/html/qemu-devel/2012-02/msg00944.html
Gerd said it should work on 32 bit systems now.

(If it is now OK on 32 bits it would be nice to get the
FAQ fixed...)

-- PMM

[Qemu-devel] [PATCH] spice: fix 32bit build

[Gerd Hoffmann]

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 hw/display/qxl.c   | 2 +-
 ui/spice-display.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/display/qxl.c b/hw/display/qxl.c
index 736fd3c..d43aa49 100644
--- a/hw/display/qxl.c
+++ b/hw/display/qxl.c
@@ -710,7 +710,7 @@ static void interface_release_resource(QXLInstance *sin,
 
     if (ext.group_id == MEMSLOT_GROUP_HOST) {
         /* host group -> vga mode update request */
-        QXLCommandExt *cmdext = (void *)(ext.info->id);
+        QXLCommandExt *cmdext = (void *)(intptr_t)(ext.info->id);
         SimpleSpiceUpdate *update;
         g_assert(cmdext->cmd.type == QXL_CMD_DRAW);
         update = container_of(cmdext, SimpleSpiceUpdate, ext);
diff --git a/ui/spice-display.c b/ui/spice-display.c
index 03040b1..66e2578 100644
--- a/ui/spice-display.c
+++ b/ui/spice-display.c
@@ -534,7 +534,7 @@ static void interface_release_resource(QXLInstance *sin,
     QXLCommandExt *ext;
 
     dprint(2, "%s/%d:\n", __func__, ssd->qxl.id);
-    ext = (void *)(rext.info->id);
+    ext = (void *)(intptr_t)(rext.info->id);
     switch (ext->cmd.type) {
     case QXL_CMD_DRAW:
         update = container_of(ext, SimpleSpiceUpdate, ext);
-- 
1.8.3.1

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Luiz Capitulino]

On Fri, 20 Jun 2014 08:12:44 +0200
Gerd Hoffmann <kraxel@redhat.com> wrote:

> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

Tested-by: Luiz Capitulino <lcapitulino@redhat.com>

> ---
>  hw/display/qxl.c   | 2 +-
>  ui/spice-display.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/hw/display/qxl.c b/hw/display/qxl.c
> index 736fd3c..d43aa49 100644
> --- a/hw/display/qxl.c
> +++ b/hw/display/qxl.c
> @@ -710,7 +710,7 @@ static void interface_release_resource(QXLInstance *sin,
>  
>      if (ext.group_id == MEMSLOT_GROUP_HOST) {
>          /* host group -> vga mode update request */
> -        QXLCommandExt *cmdext = (void *)(ext.info->id);
> +        QXLCommandExt *cmdext = (void *)(intptr_t)(ext.info->id);
>          SimpleSpiceUpdate *update;
>          g_assert(cmdext->cmd.type == QXL_CMD_DRAW);
>          update = container_of(cmdext, SimpleSpiceUpdate, ext);
> diff --git a/ui/spice-display.c b/ui/spice-display.c
> index 03040b1..66e2578 100644
> --- a/ui/spice-display.c
> +++ b/ui/spice-display.c
> @@ -534,7 +534,7 @@ static void interface_release_resource(QXLInstance *sin,
>      QXLCommandExt *ext;
>  
>      dprint(2, "%s/%d:\n", __func__, ssd->qxl.id);
> -    ext = (void *)(rext.info->id);
> +    ext = (void *)(intptr_t)(rext.info->id);
>      switch (ext->cmd.type) {
>      case QXL_CMD_DRAW:
>          update = container_of(ext, SimpleSpiceUpdate, ext);

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Peter Maydell]

On 20 June 2014 14:35, Luiz Capitulino <lcapitulino@redhat.com> wrote:
> On Fri, 20 Jun 2014 08:12:44 +0200
> Gerd Hoffmann <kraxel@redhat.com> wrote:
>
>> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
>
> Tested-by: Luiz Capitulino <lcapitulino@redhat.com>

Thanks; applied directly to master as a buildfix.

-- PMM

Re: [Qemu-devel] [PATCH] spice: fix 32bit build

[Luiz Capitulino]

On Fri, 20 Jun 2014 16:57:11 +0100
Peter Maydell <peter.maydell@linaro.org> wrote:

> On 20 June 2014 14:35, Luiz Capitulino <lcapitulino@redhat.com> wrote:
> > On Fri, 20 Jun 2014 08:12:44 +0200
> > Gerd Hoffmann <kraxel@redhat.com> wrote:
> >
> >> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> >
> > Tested-by: Luiz Capitulino <lcapitulino@redhat.com>
> 
> Thanks; applied directly to master as a buildfix.

Cool. I'll be even happier with you do this for my softfloat fixes too.