[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 0/3] qerror: proper errors for qmp_block_resize()
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PATCH 0/3] qerror: proper errors for qmp_block_resize() |
Date: |
Wed, 4 Jan 2012 17:38:20 +0000 |
The block resize command returns undefined errors if things go wrong. This is
bad since users will have no chance to understand what failed.
Patch 3 makes qmp_block_resize() use meaningful errors. We introduce new
qerrors for ENOMEDIUM and EACCES since nothing exists yet.
When doing this I noticed that qerror definitions and table entries are not in
alphabetical order as required by the comments in qerror.h and qerror.c.
That's not a surprise since there is no automated way to enforce this. I'm not
sure whether this requirement is useful or not in the first place, but felt
guilty creating more mess. So I've restored alphabetical order and added a
script to verify that the requirement is met in Patches 1 and 2.
Stefan Hajnoczi (3):
qerror: add check-qerror.sh to verify alphabetical order
qerror: restore alphabetical order over qerrors
block: use proper qerrors in qmp_block_resize
blockdev.c | 26 +++++++++----
qerror.c | 91 +++++++++++++++++++++++++---------------------
qerror.h | 78 +++++++++++++++++++++------------------
scripts/check-qerror.sh | 22 +++++++++++
4 files changed, 131 insertions(+), 86 deletions(-)
create mode 100755 scripts/check-qerror.sh
--
1.7.7.3
- [Qemu-devel] [PATCH 0/3] qerror: proper errors for qmp_block_resize(),
Stefan Hajnoczi <=