From: Michael S. Tsirkin Date: Fri, 28 Sep 2012 05:35:16 +0000 (+0930) Subject: virtio: don't crash when device is buggy X-Git-Tag: upstream/snapshot3+hdmi~6518^2~5 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5543a6ac31eb4bfd8d938db6b234ce833d14e04e;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git virtio: don't crash when device is buggy Because of a sanity check in virtio_dev_remove, a buggy device can crash kernel. And in case of rproc it's userspace so it's not a good idea. We are unloading a driver so how bad can it be? Be less aggressive in handling this error: if it's a driver bug, warning once should be enough. Signed-off-by: Michael S. Tsirkin Signed-off-by: Rusty Russell --- diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c index c3b3f7f..1e8659c 100644 --- a/drivers/virtio/virtio.c +++ b/drivers/virtio/virtio.c @@ -159,7 +159,7 @@ static int virtio_dev_remove(struct device *_d) drv->remove(dev); /* Driver should have reset device. */ - BUG_ON(dev->config->get_status(dev)); + WARN_ON_ONCE(dev->config->get_status(dev)); /* Acknowledge the device's existence again. */ add_status(dev, VIRTIO_CONFIG_S_ACKNOWLEDGE);