diff options
author | Jeff Cody <jcody@redhat.com> | 2013-01-18 12:45:35 -0500 |
---|---|---|
committer | Luiz Capitulino <lcapitulino@redhat.com> | 2013-01-25 10:36:26 -0200 |
commit | 58513bde833804bc9395d79fd81aae631b97c348 (patch) | |
tree | 82b4326f09101e164a967ad04e4320ef86c4b32c /qmp-commands.hx | |
parent | 11c29918be32be5b00f367c7da9724a5cddbbb0f (diff) |
block: Monitor command commit neglects to report some errors
The non-live bdrv_commit() function may return one of the following
errors: -ENOTSUP, -EBUSY, -EACCES, -EIO. The only error that is
checked in the HMP handler is -EBUSY, so the monitor command 'commit'
silently fails for all error cases other than 'Device is in use'.
Report error using monitor_printf() and strerror(), and convert existing
qerror_report() calls in do_commit() to monitor_printf().
Signed-off-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'qmp-commands.hx')
0 files changed, 0 insertions, 0 deletions