diff options
| author | Stefan Hajnoczi <stefanha@redhat.com> | 2013-02-13 16:53:43 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2013-02-22 21:21:09 +0100 |
| commit | 7d81c1413c9c9bdcc966453636e4ca7776b59861 (patch) | |
| tree | 25d87d2bbcb2d83ea366fe3e9d0b457f8218bc13 | |
| parent | c546194f260fb3e391193cb8cc33505618077ecb (diff) | |
| download | qemu-7d81c1413c9c9bdcc966453636e4ca7776b59861.tar.gz qemu-7d81c1413c9c9bdcc966453636e4ca7776b59861.tar.bz2 qemu-7d81c1413c9c9bdcc966453636e4ca7776b59861.zip | |
block: refuse negative iops and bps values
Negative I/O throttling iops and bps values do not make sense so reject
them with an error message.
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
| -rw-r--r-- | blockdev.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/blockdev.c b/blockdev.c index 9b0351343e..ba3759c849 100644 --- a/blockdev.c +++ b/blockdev.c @@ -274,6 +274,16 @@ static bool do_check_io_limits(BlockIOLimit *io_limits, Error **errp) return false; } + if (io_limits->bps[BLOCK_IO_LIMIT_TOTAL] < 0 || + io_limits->bps[BLOCK_IO_LIMIT_WRITE] < 0 || + io_limits->bps[BLOCK_IO_LIMIT_READ] < 0 || + io_limits->iops[BLOCK_IO_LIMIT_TOTAL] < 0 || + io_limits->iops[BLOCK_IO_LIMIT_WRITE] < 0 || + io_limits->iops[BLOCK_IO_LIMIT_READ] < 0) { + error_setg(errp, "bps and iops values must be 0 or greater"); + return false; + } + return true; } |