[Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-06 Thread Markus Armbruster
Best to use the same type, to avoid unwanted truncation or sign extension. BlockConf can't use plain int for cyls, heads and secs, because integer properties require an exact width. Signed-off-by: Markus Armbruster --- hw/block-common.h |2 +- hw/hd-geometry.c |4 ++-- hw/ide/core.c

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-05 Thread Blue Swirl
On Wed, Jul 4, 2012 at 4:36 PM, Eric Blake wrote: > On 07/04/2012 10:19 AM, Paolo Bonzini wrote: >> Il 03/07/2012 22:15, Andreas Färber ha scritto: > > I'd also vote for %u. PRI*32 do not seem very useful compared to plain > int versions. >>> If it's not useful we should use unsigned i

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-05 Thread Blue Swirl
On Tue, Jul 3, 2012 at 8:15 PM, Andreas Färber wrote: > Am 03.07.2012 21:11, schrieb Blue Swirl: >> On Mon, Jul 2, 2012 at 2:34 PM, Stefan Hajnoczi wrote: >>> On Mon, Jul 2, 2012 at 3:15 PM, Markus Armbruster wrote: Andreas Färber writes: > Am 02.07.2012 14:55, schrieb Stefan Hajn

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-04 Thread Eric Blake
On 07/04/2012 10:19 AM, Paolo Bonzini wrote: > Il 03/07/2012 22:15, Andreas Färber ha scritto: I'd also vote for %u. PRI*32 do not seem very useful compared to plain int versions. >> If it's not useful we should use unsigned int. > > It just means that we're assuming 32-bit ints, wh

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-04 Thread Paolo Bonzini
Il 03/07/2012 22:15, Andreas Färber ha scritto: >> > >> > I'd also vote for %u. PRI*32 do not seem very useful compared to plain >> > int versions. > If it's not useful we should use unsigned int. It just means that we're assuming 32-bit ints, which is a fact. The day an IL32 platform appears th

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-03 Thread Andreas Färber
Am 03.07.2012 21:11, schrieb Blue Swirl: > On Mon, Jul 2, 2012 at 2:34 PM, Stefan Hajnoczi wrote: >> On Mon, Jul 2, 2012 at 3:15 PM, Markus Armbruster wrote: >>> Andreas Färber writes: >>> Am 02.07.2012 14:55, schrieb Stefan Hajnoczi: > On Fri, Jun 29, 2012 at 05:34:40PM +0200, Markus A

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-03 Thread Blue Swirl
On Mon, Jul 2, 2012 at 2:34 PM, Stefan Hajnoczi wrote: > On Mon, Jul 2, 2012 at 3:15 PM, Markus Armbruster wrote: >> Andreas Färber writes: >> >>> Am 02.07.2012 14:55, schrieb Stefan Hajnoczi: On Fri, Jun 29, 2012 at 05:34:40PM +0200, Markus Armbruster wrote: > Best to use the same type

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-02 Thread Stefan Hajnoczi
On Mon, Jul 2, 2012 at 3:15 PM, Markus Armbruster wrote: > Andreas Färber writes: > >> Am 02.07.2012 14:55, schrieb Stefan Hajnoczi: >>> On Fri, Jun 29, 2012 at 05:34:40PM +0200, Markus Armbruster wrote: Best to use the same type, to avoid unwanted truncation or sign extension. >>>

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-02 Thread Markus Armbruster
Andreas Färber writes: > Am 02.07.2012 14:55, schrieb Stefan Hajnoczi: >> On Fri, Jun 29, 2012 at 05:34:40PM +0200, Markus Armbruster wrote: >>> Best to use the same type, to avoid unwanted truncation or sign >>> extension. >>> >>> BlockConf can't use plain int for cyls, heads and secs, because >

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-02 Thread Andreas Färber
Am 02.07.2012 14:55, schrieb Stefan Hajnoczi: > On Fri, Jun 29, 2012 at 05:34:40PM +0200, Markus Armbruster wrote: >> Best to use the same type, to avoid unwanted truncation or sign >> extension. >> >> BlockConf can't use plain int for cyls, heads and secs, because >> integer properties require an

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-07-02 Thread Stefan Hajnoczi
On Fri, Jun 29, 2012 at 05:34:40PM +0200, Markus Armbruster wrote: > Best to use the same type, to avoid unwanted truncation or sign > extension. > > BlockConf can't use plain int for cyls, heads and secs, because > integer properties require an exact width. > > Signed-off-by: Markus Armbruster

Re: [Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-06-29 Thread Blue Swirl
On Fri, Jun 29, 2012 at 3:34 PM, Markus Armbruster wrote: > Best to use the same type, to avoid unwanted truncation or sign > extension. Awesome patch! :-) > > BlockConf can't use plain int for cyls, heads and secs, because > integer properties require an exact width. > > Signed-off-by: Markus A

[Qemu-devel] [PATCH 18/32] hd-geometry: Switch to uint32_t to match BlockConf

2012-06-29 Thread Markus Armbruster
Best to use the same type, to avoid unwanted truncation or sign extension. BlockConf can't use plain int for cyls, heads and secs, because integer properties require an exact width. Signed-off-by: Markus Armbruster --- blockdev.h |2 +- hw/hd-geometry.c |4 ++-- hw/ide/core.c|