[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH] block: Make block error codes consistent
From: |
Legend Tang |
Subject: |
Re: [Qemu-block] [PATCH] block: Make block error codes consistent |
Date: |
Sun, 12 May 2019 23:42:47 +0800 |
Please ignore this submission. The signed-off-by line is somehow missing.
> 在 2019年5月12日,23:10,Wensheng Tang <address@hidden> 写道:
>
> We should keep the error handling consistent. ENOMEDIUM is more meaningful
> than ENOENT a when driver cannot be loaded.
> ---
> block.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/block.c b/block.c
> index 5c2c6aa..6685be7 100644
> --- a/block.c
> +++ b/block.c
> @@ -770,7 +770,7 @@ static int find_image_format(BlockBackend *file, const
> char *filename,
> if (!drv) {
> error_setg(errp, "Could not determine image format: No compatible "
> "driver found");
> - ret = -ENOENT;
> + ret = -ENOMEDIUM;
> }
> *pdrv = drv;
> return ret;
> @@ -1619,7 +1619,7 @@ static int bdrv_fill_options(QDict **options, const
> char *filename,
> drv = bdrv_find_format(drvname);
> if (!drv) {
> error_setg(errp, "Unknown driver '%s'", drvname);
> - return -ENOENT;
> + return -ENOMEDIUM;
> }
> /* If the user has explicitly specified the driver, this choice should
> * override the BDRV_O_PROTOCOL flag */
> @@ -1655,7 +1655,7 @@ static int bdrv_fill_options(QDict **options, const
> char *filename,
> if (filename) {
> drv = bdrv_find_protocol(filename, parse_filename, errp);
> if (!drv) {
> - return -EINVAL;
> + return -ENOMEDIUM;
> }
>
> drvname = drv->format_name;
> --
> 2.20.1 (Apple Git-117)
>