Am 10.06.2013 um 19:02 hat Luiz Capitulino geschrieben: > I was surprised to find out that we still have old users of > QERR_OPEN_FILE_FAILED that print errors like: > > (qemu) dump-guest-memory -p /lkmads/foo > Could not open '/lkmads/foo' > (qemu) > > This series converts all those users to a new helper called > error_setg_file_open(), which adds error reason to open failures: > > (qemu) dump-guest-memory -p /sfmdkjf/foo > Could not open '/sfmdkjf/foo': No such file or directory > (qemu) > > v2 > > - bdrv_reopen_prepare(): use generic error message > - s/error/-ret for bdrv_open() errors
I suggested an error message change in patch 8, but I think that's minor enough that you don't have to resend the series after changing the text. So you can add: Acked-by: Kevin Wolf <[email protected]>
