[Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[jean-christophe manciot]

Public bug reported:

Environment: Ubuntu 13.04

"hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
cc1: all warnings being treated as errors
make: *** [hw/ide/macio.o] Error 1"

I got the source files with a "git clone git://git.qemu-
project.org/qemu.git" + a recent "git fetch"

** Affects: qemu
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1205156

Title:
  Errors while compiling version 1.5.2

Status in QEMU:
  New

Bug description:
  Environment: Ubuntu 13.04

  "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
  hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
  hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
  hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  cc1: all warnings being treated as errors
  make: *** [hw/ide/macio.o] Error 1"

  I got the source files with a "git clone git://git.qemu-
  project.org/qemu.git" + a recent "git fetch"

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1205156/+subscriptions

Re: [Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Stefan Weil]

Am 26.07.2013 04:03, schrieb jean-christophe manciot:
> Public bug reported:
>
> Environment: Ubuntu 13.04
>
> "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
> hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
> hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
> hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> cc1: all warnings being treated as errors
> make: *** [hw/ide/macio.o] Error 1"
>
> I got the source files with a "git clone git://git.qemu-
> project.org/qemu.git" + a recent "git fetch"
>
> ** Affects: qemu
>      Importance: Undecided
>          Status: New
>



This patch should fix it: http://patchwork.ozlabs.org/patch/258774/.

It's also still missing in git master, but was already applied to
qemu-trivial.

Stefan

[Qemu-devel] [Bug 1205156] Re: Errors while compiling version 1.5.2

[jean-christophe manciot]

I was able to compile and link latest 1.5.2 release with the tar.bz2
source files.

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1205156

Title:
  Errors while compiling version 1.5.2

Status in QEMU:
  New

Bug description:
  Environment: Ubuntu 13.04

  "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
  hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
  hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
  hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  cc1: all warnings being treated as errors
  make: *** [hw/ide/macio.o] Error 1"

  I got the source files with a "git clone git://git.qemu-
  project.org/qemu.git" + a recent "git fetch"

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1205156/+subscriptions

[Qemu-devel] [Bug 1205156] Re: Errors while compiling version 1.5.2

[jean-christophe manciot]

I'm glad that ASAs do not crash anymore within GNS3 with this latest
qemu release: nice work guys! :)

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1205156

Title:
  Errors while compiling version 1.5.2

Status in QEMU:
  New

Bug description:
  Environment: Ubuntu 13.04

  "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
  hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
  hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
  hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  cc1: all warnings being treated as errors
  make: *** [hw/ide/macio.o] Error 1"

  I got the source files with a "git clone git://git.qemu-
  project.org/qemu.git" + a recent "git fetch"

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1205156/+subscriptions

Re: [Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Michael Roth]

Quoting Stefan Weil (2013-07-26 00:12:59)
> Am 26.07.2013 04:03, schrieb jean-christophe manciot:
> > Public bug reported:
> >
> > Environment: Ubuntu 13.04
> >
> > "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
> > hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
> > hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
> > hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > cc1: all warnings being treated as errors
> > make: *** [hw/ide/macio.o] Error 1"
> >
> > I got the source files with a "git clone git://git.qemu-
> > project.org/qemu.git" + a recent "git fetch"
> >
> > ** Affects: qemu
> >      Importance: Undecided
> >          Status: New
> >
> 
> 
> 
> This patch should fix it: http://patchwork.ozlabs.org/patch/258774/.
> 
> It's also still missing in git master, but was already applied to
> qemu-trivial.

This doesn't seem to be vanilla 1.5.2, where 04dd1259 isn't applicable
(no MACIO_DPRINTF statements), but rather a newer release or past
version with this patch on top:

  commit 80fc95d8bdaf3392106b131a97ca701fd374489a
  Author: Alexander Graf <agraf@suse.de>
  Date:   Fri Jun 28 13:30:01 2013 +0200
  
      PPC: dbdma: Support unaligned DMA access

I'd pull them both in if Alex wants to send a backported version for
1.5.2, but otherwise this doesn't seem to be an issue with stable.

> 
> Stefan

Re: [Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Michael Roth]

Quoting Michael Roth (2013-08-12 20:05:32)
> Quoting Stefan Weil (2013-07-26 00:12:59)
> > Am 26.07.2013 04:03, schrieb jean-christophe manciot:
> > > Public bug reported:
> > >
> > > Environment: Ubuntu 13.04
> > >
> > > "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
> > > hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > > hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
> > > hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
> > > hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > > hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> > > cc1: all warnings being treated as errors
> > > make: *** [hw/ide/macio.o] Error 1"
> > >
> > > I got the source files with a "git clone git://git.qemu-
> > > project.org/qemu.git" + a recent "git fetch"
> > >
> > > ** Affects: qemu
> > >      Importance: Undecided
> > >          Status: New
> > >
> > 
> > 
> > 
> > This patch should fix it: http://patchwork.ozlabs.org/patch/258774/.
> > 
> > It's also still missing in git master, but was already applied to
> > qemu-trivial.
> 
> This doesn't seem to be vanilla 1.5.2, where 04dd1259 isn't applicable
> (no MACIO_DPRINTF statements), but rather a newer release or past
> version with this patch on top:
> 
>   commit 80fc95d8bdaf3392106b131a97ca701fd374489a
>   Author: Alexander Graf <agraf@suse.de>
>   Date:   Fri Jun 28 13:30:01 2013 +0200
>   
>       PPC: dbdma: Support unaligned DMA access
> 
> I'd pull them both in if Alex wants to send a backported version for
> 1.5.2, but otherwise this doesn't seem to be an issue with stable.

Forgot to cc Alex.

> 
> > 
> > Stefan

