[S390] cio: Avoid machine check vs. not operational races.
authorCornelia Huck <cornelia.huck@de.ibm.com>
Fri, 12 Oct 2007 14:11:26 +0000 (16:11 +0200)
committerMartin Schwidefsky <schwidefsky@de.ibm.com>
Fri, 12 Oct 2007 14:13:02 +0000 (16:13 +0200)
commit3f4cf6e72f9f6a0b046b32881acc4f829f3aaa46
tree84302932647aed49fa65b506fd0f5652af276915
parent3952c8d43aa23862181ad23ecdf72895b4be3143
[S390] cio: Avoid machine check vs. not operational races.

There was the possibilty that an action like ccw_device_set_offline()
triggered by a device gone machine check might trigger a not oper
event. Unfortunately, this could lead to the situation that we tried
to unregister a subchannel twice: Once from the slow path evaluation,
and once via the not oper event.

Fix this by always using the same mechanism (css_schedule_eval()) for
triggering the unregister. This makes sure that unregistration will
only be done once. As an added bonus, it also simplyfies the code.

Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
drivers/s390/cio/device.c
drivers/s390/cio/device.h
drivers/s390/cio/device_fsm.c