On Thu, Jul 12, 2018 at 5:24 PM, Felix Kuehling <felix.kuehl...@amd.com> wrote:
> From: Yong Zhao <yong.z...@amd.com>
>
> On Raven Invalid PPRs can be reported because multiple PPRs can be
> still queued when memory is freed. Apply a rate limit to avoid
> flooding the log in this case.
>
> Signed-off-by: Yong Zhao <yong.z...@amd.com>
> Reviewed-by: Felix Kuehling <felix.kuehl...@amd.com>
> Signed-off-by: Felix Kuehling <felix.kuehl...@amd.com>

Maybe define PPRs in the description.  With that fixed:
Acked-by: Alex Deucher <alexander.deuc...@amd.com>

> ---
>  drivers/gpu/drm/amd/amdkfd/kfd_iommu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c 
> b/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c
> index c718179..7a61f38 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c
> @@ -190,7 +190,7 @@ static int iommu_invalid_ppr_cb(struct pci_dev *pdev, int 
> pasid,
>  {
>         struct kfd_dev *dev;
>
> -       dev_warn(kfd_device,
> +       dev_warn_ratelimited(kfd_device,
>                         "Invalid PPR device %x:%x.%x pasid %d address 0x%lX 
> flags 0x%X",
>                         PCI_BUS_NUM(pdev->devfn),
>                         PCI_SLOT(pdev->devfn),
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to