virtio-rng: add rate limiting support
This adds parameters to virtio-rng-pci to allow rate limiting the entropy a guest receives. An example command line: $ qemu -device virtio-rng-pci,max-bytes=1024,period=1000 Would limit entropy collection to 1Kb/s. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
parent
16c915ba42
commit
904d6f5880
@ -1015,6 +1015,13 @@ static void virtio_rng_initfn(Object *obj)
|
|||||||
|
|
||||||
static Property virtio_rng_properties[] = {
|
static Property virtio_rng_properties[] = {
|
||||||
DEFINE_VIRTIO_COMMON_FEATURES(VirtIOPCIProxy, host_features),
|
DEFINE_VIRTIO_COMMON_FEATURES(VirtIOPCIProxy, host_features),
|
||||||
|
/* Set a default rate limit of 2^47 bytes per minute or roughly 2TB/s. If
|
||||||
|
you have an entropy source capable of generating more entropy than this
|
||||||
|
and you can pass it through via virtio-rng, then hats off to you. Until
|
||||||
|
then, this is unlimited for all practical purposes.
|
||||||
|
*/
|
||||||
|
DEFINE_PROP_UINT64("max-bytes", VirtIOPCIProxy, rng.max_bytes, INT64_MAX),
|
||||||
|
DEFINE_PROP_UINT32("period", VirtIOPCIProxy, rng.period_ms, 1 << 16),
|
||||||
DEFINE_PROP_END_OF_LIST(),
|
DEFINE_PROP_END_OF_LIST(),
|
||||||
};
|
};
|
||||||
|
|
||||||
|
@ -31,6 +31,12 @@ typedef struct VirtIORNG {
|
|||||||
bool popped;
|
bool popped;
|
||||||
|
|
||||||
RngBackend *rng;
|
RngBackend *rng;
|
||||||
|
|
||||||
|
/* We purposefully don't migrate this state. The quota will reset on the
|
||||||
|
* destination as a result. Rate limiting is host state, not guest state.
|
||||||
|
*/
|
||||||
|
QEMUTimer *rate_limit_timer;
|
||||||
|
int64_t quota_remaining;
|
||||||
} VirtIORNG;
|
} VirtIORNG;
|
||||||
|
|
||||||
static bool is_guest_ready(VirtIORNG *vrng)
|
static bool is_guest_ready(VirtIORNG *vrng)
|
||||||
@ -55,6 +61,8 @@ static size_t pop_an_elem(VirtIORNG *vrng)
|
|||||||
return size;
|
return size;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static void virtio_rng_process(VirtIORNG *vrng);
|
||||||
|
|
||||||
/* Send data from a char device over to the guest */
|
/* Send data from a char device over to the guest */
|
||||||
static void chr_read(void *opaque, const void *buf, size_t size)
|
static void chr_read(void *opaque, const void *buf, size_t size)
|
||||||
{
|
{
|
||||||
@ -66,6 +74,8 @@ static void chr_read(void *opaque, const void *buf, size_t size)
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
vrng->quota_remaining -= size;
|
||||||
|
|
||||||
offset = 0;
|
offset = 0;
|
||||||
while (offset < size) {
|
while (offset < size) {
|
||||||
if (!pop_an_elem(vrng)) {
|
if (!pop_an_elem(vrng)) {
|
||||||
@ -85,21 +95,30 @@ static void chr_read(void *opaque, const void *buf, size_t size)
|
|||||||
* didn't have enough data to fill them all, indicate we want more
|
* didn't have enough data to fill them all, indicate we want more
|
||||||
* data.
|
* data.
|
||||||
*/
|
*/
|
||||||
len = pop_an_elem(vrng);
|
virtio_rng_process(vrng);
|
||||||
if (len) {
|
}
|
||||||
|
|
||||||
|
static void virtio_rng_process(VirtIORNG *vrng)
|
||||||
|
{
|
||||||
|
ssize_t size;
|
||||||
|
|
||||||
|
if (!is_guest_ready(vrng)) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
size = pop_an_elem(vrng);
|
||||||
|
size = MIN(vrng->quota_remaining, size);
|
||||||
|
|
||||||
|
if (size > 0) {
|
||||||
rng_backend_request_entropy(vrng->rng, size, chr_read, vrng);
|
rng_backend_request_entropy(vrng->rng, size, chr_read, vrng);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
static void handle_input(VirtIODevice *vdev, VirtQueue *vq)
|
static void handle_input(VirtIODevice *vdev, VirtQueue *vq)
|
||||||
{
|
{
|
||||||
VirtIORNG *vrng = DO_UPCAST(VirtIORNG, vdev, vdev);
|
VirtIORNG *vrng = DO_UPCAST(VirtIORNG, vdev, vdev);
|
||||||
size_t size;
|
virtio_rng_process(vrng);
|
||||||
|
|
||||||
size = pop_an_elem(vrng);
|
|
||||||
if (size) {
|
|
||||||
rng_backend_request_entropy(vrng->rng, size, chr_read, vrng);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static uint32_t get_features(VirtIODevice *vdev, uint32_t f)
|
static uint32_t get_features(VirtIODevice *vdev, uint32_t f)
|
||||||
@ -163,9 +182,27 @@ static int virtio_rng_load(QEMUFile *f, void *opaque, int version_id)
|
|||||||
virtqueue_map_sg(vrng->elem.out_sg, vrng->elem.out_addr,
|
virtqueue_map_sg(vrng->elem.out_sg, vrng->elem.out_addr,
|
||||||
vrng->elem.out_num, 0);
|
vrng->elem.out_num, 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* We may have an element ready but couldn't process it due to a quota
|
||||||
|
limit. Make sure to try again after live migration when the quota may
|
||||||
|
have been reset.
|
||||||
|
*/
|
||||||
|
virtio_rng_process(vrng);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static void check_rate_limit(void *opaque)
|
||||||
|
{
|
||||||
|
VirtIORNG *s = opaque;
|
||||||
|
|
||||||
|
s->quota_remaining = s->conf->max_bytes;
|
||||||
|
virtio_rng_process(s);
|
||||||
|
qemu_mod_timer(s->rate_limit_timer,
|
||||||
|
qemu_get_clock_ms(vm_clock) + s->conf->period_ms);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
VirtIODevice *virtio_rng_init(DeviceState *dev, VirtIORNGConf *conf)
|
VirtIODevice *virtio_rng_init(DeviceState *dev, VirtIORNGConf *conf)
|
||||||
{
|
{
|
||||||
VirtIORNG *vrng;
|
VirtIORNG *vrng;
|
||||||
@ -196,6 +233,16 @@ VirtIODevice *virtio_rng_init(DeviceState *dev, VirtIORNGConf *conf)
|
|||||||
vrng->qdev = dev;
|
vrng->qdev = dev;
|
||||||
vrng->conf = conf;
|
vrng->conf = conf;
|
||||||
vrng->popped = false;
|
vrng->popped = false;
|
||||||
|
vrng->quota_remaining = vrng->conf->max_bytes;
|
||||||
|
|
||||||
|
g_assert_cmpint(vrng->conf->max_bytes, <=, INT64_MAX);
|
||||||
|
|
||||||
|
vrng->rate_limit_timer = qemu_new_timer_ms(vm_clock,
|
||||||
|
check_rate_limit, vrng);
|
||||||
|
|
||||||
|
qemu_mod_timer(vrng->rate_limit_timer,
|
||||||
|
qemu_get_clock_ms(vm_clock) + vrng->conf->period_ms);
|
||||||
|
|
||||||
register_savevm(dev, "virtio-rng", -1, 1, virtio_rng_save,
|
register_savevm(dev, "virtio-rng", -1, 1, virtio_rng_save,
|
||||||
virtio_rng_load, vrng);
|
virtio_rng_load, vrng);
|
||||||
|
|
||||||
|
@ -19,6 +19,8 @@
|
|||||||
|
|
||||||
struct VirtIORNGConf {
|
struct VirtIORNGConf {
|
||||||
RngBackend *rng;
|
RngBackend *rng;
|
||||||
|
uint64_t max_bytes;
|
||||||
|
uint32_t period_ms;
|
||||||
};
|
};
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
Loading…
Reference in New Issue
Block a user