cxl: Fix NULL dereference in cxl_context_init() on PowerVM guests
authorAndrew Donnellan <andrew.donnellan@au1.ibm.com>
Thu, 28 Jul 2016 05:39:41 +0000 (15:39 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 9 Aug 2016 06:52:02 +0000 (16:52 +1000)
commit164793379ad3b7ef5fc5a28260c111358892dff3
tree007c1bc56c5541efe87ee836f86ec58423bc4bef
parentcbd74e1bc8129efb9908f130a8a6e60fd95d2106
cxl: Fix NULL dereference in cxl_context_init() on PowerVM guests

Commit f67a6722d650 ("cxl: Workaround PE=0 hardware limitation in
Mellanox CX4") added a "min_pe" field to struct cxl_service_layer_ops,
to allow us to work around a Mellanox CX-4 hardware limitation.

When allocating the PE number in cxl_context_init(), we read from
ctx->afu->adapter->native->sl_ops->min_pe to get the minimum PE number.
Unsurprisingly, in a PowerVM guest ctx->afu->adapter->native is NULL,
and guests don't have a cxl_service_layer_ops struct anywhere.

Move min_pe from struct cxl_service_layer_ops to struct cxl so it's
accessible in both native and PowerVM environments. For the Mellanox
CX-4, set the min_pe value in set_sl_ops().

Fixes: f67a6722d650 ("cxl: Workaround PE=0 hardware limitation in Mellanox CX4")
Reported-by: Frederic Barrat <fbarrat@linux.vnet.ibm.com>
Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Acked-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>
drivers/misc/cxl/context.c
drivers/misc/cxl/cxl.h
drivers/misc/cxl/pci.c