Skip to content
/ linux Public

Commit da0ce17

Browse files
Sebastian Andrzej SiewiorSasha Levin
authored andcommitted
perf/cxlpmu: Replace IRQF_ONESHOT with IRQF_NO_THREAD
[ Upstream commit ab26d9c ] Passing IRQF_ONESHOT ensures that the interrupt source is masked until the secondary (threaded) handler is done. If only a primary handler is used then the flag makes no sense because the interrupt can not fire (again) while its handler is running. The flag also disallows force-threading of the primary handler and the irq-core will warn about this. The intention here was probably not allowing forced-threading. Replace IRQF_ONESHOT with IRQF_NO_THREAD. Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> Signed-off-by: Will Deacon <will@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
1 parent 8e6ea0b commit da0ce17

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/perf/cxl_pmu.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -877,7 +877,7 @@ static int cxl_pmu_probe(struct device *dev)
877877
if (!irq_name)
878878
return -ENOMEM;
879879

880-
rc = devm_request_irq(dev, irq, cxl_pmu_irq, IRQF_SHARED | IRQF_ONESHOT,
880+
rc = devm_request_irq(dev, irq, cxl_pmu_irq, IRQF_SHARED | IRQF_NO_THREAD,
881881
irq_name, info);
882882
if (rc)
883883
return rc;

0 commit comments

Comments
 (0)