virtio_blk: fix config handler race
authorMichael S. Tsirkin <mst@redhat.com>
Thu, 12 Jan 2012 05:14:44 +0000 (15:44 +1030)
committerBen Hutchings <ben@decadent.org.uk>
Wed, 12 Sep 2012 02:37:31 +0000 (03:37 +0100)
commit0984c4c423da091cf8221e74b030c116d865dacf
tree75742758e4aa011e78d22c6fd5736de2544f930d
parentd9033bcc75a39d79411c722502665ccaab0dd4d6
virtio_blk: fix config handler race

commit 4678d6f970c2f7c0cbfefc0cc666432d153b321b upstream.

Fix a theoretical race related to config work
handler: a config interrupt might happen
after we flush config work but before we
reset the device. It will then cause the
config work to run during or after reset.

Two problems with this:
- if this runs after device is gone we will get use after free
- access of config while reset is in progress is racy
(as layout is changing).

As a solution
1. flush after reset when we know there will be no more interrupts
2. add a flag to disable config access before reset

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/block/virtio_blk.c