[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 07/14] qerror: add additional parameter to QERR_DEVI
From: |
Anthony Liguori |
Subject: |
[Qemu-devel] [PATCH 07/14] qerror: add additional parameter to QERR_DEVICE_ENCRYPTED |
Date: |
Wed, 24 Aug 2011 13:43:02 -0500 |
Changing an encrypted block device requires a flow of:
1) change device
2) receive error from change
3) block_passwd
DeviceEncrypted receives the device name but with a block backend with multiple
backing files, any one of the backing files could be encrypted. From a UI
perspective, you have no way of knowing which file is encrypted without parsing
the qcow2 files :-/
Add the actual encrypted filename to the error message so that a UI can display
that information to the user.
Signed-off-by: Anthony Liguori <address@hidden>
---
monitor.c | 3 ++-
qerror.h | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/monitor.c b/monitor.c
index 5595565..5cb36cd 100644
--- a/monitor.c
+++ b/monitor.c
@@ -5319,7 +5319,8 @@ int monitor_read_bdrv_key_start(Monitor *mon,
BlockDriverState *bs,
}
if (monitor_ctrl_mode(mon)) {
- qerror_report(QERR_DEVICE_ENCRYPTED, bdrv_get_device_name(bs));
+ qerror_report(QERR_DEVICE_ENCRYPTED, bdrv_get_device_name(bs),
+ bdrv_get_encrypted_filename(bs));
return -1;
}
diff --git a/qerror.h b/qerror.h
index 4fe24aa..3bbff5c 100644
--- a/qerror.h
+++ b/qerror.h
@@ -64,7 +64,7 @@ QError *qobject_to_qerror(const QObject *obj);
"{ 'class': 'CommandNotFound', 'data': { 'name': %s } }"
#define QERR_DEVICE_ENCRYPTED \
- "{ 'class': 'DeviceEncrypted', 'data': { 'device': %s } }"
+ "{ 'class': 'DeviceEncrypted', 'data': { 'device': %s, 'filename': %s} }"
#define QERR_DEVICE_INIT_FAILED \
"{ 'class': 'DeviceInitFailed', 'data': { 'device': %s } }"
--
1.7.4.1
- [Qemu-devel] [PATCH 00/14] Convert commands to QAPI (batch 1), Anthony Liguori, 2011/08/24
- [Qemu-devel] [PATCH 01/14] qerror: add qerror_report_err(), Anthony Liguori, 2011/08/24
- [Qemu-devel] [PATCH 07/14] qerror: add additional parameter to QERR_DEVICE_ENCRYPTED,
Anthony Liguori <=
- [Qemu-devel] [PATCH 05/14] block: add unsafe_probe, Anthony Liguori, 2011/08/24
- [Qemu-devel] [PATCH 11/14] qapi: add change-vnc-listen, Anthony Liguori, 2011/08/24
- [Qemu-devel] [PATCH 06/14] monitor: expose readline state, Anthony Liguori, 2011/08/24
- [Qemu-devel] [PATCH 08/14] qapi: convert eject (qmp and hmp) to QAPI, Anthony Liguori, 2011/08/24