Re: [Qemu-devel] [PATCH 0/2] Fix werror=enospc for qcow2 on iscsi

[Kevin Wolf <kwolf@redhat.com>]

Am 22.10.2015 um 10:45 hat Paolo Bonzini geschrieben:
> On 22/10/2015 10:17, Fam Zheng wrote:
> > When qcow2 is created on iscsi target with a virtual size greater than physical
> > capacity of the LUN, over time it's possible that guest fills too much data and
> > at that point, new clusters in qcow2 will be allocated beyond the end of disk.
> > 
> > werror=enospc is useful for that purpose to allocate more data for the guest,
> > except in this case, unlike a host file system, iscsi returns -EIO instead of
> > -ENOSPC, which makes it hard to detect and report proper error.
> > 
> > Fix this by improving iscsi error handling code to return meaningful error
> > codes (-ERANGE here), then further translate it to -ENOSPC in qcow2.
> 
> FWIW, Linux uses ENOSPC if it detects out of range LBAs:
> 
>         if (iocb->ki_pos >= size)
>                 return -ENOSPC;
> 
> so I think it's okay to convert LBA_OUT_OF_RANGE to ENOSPC directly and
> avoid patch 2.

Yes, definitely. Fixing this up in qcow2 would be wrong, all other image
formats are affected as well. The iscsi block driver already needs to
return the correct errno value, which is ENOSPC.

Kevin