[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 7/8] iscsi: assert that sectors are aligned to L
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 7/8] iscsi: assert that sectors are aligned to LUN blocksize |
Date: |
Mon, 24 Jun 2013 18:13:05 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
Il 24/06/2013 18:10, Peter Lieven ha scritto:
> Am 24.06.2013 16:30, schrieb Paolo Bonzini:
>> Il 22/06/2013 22:58, Peter Lieven ha scritto:
>>> if the blocksize of an iSCSI LUN is bigger than the BDRV_SECTOR_SIZE
>>> it is possible that sector_num or nb_sectors are not correctly
>>> alligned.
>>>
>>> for now assert that there is no misalignment to avoid data
>>> corruption.
>> You should just fail to open the LUN instead.
> Ronnie added support for reading fragments of blocksize if the
> offset is aligned. I don't know what the idea was, maybe for qemu
> reading the boot sector. If the OS does only read multiple of
> blocksize at aligned offsets everything should work. I now that
> our storages support 4K blocksize. I can check if it is usable.
> I could also just fail the operations instead of asserting.
So far, 4K blocksize is usable if you also specify the same block size
for the guest device. I have posted once the patches to do
read-modify-write, but I never really pursued inclusion of those.
In any case, the right place to fix this is the block layer;
driver-specific hacks are... hacks. :)
Paolo
- [Qemu-devel] [PATCH 6/8] qemu-img: use bdrv_write_zeroes to write zeroes, (continued)
[Qemu-devel] [PATCH 5/8] block/raw: add bdrv_co_write_zeroes, Peter Lieven, 2013/06/22
[Qemu-devel] [PATCH 4/8] block: add bdrv_write_zeroes(), Peter Lieven, 2013/06/22
[Qemu-devel] [PATCH 7/8] iscsi: assert that sectors are aligned to LUN blocksize, Peter Lieven, 2013/06/22
[Qemu-devel] [PATCH 8/8] block-migration: efficiently encode zero blocks, Peter Lieven, 2013/06/22