I don’t know, but it is obvious issue. Later I’ll run test with the patch.
From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of zhoucm1 Sent: Wednesday, August 16, 2017 5:10 PM To: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 1/1] drm/ttm: Fix accounting error when fail to get pages for pool Fix memory leak of ttm kernel issue? On 2017年08月16日 15:55, Christian König wrote: CC stable? Reviewed-by: Christian König <christian.koe...@amd.com><mailto:christian.koe...@amd.com> Am 16.08.2017 um 09:25 schrieb Liu, Monk: Reviewed-by: Monk Liu <monk....@amd.com><mailto:monk....@amd.com> ________________________________ From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org><mailto:amd-gfx-boun...@lists.freedesktop.org> on behalf of Xiangliang.Yu <xiangliang...@amd.com><mailto:xiangliang...@amd.com> Sent: Wednesday, August 16, 2017 3:20:46 PM To: a...@linux-foundation.org<mailto:a...@linux-foundation.org>; labb...@redhat.com<mailto:labb...@redhat.com>; dri-de...@lists.freedesktop.org<mailto:dri-de...@lists.freedesktop.org>; amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> Cc: Yu, Xiangliang Subject: [PATCH 1/1] drm/ttm: Fix accounting error when fail to get pages for pool When fail to get needed page for pool, need to put allocated pages into pool. But current code has a miscalculation of allocated pages, correct it. Signed-off-by: Xiangliang.Yu <xiangliang...@amd.com><mailto:xiangliang...@amd.com> --- drivers/gpu/drm/ttm/ttm_page_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/ttm/ttm_page_alloc.c b/drivers/gpu/drm/ttm/ttm_page_alloc.c index eeddc1e..8715998 100644 --- a/drivers/gpu/drm/ttm/ttm_page_alloc.c +++ b/drivers/gpu/drm/ttm/ttm_page_alloc.c @@ -615,7 +615,7 @@ static void ttm_page_pool_fill_locked(struct ttm_page_pool *pool, } else { pr_err("Failed to fill pool (%p)\n", pool); /* If we have any pages left put them to the pool. */ - list_for_each_entry(p, &pool->list, lru) { + list_for_each_entry(p, &new_pages, lru) { ++cpages; } list_splice(&new_pages, &pool->list); -- 2.7.4 _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org<mailto: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
