Re: [Qemu-devel] [PATCH 4/7] block-vpc: Fix support for images > 4 GB

[Kevin Wolf <kwolf@suse.de>]

Kevin Wolf schrieb:
> This patch fixes the truncation of sector offsets to 32 bits.
> 
> Signed-off-by: Kevin Wolf <kwolf@suse.de>
> ---
>  block-vpc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/block-vpc.c b/block-vpc.c
> index 3c92bb3..51644e6 100644
> --- a/block-vpc.c
> +++ b/block-vpc.c
> @@ -207,7 +207,7 @@ static inline int seek_to_sector(BlockDriverState *bs, int64_t sector_num)
>      if (pagetable_index > s->max_table_entries || s->pagetable[pagetable_index] == 0xffffffff)
>  	return -1; // not allocated
>  
> -    bitmap_offset = 512 * s->pagetable[pagetable_index];
> +    bitmap_offset = 512 * (uint64_t) s->pagetable[pagetable_index];
>      block_offset = bitmap_offset + 512 + (512 * pageentry_index);
>  
>  //    printf("sector: %" PRIx64 ", index: %x, offset: %x, bioff: %" PRIx64 ", bloff: %" PRIx64 "\n",

Anthony, I think you missed this fix. I guess that's because the mail
was delivered to the list only some hours after I sent the series.
Anyway, it's still needed.

Kevin