Skip to content
  • Eric Blake's avatar
    f0a9c18f
    qemu-io: Relax 'alloc' now that block-status doesn't assert · f0a9c18f
    Eric Blake authored
    
    
    Previously, the alloc command required that input parameters be
    sector-aligned and clamped to 32 bits, because the underlying
    bdrv_is_allocated used a 32-bit parameter and asserted aligned
    inputs.  But now that we have fixed block status to report a
    64-bit bytes value, and to properly round requests on behalf of
    guests, we can pass any values, and can use qemu-io to add
    coverage that our rounding is correct regardless of the guest
    alignment constraints.
    
    Update iotest 177 to intentionally probe block status at
    unaligned boundaries as well as with a bytes value that does not
    map to 32-bit sectors, which also required tweaking the image
    prep to leave an unallocated portion to the image under test.
    
    Signed-off-by: default avatarEric Blake <eblake@redhat.com>
    Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
    f0a9c18f
    qemu-io: Relax 'alloc' now that block-status doesn't assert
    Eric Blake authored
    
    
    Previously, the alloc command required that input parameters be
    sector-aligned and clamped to 32 bits, because the underlying
    bdrv_is_allocated used a 32-bit parameter and asserted aligned
    inputs.  But now that we have fixed block status to report a
    64-bit bytes value, and to properly round requests on behalf of
    guests, we can pass any values, and can use qemu-io to add
    coverage that our rounding is correct regardless of the guest
    alignment constraints.
    
    Update iotest 177 to intentionally probe block status at
    unaligned boundaries as well as with a bytes value that does not
    map to 32-bit sectors, which also required tweaking the image
    prep to leave an unallocated portion to the image under test.
    
    Signed-off-by: default avatarEric Blake <eblake@redhat.com>
    Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
Loading