block: kill BDRV_O_CREAT

The BDRV_O_CREAT option is unused inside qemu and partially duplicates
the bdrv_create method.  Remove it, and the -C option to qemu-io which
isn't used in qemu-iotests anyway.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
Christoph Hellwig 2010-01-20 18:13:42 +01:00 committed by Anthony Liguori
parent 6987307ca3
commit 9a2d77ad0d
5 changed files with 7 additions and 28 deletions

View File

@ -28,7 +28,6 @@ typedef struct QEMUSnapshotInfo {
} QEMUSnapshotInfo; } QEMUSnapshotInfo;
#define BDRV_O_RDWR 0x0002 #define BDRV_O_RDWR 0x0002
#define BDRV_O_CREAT 0x0004 /* create an empty file */
#define BDRV_O_SNAPSHOT 0x0008 /* open the file read only and save writes in a snapshot */ #define BDRV_O_SNAPSHOT 0x0008 /* open the file read only and save writes in a snapshot */
#define BDRV_O_FILE 0x0010 /* open as a raw file (do not try to #define BDRV_O_FILE 0x0010 /* open as a raw file (do not try to
use a disk image format on top of use a disk image format on top of

View File

@ -49,9 +49,6 @@ static int nbd_open(BlockDriverState *bs, const char* filename, int flags)
size_t blocksize; size_t blocksize;
int ret; int ret;
if ((flags & BDRV_O_CREAT))
return -EINVAL;
if (!strstart(filename, "nbd:", &host)) if (!strstart(filename, "nbd:", &host))
return -EINVAL; return -EINVAL;

View File

