On 04/05/2017 06:37 PM, Eric Blake wrote: > On 04/05/2017 10:18 AM, Denis V. Lunev wrote: >> As long as BDRV_O_INACTIVE is set, the image file is only opened so we >> have a file descriptor for it. We're definitely not supposed to modify >> the image, it's still owned by the migration source. >> >> This commit is an addition to 09e0c771 but the assert() is added to >> bdrv_truncate(). >> >> Signed-off-by: Denis V. Lunev <[email protected]> >> CC: Kevin Wolf <[email protected]> >> CC: Max Reitz <[email protected]> >> --- >> block.c | 2 ++ >> 1 file changed, 2 insertions(+) > I don't feel comfortable adding an assert this late in the 2.9 phase, > but think it makes perfect sense as a 2.10 addition. > > Reviewed-by: Eric Blake <[email protected]> no-no, this is for 2.10 for sure. No way for 2.9. I have stated that in cover letter :( Sorry for inconvenience.
Den
