~vcs-imports/qemu/git

~vcs-imports/qemu/git » Changes to qemu-io.c from revision 8491

From Revision 8491

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
8405		raw-posix: add Linux native AIO support raw-posix: add Linux native AIO support Now that do have a nicer interface to work against we can add Linux native AIO support.  It's an extremly thing layer just setting up an iocb for the io_submit system call in the submission path, and registering an eventfd with the qemu poll handler to do complete the iocbs directly from there. This started out based on Anthony's earlier AIO patch, but after estimated 42,000 rewrites and just as many build system changes there's not much left of it. To enable native kernel aio use the aio=native sub-command on the drive command line.  I have also added an option to qemu-io to test the aio support without needing a guest. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Christoph Hellwig	15 years ago
8102		Fix Sparse warnings: "Using plain integer as NULL pointer" Fix Sparse warnings: "Using plain integer as NULL pointer" Signed-off-by: Blue Swirl <blauwirbel@gmail.com>	Blue Swirl	15 years ago
7977		qemu-io: reject invalid pattern qemu-io: reject invalid pattern Replace the use of atoi which is used for pattern parsing currently with strtol.  Atoi won't parse sedecimal pattern values (it always returns 0), but qemu-iotests use such pattern values.  Also reject every pattern that is not a unsigned char as we pass the pattern to memset which expect a bye value (despite having the pattern argument declared as int). Based on an earlier patch by Stefan Weil which did not include the error handling. Signed-off-by: Christoph Hellwig <hch@lst.de> Reported-by: Stefan Weil <weil@mail.berlios.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Christoph Hellwig	15 years ago
7976		qemu-io: Rework alloc command qemu-io: Rework alloc command The alloc command in qemu-io is mostly useless currently. Instead of doing a single call to bdrv_is_allocated, we must call bdrv_is_allocated in a loop until we have found out for each requested sector if it is allocated or not (bdrv_is_allocated returns a number of sectors that are known to be in the same state as the first one, but it is not required to include all of them) This changes the output format of the alloc command so that a change to the expected output of qemu-iotests 019 is necessary once this is included. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Kevin Wolf	15 years ago
7873		qemu-io: use BDRV_O_FILE to implement the growable open opti... qemu-io: use BDRV_O_FILE to implement the growable open option Instead of doing our own check for protocols which fails because raw isn't formally a protocol but special cased in find_protocol specify the BDRV_O_FILE option to use the same code as bdrv_file_open does. While we're at it also add the missing documentation for -g to the main qemu-io help string. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Christoph Hellwig	15 years ago
7870		qemu-io: Implement bdrv_load_vmstate/bdrv_save_vmstate qemu-io: Implement bdrv_load_vmstate/bdrv_save_vmstate The load_vmstate and save_vmstate functions are implemented as a variation of the normal read/write operation, enabled by the -b option. This is the same mechanism as is used to switch from read/write to pread/pwrite. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Kevin Wolf	15 years ago
7795		qemu-io: fix memory leak qemu-io: fix memory leak qemu-io leaks the request buffer whenever the read or write function isn't executed completely down the "normal" code path. [hch: also fix the aio and vectored variants the same way] Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Christoph Hellwig <hch@lst.de>	Kevin Wolf	15 years ago
7794		qemu-io: add flag to mark files growable qemu-io: add flag to mark files growable Add a -g flag to the open command and the main qemu-io command line to allow opening a file growable.  This is only allowed for protocols, mirroring the limitation exposed through bdrv_file_open. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Kevin Wolf <kwolf@redhat.com>	Christoph Hellwig	15 years ago
7793		qemu-io: better input validation for vector-based commands qemu-io: better input validation for vector-based commands Fix up a couple of issues with validating the input of the various length arguments for the vectored I/O commands:  - do the alignment check on each length instead the always 0 count argument  - use a long long varibale for the cvtnum return value so that we can check    wether it wasn't a number  - check for a too large argument instead of truncating it Also refactor it into a common helper for all four calers and avoid parsing the numbers twice. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Kevin Wolf <kwolf@redhat.com>	Christoph Hellwig	15 years ago
7792		qemu-io: small cleanups for the aio code qemu-io: small cleanups for the aio code Address a couple of review comments from Kevin. Signed-off-by: Christoph Hellwig <hch@lst.de>	Christoph Hellwig	15 years ago
7650		qemu-io: add aio read/write/flush commands qemu-io: add aio read/write/flush commands Add commands to exercise asynchronous reads/writes and to flush all outstanding aio commands.  Commands to exercise aio cancellations will follow in a separate patch. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Christoph Hellwig	15 years ago
7649		qemu-io: Fix spelling in help message. qemu-io: Fix spelling in help message. excerciser -> exerciser Signed-off-by: Stefan Weil <weil@mail.berlios.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Stefan Weil	15 years ago
7648		Fix dump output in qemu-io. Fix dump output in qemu-io. The dump output was not nicely formatted for bytes larger than 0x7f, because signed values expanded to sizeof(int) bytes. So for example 0xab did not print as "ab", but as "ffffffab". I also cleaned the function prototype, which avoids new type casts and allows to remove an existing type cast. Signed-off-by: Stefan Weil <weil@mail.berlios.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Stefan Weil	15 years ago
7252		Workaround compiler warnings at -O1 Workaround compiler warnings at -O1 Signed-off-by: Paul Brook <paul@codesourcery.com>	Paul Brook	15 years ago
7198		qemu-io: Optionally verify only part of read data qemu-io: Optionally verify only part of read data There are reasonable test cases where a read must span areas that are not uniformly filled with one pattern but contains several parts. This makes -P useless for them currently. Introducing additional options which determine the part of the read data that should be verified with the given pattern allows to check such reads. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>	Kevin Wolf	15 years ago
7100		qemu-io: Verify read data by patterns (Kevin Wolf) qemu-io: Verify read data by patterns (Kevin Wolf) This patch adds a -P option to read and readv which allows to compare the read data to a given pattern. This can be used to verify data written by write -P. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@7182 c046a42c-6fe2-441c-8c8c-71466251a162	aliguori	15 years ago
7099		qemu-io: Fix handling of bdrv_is_allocated() return value (K... qemu-io: Fix handling of bdrv_is_allocated() return value (Kevin Wolf) bdrv_is_allocated() returns a boolean which indicates if the offset is allocated, not 0 on success and everything else is an error. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@7181 c046a42c-6fe2-441c-8c8c-71466251a162	aliguori	15 years ago
7096		a couple of qemu-io fixes (Christoph Hellwig) a couple of qemu-io fixes (Christoph Hellwig) Fix two stupid bugs that I forgot to push out:  - qiov->size already is in bytes, no need to shift it  - actually use the supplied patter in the writev command Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@7178 c046a42c-6fe2-441c-8c8c-71466251a162	aliguori	15 years ago
6938		Add files not included in previous commit. Add files not included in previous commit. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@6995 c046a42c-6fe2-441c-8c8c-71466251a162	aliguori	15 years ago