@ -205,13 +205,9 @@ out_close:
static int raw_open(BlockDriverState *bs, const char *filename, int flags) static int raw_open(BlockDriverState *bs, const char *filename, int flags)
{ {
BDRVRawState *s = bs->opaque; BDRVRawState *s = bs->opaque;
int open_flags = 0;
s->type = FTYPE_FILE; s->type = FTYPE_FILE;
if (flags & BDRV_O_CREAT) return raw_open_common(bs, filename, flags, 0);
open_flags = O_CREAT | O_TRUNC;
return raw_open_common(bs, filename, flags, open_flags);
} }
/* XXX: use host sector size if necessary with: /* XXX: use host sector size if necessary with:

View File

@ -76,7 +76,7 @@ static int set_sparse(int fd)
static int raw_open(BlockDriverState *bs, const char *filename, int flags) static int raw_open(BlockDriverState *bs, const char *filename, int flags)
{ {
BDRVRawState *s = bs->opaque; BDRVRawState *s = bs->opaque;
int access_flags, create_flags; int access_flags;
DWORD overlapped; DWORD overlapped;
s->type = FTYPE_FILE; s->type = FTYPE_FILE;
@ -86,11 +86,7 @@ static int raw_open(BlockDriverState *bs, const char *filename, int flags)
} else { } else {
access_flags = GENERIC_READ; access_flags = GENERIC_READ;
} }
if (flags & BDRV_O_CREAT) {
create_flags = CREATE_ALWAYS;
} else {
create_flags = OPEN_EXISTING;
}
overlapped = FILE_ATTRIBUTE_NORMAL; overlapped = FILE_ATTRIBUTE_NORMAL;
if ((flags & BDRV_O_NOCACHE)) if ((flags & BDRV_O_NOCACHE))
overlapped |= FILE_FLAG_NO_BUFFERING | FILE_FLAG_WRITE_THROUGH; overlapped |= FILE_FLAG_NO_BUFFERING | FILE_FLAG_WRITE_THROUGH;
@ -98,7 +94,7 @@ static int raw_open(BlockDriverState *bs, const char *filename, int flags)
overlapped |= FILE_FLAG_WRITE_THROUGH; overlapped |= FILE_FLAG_WRITE_THROUGH;
s->hfile = CreateFile(filename, access_flags, s->hfile = CreateFile(filename, access_flags,
FILE_SHARE_READ, NULL, FILE_SHARE_READ, NULL,
create_flags, overlapped, NULL); OPEN_EXISTING, overlapped, NULL);
if (s->hfile == INVALID_HANDLE_VALUE) { if (s->hfile == INVALID_HANDLE_VALUE) {
int err = GetLastError(); int err = GetLastError();

View File

@ -1307,7 +1307,6 @@ open_help(void)
" 'open -Cn /tmp/data' - creates/opens data file read-write and uncached\n" " 'open -Cn /tmp/data' - creates/opens data file read-write and uncached\n"
"\n" "\n"
" Opens a file for subsequent use by all of the other qemu-io commands.\n" " Opens a file for subsequent use by all of the other qemu-io commands.\n"
" -C, -- create new file if it doesn't exist\n"
" -r, -- open file read-only\n" " -r, -- open file read-only\n"
" -s, -- use snapshot file\n" " -s, -- use snapshot file\n"
" -n, -- disable host cache\n" " -n, -- disable host cache\n"
@ -1337,7 +1336,7 @@ open_f(int argc, char **argv)
int growable = 0; int growable = 0;
int c; int c;
while ((c = getopt(argc, argv, "snCrg")) != EOF) { while ((c = getopt(argc, argv, "snrg")) != EOF) {
switch (c) { switch (c) {
case 's': case 's':
flags |= BDRV_O_SNAPSHOT; flags |= BDRV_O_SNAPSHOT;
@ -1345,9 +1344,6 @@ open_f(int argc, char **argv)
case 'n': case 'n':
flags |= BDRV_O_NOCACHE; flags |= BDRV_O_NOCACHE;
break; break;
case 'C':
flags |= BDRV_O_CREAT;
break;
case 'r': case 'r':
readonly = 1; readonly = 1;
break; break;
@ -1395,10 +1391,9 @@ init_check_command(
static void usage(const char *name) static void usage(const char *name)
{ {
printf( printf(
"Usage: %s [-h] [-V] [-Crsnm] [-c cmd] ... [file]\n" "Usage: %s [-h] [-V] [-rsnm] [-c cmd] ... [file]\n"
"QEMU Disk exerciser\n" "QEMU Disk exerciser\n"
"\n" "\n"
" -C, --create create new file if it doesn't exist\n"
" -c, --cmd command to execute\n" " -c, --cmd command to execute\n"
" -r, --read-only export read-only\n" " -r, --read-only export read-only\n"
" -s, --snapshot use snapshot file\n" " -s, --snapshot use snapshot file\n"
@ -1417,13 +1412,12 @@ int main(int argc, char **argv)
{ {
int readonly = 0; int readonly = 0;
int growable = 0; int growable = 0;
const char *sopt = "hVc:Crsnmgk"; const char *sopt = "hVc:rsnmgk";
const struct option lopt[] = { const struct option lopt[] = {
{ "help", 0, NULL, 'h' }, { "help", 0, NULL, 'h' },
{ "version", 0, NULL, 'V' }, { "version", 0, NULL, 'V' },
{ "offset", 1, NULL, 'o' }, { "offset", 1, NULL, 'o' },
{ "cmd", 1, NULL, 'c' }, { "cmd", 1, NULL, 'c' },
{ "create", 0, NULL, 'C' },
{ "read-only", 0, NULL, 'r' }, { "read-only", 0, NULL, 'r' },
{ "snapshot", 0, NULL, 's' }, { "snapshot", 0, NULL, 's' },
{ "nocache", 0, NULL, 'n' }, { "nocache", 0, NULL, 'n' },
@ -1449,9 +1443,6 @@ int main(int argc, char **argv)
case 'c': case 'c':
add_user_command(optarg); add_user_command(optarg);
break; break;
case 'C':
flags |= BDRV_O_CREAT;
break;
case 'r': case 'r':
readonly = 1; readonly = 1;
break; break;