[PATCH] cxl: Test the correct mmio space before unmapping

Michael Neuling mikey at neuling.org
Fri Jul 3 10:23:16 AEST 2015


On Thu, 2015-07-02 at 15:55 +1000, Daniel Axtens wrote:
> Before freeing p2n, test p2n, not p1n.
> 
> Signed-off-by: Daniel Axtens <dja at axtens.net>

Nice catch but can you resend with a CC stable?

Mikey

> 
> ---
> 
> While a potentially nasty bug, this is only hit (at the moment)
> in cxl_remove, so it's probably not a candidate for stable.
> ---
>  drivers/misc/cxl/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c
> index a16988d..fd43462 100644
> --- a/drivers/misc/cxl/pci.c
> +++ b/drivers/misc/cxl/pci.c
> @@ -540,7 +540,7 @@ err:
>  
>  static void cxl_unmap_slice_regs(struct cxl_afu *afu)
>  {
> -	if (afu->p1n_mmio)
> +	if (afu->p2n_mmio)
>  		iounmap(afu->p2n_mmio);
>  	if (afu->p1n_mmio)
>  		iounmap(afu->p1n_mmio);



More information about the Linuxppc-dev mailing list