OpenE2K
/
qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

qed: Use BB functions in .bdrv_create() 

All users of the block layers are supposed to go through a BlockBackend.
The .bdrv_create() implementation is one such user, so this patch
converts it.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
Kevin Wolf 2016-03-08 15:57:05 +01:00
parent 23588797b6
commit 8a56fdadaf
1 changed files with 15 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
block/qed.c
View File

@ -18,6 +18,7 @@
#include "qed.h" #include "qed.h"
#include "qapi/qmp/qerror.h" #include "qapi/qmp/qerror.h"
#include "migration/migration.h" #include "migration/migration.h"
#include "sysemu/block-backend.h"
static const AIOCBInfo qed_aiocb_info = { static const AIOCBInfo qed_aiocb_info = {
.aiocb_size = sizeof(QEDAIOCB), .aiocb_size = sizeof(QEDAIOCB),
@ -580,7 +581,7 @@ static int qed_create(const char *filename, uint32_t cluster_size,
size_t l1_size = header.cluster_size * header.table_size; size_t l1_size = header.cluster_size * header.table_size;
Error *local_err = NULL; Error *local_err = NULL;
int ret = 0; int ret = 0;
BlockDriverState *bs; BlockBackend *blk;
ret = bdrv_create_file(filename, opts, &local_err); ret = bdrv_create_file(filename, opts, &local_err);
if (ret < 0) { if (ret < 0) {
@ -588,17 +589,18 @@ static int qed_create(const char *filename, uint32_t cluster_size,
return ret; return ret;
} }
bs = NULL; blk = blk_new_open("image", filename, NULL, NULL,
ret = bdrv_open(&bs, filename, NULL, NULL, BDRV_O_RDWR | BDRV_O_CACHE_WB | BDRV_O_PROTOCOL,
BDRV_O_RDWR | BDRV_O_CACHE_WB | BDRV_O_PROTOCOL, &local_err);
&local_err); if (blk == NULL) {
if (ret < 0) {
error_propagate(errp, local_err); error_propagate(errp, local_err);
return ret; return -EIO;
} }
blk_set_allow_write_beyond_eof(blk, true);
/* File must start empty and grow, check truncate is supported */ /* File must start empty and grow, check truncate is supported */
ret = bdrv_truncate(bs, 0); ret = blk_truncate(blk, 0);
if (ret < 0) { if (ret < 0) {
goto out; goto out;
} }
@ -614,18 +616,18 @@ static int qed_create(const char *filename, uint32_t cluster_size,
} }
qed_header_cpu_to_le(&header, &le_header); qed_header_cpu_to_le(&header, &le_header);
ret = bdrv_pwrite(bs, 0, &le_header, sizeof(le_header)); ret = blk_pwrite(blk, 0, &le_header, sizeof(le_header));
if (ret < 0) { if (ret < 0) {
goto out; goto out;
} }
ret = bdrv_pwrite(bs, sizeof(le_header), backing_file, ret = blk_pwrite(blk, sizeof(le_header), backing_file,
header.backing_filename_size); header.backing_filename_size);
if (ret < 0) { if (ret < 0) {
goto out; goto out;
} }
l1_table = g_malloc0(l1_size); l1_table = g_malloc0(l1_size);
ret = bdrv_pwrite(bs, header.l1_table_offset, l1_table, l1_size); ret = blk_pwrite(blk, header.l1_table_offset, l1_table, l1_size);
if (ret < 0) { if (ret < 0) {
goto out; goto out;
} }
@ -633,7 +635,7 @@ static int qed_create(const char *filename, uint32_t cluster_size,
ret = 0; /* success */ ret = 0; /* success */
out: out:
g_free(l1_table); g_free(l1_table);
bdrv_unref(bs); blk_unref(blk);
return ret; return ret;
} }