diff options
| author | Jason Wang <jasowang@redhat.com> | 2014-05-12 16:35:39 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-07-28 08:06:02 -0700 |
| commit | e96faee7f217d9998e6602bc331780654fbdc023 (patch) | |
| tree | ad164de720c7f57502e1171b37751f3e0d5a1e60 /drivers/gpu | |
| parent | 345e666b2c8c8c78fbb5016f1807ef83279bece5 (diff) | |
| download | kernel-common-e96faee7f217d9998e6602bc331780654fbdc023.tar.gz kernel-common-e96faee7f217d9998e6602bc331780654fbdc023.tar.bz2 kernel-common-e96faee7f217d9998e6602bc331780654fbdc023.zip | |
drm/qxl: return IRQ_NONE if it was not our irq
commit fbb60fe35ad579b511de8604b06a30b43846473b upstream.
Return IRQ_NONE if it was not our irq. This is necessary for the case
when qxl is sharing irq line with a device A in a crash kernel. If qxl
is initialized before A and A's irq was raised during this gap,
returning IRQ_HANDLED in this case will cause this irq to be raised
again after EOI since kernel think it was handled but in fact it was
not.
Cc: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/qxl/qxl_irq.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/drm/qxl/qxl_irq.c b/drivers/gpu/drm/qxl/qxl_irq.c index 28f84b4fce32..3485bdccf8b8 100644 --- a/drivers/gpu/drm/qxl/qxl_irq.c +++ b/drivers/gpu/drm/qxl/qxl_irq.c @@ -33,6 +33,9 @@ irqreturn_t qxl_irq_handler(int irq, void *arg) pending = xchg(&qdev->ram_header->int_pending, 0); + if (!pending) + return IRQ_NONE; + atomic_inc(&qdev->irq_received); if (pending & QXL_INTERRUPT_DISPLAY) { |