Re: [Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Alexander Graf]

Am 13.08.2013 um 03:07 schrieb Michael Roth <mdroth@linux.vnet.ibm.com>:

> Quoting Michael Roth (2013-08-12 20:05:32)
>> Quoting Stefan Weil (2013-07-26 00:12:59)
>>> Am 26.07.2013 04:03, schrieb jean-christophe manciot:
>>>> Public bug reported:
>>>> 
>>>> Environment: Ubuntu 13.04
>>>> 
>>>> "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
>>>> hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
>>>> hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
>>>> hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
>>>> hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
>>>> hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
>>>> cc1: all warnings being treated as errors
>>>> make: *** [hw/ide/macio.o] Error 1"
>>>> 
>>>> I got the source files with a "git clone git://git.qemu-
>>>> project.org/qemu.git" + a recent "git fetch"
>>>> 
>>>> ** Affects: qemu
>>>>     Importance: Undecided
>>>>         Status: New
>>> 
>>> 
>>> 
>>> This patch should fix it: http://patchwork.ozlabs.org/patch/258774/.
>>> 
>>> It's also still missing in git master, but was already applied to
>>> qemu-trivial.
>> 
>> This doesn't seem to be vanilla 1.5.2, where 04dd1259 isn't applicable
>> (no MACIO_DPRINTF statements), but rather a newer release or past
>> version with this patch on top:
>> 
>>  commit 80fc95d8bdaf3392106b131a97ca701fd374489a
>>  Author: Alexander Graf <agraf@suse.de>
>>  Date:   Fri Jun 28 13:30:01 2013 +0200
>> 
>>      PPC: dbdma: Support unaligned DMA access
>> 
>> I'd pull them both in if Alex wants to send a backported version for
>> 1.5.2, but otherwise this doesn't seem to be an issue with stable.
> 
> Forgot to cc Alex.

I'd rather not backport these to 1.5, as the patches only make sense as a whole with an OpenBIOS update.

IIUC the compile error has been fixed for 1.6, correct?


Alex

> 
>> 
>>> 
>>> Stefan

Re: [Qemu-devel] [Bug 1205156] [NEW] Errors while compiling version 1.5.2

[Michael Roth]

Quoting Alexander Graf (2013-08-12 23:06:19)
> Am 13.08.2013 um 03:07 schrieb Michael Roth <mdroth@linux.vnet.ibm.com>:
> 
> > Quoting Michael Roth (2013-08-12 20:05:32)
> >> Quoting Stefan Weil (2013-07-26 00:12:59)
> >>> Am 26.07.2013 04:03, schrieb jean-christophe manciot:
> >>>> Public bug reported:
> >>>> 
> >>>> Environment: Ubuntu 13.04
> >>>> 
> >>>> "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
> >>>> hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> >>>> hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
> >>>> hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
> >>>> hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> >>>> hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
> >>>> cc1: all warnings being treated as errors
> >>>> make: *** [hw/ide/macio.o] Error 1"
> >>>> 
> >>>> I got the source files with a "git clone git://git.qemu-
> >>>> project.org/qemu.git" + a recent "git fetch"
> >>>> 
> >>>> ** Affects: qemu
> >>>>     Importance: Undecided
> >>>>         Status: New
> >>> 
> >>> 
> >>> 
> >>> This patch should fix it: http://patchwork.ozlabs.org/patch/258774/.
> >>> 
> >>> It's also still missing in git master, but was already applied to
> >>> qemu-trivial.
> >> 
> >> This doesn't seem to be vanilla 1.5.2, where 04dd1259 isn't applicable
> >> (no MACIO_DPRINTF statements), but rather a newer release or past
> >> version with this patch on top:
> >> 
> >>  commit 80fc95d8bdaf3392106b131a97ca701fd374489a
> >>  Author: Alexander Graf <agraf@suse.de>
> >>  Date:   Fri Jun 28 13:30:01 2013 +0200
> >> 
> >>      PPC: dbdma: Support unaligned DMA access
> >> 
> >> I'd pull them both in if Alex wants to send a backported version for
> >> 1.5.2, but otherwise this doesn't seem to be an issue with stable.
> > 
> > Forgot to cc Alex.
> 
> I'd rather not backport these to 1.5, as the patches only make sense as a whole with an OpenBIOS update.

Ok, makes sense, just thought I'd double-check. Got the impression from the bug
report that maybe some downstreams were carrying your patch on top of 1.5.2 for
OSX support, but I'm not really sure what's going on here.

> 
> IIUC the compile error has been fixed for 1.6, correct?

Yup, should be fixed upstream with 04dd1259

> 
> 
> Alex
> 
> > 
> >> 
> >>> 
> >>> Stefan

[Qemu-devel] [Bug 1205156] Re: Errors while compiling version 1.5.2

[T. Huth]

Marking this ticket as "Fix released" according to comment #6.

** Changed in: qemu
       Status: New => Fix Released

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1205156

Title:
  Errors while compiling version 1.5.2

Status in QEMU:
  Fix Released

Bug description:
  Environment: Ubuntu 13.04

  "hw/ide/macio.c: In function ‘pmac_ide_atapi_transfer_cb’:
  hw/ide/macio.c:134:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c: In function ‘pmac_ide_transfer_cb’:
  hw/ide/macio.c:215:5: error: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int64_t’ [-Werror=format]
  hw/ide/macio.c:222:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  hw/ide/macio.c:264:9: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘hwaddr’ [-Werror=format]
  cc1: all warnings being treated as errors
  make: *** [hw/ide/macio.o] Error 1"

  I got the source files with a "git clone git://git.qemu-
  project.org/qemu.git" + a recent "git fetch"

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1205156/+subscriptions