dmaengine.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] dmaengine: idxd: fix misc interrupt handler thread unmasking
@ 2020-06-25 19:16 Dave Jiang
  2020-07-02 13:32 ` Vinod Koul
  2020-07-02 13:33 ` Vinod Koul
  0 siblings, 2 replies; 3+ messages in thread
From: Dave Jiang @ 2020-06-25 19:16 UTC (permalink / raw)
  To: vkoul; +Cc: dmaengine

Fix unmasking of misc interrupt handler when completing normal. It exits
early and skips the unmasking with the current implementation. Fix to
unmask interrupt when exiting normally.

Fixes: bfe1d56091c1 ("dmaengine: idxd: Init and probe for Intel data accelerators")
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
---

Rebased against dmaengine fixes branch

 drivers/dma/idxd/irq.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/dma/idxd/irq.c b/drivers/dma/idxd/irq.c
index 6510791b9921..8a35f58da689 100644
--- a/drivers/dma/idxd/irq.c
+++ b/drivers/dma/idxd/irq.c
@@ -141,7 +141,7 @@ irqreturn_t idxd_misc_thread(int vec, void *data)
 
 	iowrite32(cause, idxd->reg_base + IDXD_INTCAUSE_OFFSET);
 	if (!err)
-		return IRQ_HANDLED;
+		goto out;
 
 	gensts.bits = ioread32(idxd->reg_base + IDXD_GENSTATS_OFFSET);
 	if (gensts.state == IDXD_DEVICE_STATE_HALT) {
@@ -162,6 +162,7 @@ irqreturn_t idxd_misc_thread(int vec, void *data)
 		spin_unlock_bh(&idxd->dev_lock);
 	}
 
+ out:
 	idxd_unmask_msix_vector(idxd, irq_entry->id);
 	return IRQ_HANDLED;
 }


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] dmaengine: idxd: fix misc interrupt handler thread unmasking
  2020-06-25 19:16 [PATCH v2] dmaengine: idxd: fix misc interrupt handler thread unmasking Dave Jiang
@ 2020-07-02 13:32 ` Vinod Koul
  2020-07-02 13:33 ` Vinod Koul
  1 sibling, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2020-07-02 13:32 UTC (permalink / raw)
  To: Dave Jiang; +Cc: dmaengine

On 25-06-20, 12:16, Dave Jiang wrote:
> Fix unmasking of misc interrupt handler when completing normal. It exits
> early and skips the unmasking with the current implementation. Fix to
> unmask interrupt when exiting normally.

Applied, thanks

-- 
~Vinod

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] dmaengine: idxd: fix misc interrupt handler thread unmasking
  2020-06-25 19:16 [PATCH v2] dmaengine: idxd: fix misc interrupt handler thread unmasking Dave Jiang
  2020-07-02 13:32 ` Vinod Koul
@ 2020-07-02 13:33 ` Vinod Koul
  1 sibling, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2020-07-02 13:33 UTC (permalink / raw)
  To: Dave Jiang; +Cc: dmaengine

On 25-06-20, 12:16, Dave Jiang wrote:
> Fix unmasking of misc interrupt handler when completing normal. It exits
> early and skips the unmasking with the current implementation. Fix to
> unmask interrupt when exiting normally.

Applied, thanks

-- 
~Vinod

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-07-02 13:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-25 19:16 [PATCH v2] dmaengine: idxd: fix misc interrupt handler thread unmasking Dave Jiang
2020-07-02 13:32 ` Vinod Koul
2020-07-02 13:33 ` Vinod Koul

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).