block: Don't check events on close unless it was blocked
authorTejun Heo <tj@kernel.org>
Wed, 9 Mar 2011 18:54:27 +0000 (19:54 +0100)
committerTejun Heo <tj@kernel.org>
Wed, 9 Mar 2011 18:54:27 +0000 (19:54 +0100)
commit6936217cc7e58573026bdba25b1bfb778e8f2267
tree7ddfbc4912d9a29a9719c13b8d1b58a3b3291762
parentfacc31ddc3570a3a0d8951c94f16b898e01b464d
block: Don't check events on close unless it was blocked

The block event mechanism currently always checks events when the
device is being closed regardless of the open mode.  The intention was
to allow detection of EJECT_REQUEST when a device is closed whether
disk event polling is enabled or not.

This is unnecessary as, for devices of interest, events are checked
from either userland or kernel and in the former case ->check_events()
is performed on open of each poll attempt anyway.  Furthermore, this
unconditional event check on close makes the code susceptible to event
loop if the block driver doesn't clear reported events correctly - an
event triggers userland to open and close the device which in turn
causes another event, rinse and repeat.

Check events on close only if it was blocked by excl write open.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Jens Axboe <axboe@kernel.dk>
Cc: Kay Sievers <kay.sievers@vrfy.org>
fs/block_dev.c