[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] block: drive_init(): Improve CHS setting er
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] block: drive_init(): Improve CHS setting error message |
Date: |
Fri, 01 Jul 2011 09:21:33 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Luiz Capitulino <address@hidden> writes:
> The current media doesn't clearly say the error cause.
>
> Signed-off-by: Luiz Capitulino <address@hidden>
> ---
> blockdev.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/blockdev.c b/blockdev.c
> index 0a90ae8..2dbdd1b 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -311,7 +311,7 @@ DriveInfo *drive_init(QemuOpts *opts, int default_to_scsi)
> media = MEDIA_DISK;
> } else if (!strcmp(buf, "cdrom")) {
> if (cyls || secs || heads) {
> - error_report("'%s' invalid physical CHS format", buf);
> + error_report("CHS can't be set for CDROM media '%s'", buf);
> return NULL;
> }
> media = MEDIA_CDROM;
It's an improvement. I'd like "CHS can't be set with media=%s" even
better, because it's closer to the actual option string. Matter of
taste.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 3/3] block: drive_init(): Improve CHS setting error message,
Markus Armbruster <=