s390/vmur: remove unnecessary BUG statement
authorPeter Oberparleiter <oberpar@linux.ibm.com>
Wed, 28 Sep 2022 13:34:33 +0000 (15:34 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Mon, 10 Oct 2022 08:15:10 +0000 (10:15 +0200)
commitf3e59ff348c077a6afd4edb23d7e69e9cba62fdc
tree0e066cba6a6c804ee71bba2a2bedac68e8afaf37
parent03785a69ae47a17fe57fee31058fef7cd3042977
s390/vmur: remove unnecessary BUG statement

An existing BUG statement in vmur's interrupt handler triggers if:

  1. An online vmur device is removed (e.g. due to driver unload, manual
     unbind or channel-report words indicating hypervisor-side device
     removal)
  2. Device deactivation fails due to firmware/hypervisor error, leaving
     subchannel enabled for interrupts + drvdata=NULL
  3. Interrupt occurs

This situation is highly unlikely and not a clear indication of a
general system error that would warrant stopping the full Linux system.
Also it can be prevented completely by clearing the interrupt handler
when unsetting a vmur device's drvdata.

Replace the BUG statement in vmur's interrupt handler by clearing the
interrupt handler callback during device removal. Also move the initial
setting of the interrupt handler callback under lock for consistency
reasons.

Reviewed-by: Sven Schnelle <svens@linux.ibm.com>
Reviewed-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Peter Oberparleiter <oberpar@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
drivers/s390/char/vmur.c