aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/misc/cxl/native.c
diff options
context:
space:
mode:
authorIan Munsie <imunsie@au1.ibm.com>2016-05-04 00:52:58 -0400
committerMichael Ellerman <mpe@ellerman.id.au>2016-05-11 07:54:10 -0400
commit3c206fa77aaaac8cd7d4cfcd840c82495b01b288 (patch)
tree359020d4afbb21676e01d3df6dbf903ccdd558a2 /drivers/misc/cxl/native.c
parent0e5b5ba17ac33a05d9f4a48b5eb8b5e30f2274d7 (diff)
cxl: Ensure PSL interrupt is configured for contexts with no AFU IRQs
In the cxl kernel API, it is possible to create a context and start it without allocating any interrupts. Since we assign or allocate the PSL interrupt when allocating AFU interrupts this will lead to a situation where we start the context with no means to take any faults. The user API is not affected as it always goes through the cxl interrupt allocation code paths and will have the PSL interrupt allocated or assigned, even if no AFU interrupts were requested. This checks that at least one interrupt is configured at the time of attach, and if not it will assign the multiplexed PSL interrupt for powernv, or allocate a single interrupt for PowerVM. Signed-off-by: Ian Munsie <imunsie@au1.ibm.com> Reviewed-by: Frederic Barrat <fbarrat@linux.vnet.ibm.com> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'drivers/misc/cxl/native.c')
-rw-r--r--drivers/misc/cxl/native.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/drivers/misc/cxl/native.c b/drivers/misc/cxl/native.c
index b8b547a162eb..5d4fb9fd84bc 100644
--- a/drivers/misc/cxl/native.c
+++ b/drivers/misc/cxl/native.c
@@ -534,6 +534,15 @@ static int attach_afu_directed(struct cxl_context *ctx, u64 wed, u64 amr)
534 ctx->elem->common.sstp0 = cpu_to_be64(ctx->sstp0); 534 ctx->elem->common.sstp0 = cpu_to_be64(ctx->sstp0);
535 ctx->elem->common.sstp1 = cpu_to_be64(ctx->sstp1); 535 ctx->elem->common.sstp1 = cpu_to_be64(ctx->sstp1);
536 536
537 /*
538 * Ensure we have the multiplexed PSL interrupt set up to take faults
539 * for kernel contexts that may not have allocated any AFU IRQs at all:
540 */
541 if (ctx->irqs.range[0] == 0) {
542 ctx->irqs.offset[0] = ctx->afu->native->psl_hwirq;
543 ctx->irqs.range[0] = 1;
544 }
545
537 for (r = 0; r < CXL_IRQ_RANGES; r++) { 546 for (r = 0; r < CXL_IRQ_RANGES; r++) {
538 ctx->elem->ivte_offsets[r] = cpu_to_be16(ctx->irqs.offset[r]); 547 ctx->elem->ivte_offsets[r] = cpu_to_be16(ctx->irqs.offset[r]);
539 ctx->elem->ivte_ranges[r] = cpu_to_be16(ctx->irqs.range[r]); 548 ctx->elem->ivte_ranges[r] = cpu_to_be16(ctx->irqs.range[r]);