qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Kevin Wolf
Subject: Re: [Qemu-devel] [PATCH 0/2] Fix werror=enospc for qcow2 on iscsi
Date: Thu, 22 Oct 2015 11:03:30 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]