Re: [Qemu-devel] [PATCH 0/4] qemu-img: conditionally discard target on convert

[Peter Lieven]

Am 17.07.2013 um 16:26 schrieb Paolo Bonzini <address@hidden>:

> Il 17/07/2013 16:18, Peter Lieven ha scritto:
>>>>>> That would be ok if the patches are merged first. Otherwise I could ask 
>>>>>> Kevin
>>>>>> to merge my old series (except the iscsi_co_write_zeroes patch as there
>>>>>> obviously is still room for discussion and improvement) and you could 
>>>>>> tweak
>>>>>> iscsi_co_is_allocated later?
>>>> 
>>>> I'll look at your old series, I think the conflicts are relatively
>>>> trivial.  But I think that this series must wait for 1.7.
>> Would it be an Option to merge it except for the iscsi_co_is_allocated and 
>> the iscsi_co_write_zeroes patch. 3 of the Patches fix potential bugs.
> 
> Yes, of course!  And I think I can merge them via scsi/next, too.  What
> patches are you thinking of exactly?  Can you write the numbers?

Mandatory:
[PATCHv3 07/10] iscsi: fix -ENOSPC in iscsi_create()
[PATCHv3 08/10] iscsi: factor out sector conversions
[PATCHv3 09/10] iscsi: remove support for misaligned nb_sectors in aio_readv
[PATCHv3 10/10] iscsi: assert that sectors are aligned to LUN blocksize

Optional (non-conflicting patches):
[PATCHv3 01/10] iscsi: add logical block provisioning information to iscsilun
[PATCHv3 05/10] block: add bdrv_write_zeroes()
[PATCHv3 06/10] block/raw: add bdrv_co_write_zeroes

If you want you can also already merge this and just rename it to 
iscsi_co_get_block_status.
This would enable your qemu-img map command to also work for iscsi.
[PATCHv3 02/10] iscsi: add .bdrv_co_is_allocated

Thank you,
Peter