ACK: [PATCH] [Zesty] SAUCE: powernv/npu-dma.c: Fix opal_npu_destroy_context call

[Thadeu Lima de Souza Cascardo]

On Mon, May 22, 2017 at 02:06:00PM -0300, Breno Leitao wrote:
> From: Alistair Popple <alistair at popple.id.au>
> 
> BugLink: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1692580
> 
> opal_npu_destroy_context() should be called with the NPU PHB, not the
> PCIe PHB.
> 
> (This patch depends on the following patchset: "[PATCH 1/4] [Zesty]
> powerpc/powernv: Require MMU_NOTIFIER to fix NPU build")
> 
> 
> Signed-off-by: Alistair Popple <alistair at popple.id.au>
> 
> Upstream submission: http://patchwork.ozlabs.org/patch/764857/
> Signed-off-by: Breno Leitao <breno.leitao at gmail.com>
> ---
>  arch/powerpc/platforms/powernv/npu-dma.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/powerpc/platforms/powernv/npu-dma.c b/arch/powerpc/platforms/powernv/npu-dma.c
> index 27a2520c99ef..3184c9e265fb 100644
> --- a/arch/powerpc/platforms/powernv/npu-dma.c
> +++ b/arch/powerpc/platforms/powernv/npu-dma.c
> @@ -714,7 +714,7 @@ static void pnv_npu2_release_context(struct kref *kref)
>  void pnv_npu2_destroy_context(struct npu_context *npu_context,
>  			struct pci_dev *gpdev)
>  {
> -	struct pnv_phb *nphb, *phb;
> +	struct pnv_phb *nphb;
>  	struct npu *npu;
>  	struct pci_dev *npdev = pnv_pci_get_npu_dev(gpdev, 0);
>  	struct device_node *nvlink_dn;
> @@ -728,13 +728,12 @@ void pnv_npu2_destroy_context(struct npu_context *npu_context,
>  
>  	nphb = pci_bus_to_host(npdev->bus)->private_data;
>  	npu = &nphb->npu;
> -	phb = pci_bus_to_host(gpdev->bus)->private_data;
>  	nvlink_dn = of_parse_phandle(npdev->dev.of_node, "ibm,nvlink", 0);
>  	if (WARN_ON(of_property_read_u32(nvlink_dn, "ibm,npu-link-index",
>  							&nvlink_index)))
>  		return;
>  	npu_context->npdev[npu->index][nvlink_index] = NULL;
> -	opal_npu_destroy_context(phb->opal_id, npu_context->mm->context.id,
> +	opal_npu_destroy_context(nphb->opal_id, npu_context->mm->context.id,
>  				PCI_DEVID(gpdev->bus->number, gpdev->devfn));
>  	kref_put(&npu_context->kref, pnv_npu2_release_context);
>  }
> -- 
> 2.11.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Reasonable fix, affects only powernv, already applied upstream, fixes
the other ACKed SRU.

Acked-by: Thadeu Lima de Souza Cascardo <cascardo at canonical.com>