diff options
| author |   Rusty Russell <rusty@rustcorp.com.au> | 2014-03-13 11:23:40 +1030 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2014-03-13 11:27:57 +1030 |
| commit | 9914a766174d50eb2343f204fef3ee23dbe07c4c (patch) | |
| tree | b234adcb592f1fca7b82a761b701a16c129a0280 /drivers/char/hw_random | |
| parent | virtio_balloon: don't crash if virtqueue is broken. (diff) | |
| download | linux-dev-9914a766174d50eb2343f204fef3ee23dbe07c4c.tar.xz linux-dev-9914a766174d50eb2343f204fef3ee23dbe07c4c.zip | |
virtio-rng: don't crash if virtqueue is broken.
A bad implementation of virtio might cause us to mark the virtqueue
broken: we'll dev_err() in that case, and the device is useless, but
let's not BUG().
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'drivers/char/hw_random')
| -rw-r--r-- | drivers/char/hw_random/virtio-rng.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/char/hw_random/virtio-rng.c b/drivers/char/hw_random/virtio-rng.c index c12398d1517c..2ce0e225e58c 100644 --- a/drivers/char/hw_random/virtio-rng.c +++ b/drivers/char/hw_random/virtio-rng.c @@ -47,8 +47,7 @@ static void register_buffer(u8 *buf, size_t size) sg_init_one(&sg, buf, size); /* There should always be room for one buffer. */ - if (virtqueue_add_inbuf(vq, &sg, 1, buf, GFP_KERNEL) < 0) - BUG(); + virtqueue_add_inbuf(vq, &sg, 1, buf, GFP_KERNEL); virtqueue_kick(vq); } |