[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 6/6] block/fileb: switch to use qemu_open/qemu_create for
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v4 6/6] block/fileb: switch to use qemu_open/qemu_create for improved errors |
Date: |
Tue, 25 Aug 2020 17:28:24 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Daniel P. Berrangé <berrange@redhat.com> writes:
> Currently at startup if using cache=none on a filesystem lacking
> O_DIRECT such as tmpfs, at startup QEMU prints
>
> qemu-system-x86_64: -drive file=/tmp/foo.img,cache=none: file system may not
> support O_DIRECT
> qemu-system-x86_64: -drive file=/tmp/foo.img,cache=none: Could not open
> '/tmp/foo.img': Invalid argument
>
> while at QMP level the hint is missing, so QEMU reports just
>
> "error": {
> "class": "GenericError",
> "desc": "Could not open '/tmp/foo.img': Invalid argument"
> }
>
> which is close to useless for the end user trying to figure out what
> they did wrong.
>
> With this change at startup QEMU prints
>
> qemu-system-x86_64: -drive file=/tmp/foo.img,cache=none: Unable to open
> '/tmp/foo.img' flags 0x4000: filesystem does not support O_DIRECT
>
> while at the QMP level QEMU reports a massively more informative
>
> "error": {
> "class": "GenericError",
> "desc": "Unable to open '/tmp/foo.img' flags 0x4002: filesystem does not
> support O_DIRECT"
> }
>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Both commit message and expected iotest results demonstrate the less
than helpful flags error reporting I pointed out in my review of PATCH
3.
Reviewed-by: Markus Armbruster <armbru@redhat.com>
- Re: [PATCH v4 3/6] util: add Error object for qemu_open_internal error reporting, (continued)
[PATCH v4 4/6] util: introduce qemu_open and qemu_create with error reporting, Daniel P . Berrangé, 2020/08/21
[PATCH v4 5/6] util: give a specific error message when O_DIRECT doesn't work, Daniel P . Berrangé, 2020/08/21
[PATCH v4 6/6] block/fileb: switch to use qemu_open/qemu_create for improved errors, Daniel P . Berrangé, 2020/08/21
- Re: [PATCH v4 6/6] block/fileb: switch to use qemu_open/qemu_create for improved errors,
Markus Armbruster <=