[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH for-2.10 05/16] block/file-posix: Small fixes in raw
From: |
Max Reitz |
Subject: |
[Qemu-block] [PATCH for-2.10 05/16] block/file-posix: Small fixes in raw_create() |
Date: |
Mon, 13 Mar 2017 22:40:34 +0100 |
Variables should be declared at the start of a block, and if a certain
parameter value is not supported it may be better to return -ENOTSUP
instead of -EINVAL.
Signed-off-by: Max Reitz <address@hidden>
---
block/file-posix.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/block/file-posix.c b/block/file-posix.c
index dde8c101c8..b02d8fc37b 100644
--- a/block/file-posix.c
+++ b/block/file-posix.c
@@ -1670,6 +1670,8 @@ static int raw_create(const char *filename, QemuOpts
*opts, Error **errp)
#endif
case PREALLOC_MODE_FULL:
{
+ int64_t num = 0, left = total_size;
+
/*
* Knowing the final size from the beginning could allow the file
* system driver to do less allocations and possibly avoid
@@ -1681,7 +1683,6 @@ static int raw_create(const char *filename, QemuOpts
*opts, Error **errp)
goto out_close;
}
- int64_t num = 0, left = total_size;
buf = g_malloc0(65536);
while (left > 0) {
@@ -1713,7 +1714,7 @@ static int raw_create(const char *filename, QemuOpts
*opts, Error **errp)
}
break;
default:
- result = -EINVAL;
+ result = -ENOTSUP;
error_setg(errp, "Unsupported preallocation mode: %s",
PreallocMode_lookup[prealloc]);
break;
--
2.12.0
- Re: [Qemu-block] [PATCH for-2.10 01/16] block: Add PreallocMode to BD.bdrv_truncate(), (continued)
[Qemu-block] [PATCH for-2.10 02/16] block: Add PreallocMode to bdrv_truncate(), Max Reitz, 2017/03/13
[Qemu-block] [PATCH for-2.10 03/16] block: Add PreallocMode to blk_truncate(), Max Reitz, 2017/03/13
[Qemu-block] [PATCH for-2.10 04/16] qemu-img: Expose PreallocMode for resizing, Max Reitz, 2017/03/13
[Qemu-block] [PATCH for-2.10 05/16] block/file-posix: Small fixes in raw_create(),
Max Reitz <=
[Qemu-block] [PATCH for-2.10 06/16] block/file-posix: Extract raw_regular_truncate(), Max Reitz, 2017/03/13
[Qemu-block] [PATCH for-2.10 07/16] block/file-posix: Generalize raw_regular_truncate, Max Reitz, 2017/03/13
[Qemu-block] [PATCH for-2.10 08/16] block/file-posix: Preallocation for truncate, Max Reitz, 2017/03/13