diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2011-05-26 03:48:23 -0400 |
---|---|---|
committer | Heiko Carstens <heiko.carstens@de.ibm.com> | 2011-05-26 03:48:24 -0400 |
commit | df7997ab1ca82ae3c37a2f5eb98613fc24527f95 (patch) | |
tree | 51794f46cb7fc7fa4db8fbadb8feb265fc8ef499 /arch/s390/mm/fault.c | |
parent | 902050bcdece6191565c055539e82c5cc534feed (diff) |
[S390] irq: fix service signal external interrupt handling
Interrupt sources like pfault, sclp, dasd_diag and virtio all use the
service signal external interrupt subclass mask in control register 0
to enable and disable the corresponding interrupt.
Because no reference counting is implemented each subsystem thinks it
is the only user of subclass and sets and clears the bit like it wants.
This leads to case that unloading the dasd diag module under z/VM
causes both sclp and pfault interrupts to be masked. The result will
be locked up system sooner or later.
Fix this by introducing a new way to set (register) and clear
(unregister) the service signal subclass mask bit in cr0.
Also convert all drivers.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'arch/s390/mm/fault.c')
-rw-r--r-- | arch/s390/mm/fault.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/s390/mm/fault.c b/arch/s390/mm/fault.c index e46ba2927424..6e922b50efa4 100644 --- a/arch/s390/mm/fault.c +++ b/arch/s390/mm/fault.c | |||
@@ -613,7 +613,7 @@ static int __init pfault_irq_init(void) | |||
613 | rc = pfault_init() == 0 ? 0 : -EOPNOTSUPP; | 613 | rc = pfault_init() == 0 ? 0 : -EOPNOTSUPP; |
614 | if (rc) | 614 | if (rc) |
615 | goto out_pfault; | 615 | goto out_pfault; |
616 | ctl_set_bit(0, 9); | 616 | service_subclass_irq_register(); |
617 | hotcpu_notifier(pfault_cpu_notify, 0); | 617 | hotcpu_notifier(pfault_cpu_notify, 0); |
618 | return 0; | 618 | return 0; |
619 | 619 | ||