From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id DAE77C83F1B for ; Mon, 14 Jul 2025 05:23:34 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7D1AB6B0096; Mon, 14 Jul 2025 01:23:34 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 781FB6B0098; Mon, 14 Jul 2025 01:23:34 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 698286B0099; Mon, 14 Jul 2025 01:23:34 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 5AE996B0096 for ; Mon, 14 Jul 2025 01:23:34 -0400 (EDT) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id D0416804F8 for ; Mon, 14 Jul 2025 05:23:33 +0000 (UTC) X-FDA: 83661727506.05.ACE9BE6 Received: from us-smtp-delivery-44.mimecast.com (us-smtp-delivery-44.mimecast.com [205.139.111.44]) by imf15.hostedemail.com (Postfix) with ESMTP id 07966A0004 for ; Mon, 14 Jul 2025 05:23:31 +0000 (UTC) Authentication-Results: imf15.hostedemail.com; dkim=none; spf=softfail (imf15.hostedemail.com: 205.139.111.44 is neither permitted nor denied by domain of airlied@gmail.com) smtp.mailfrom=airlied@gmail.com; dmarc=fail reason="No valid SPF, No valid DKIM" header.from=gmail.com (policy=none) ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1752470612; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/I6dYyMhwqMYYfKI4O64P9xmeKWdi2tmt/J92aqsMHs=; b=zazPI1o04hW5OxbhDK/Zv2c7BsSXhYBYKZ32GXxdaq5pbGvqfENu3P0faD9lqjWRPlld09 x010KWlox56edeWO+IGgayzH75Njr1U6sBQauJhXOKm5ESdmmrFWD1JM/62Ixr5r13iyVf kAKIrET7W3Flb2BuSlLVDjr5uqC5wws= ARC-Authentication-Results: i=1; imf15.hostedemail.com; dkim=none; spf=softfail (imf15.hostedemail.com: 205.139.111.44 is neither permitted nor denied by domain of airlied@gmail.com) smtp.mailfrom=airlied@gmail.com; dmarc=fail reason="No valid SPF, No valid DKIM" header.from=gmail.com (policy=none) ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1752470612; a=rsa-sha256; cv=none; b=C0asHQ+UqXSLO3yMfQoFYremwiZcDhih50zo8BjaT7Ba9Y9WQ4Qim7GFY+XDtlaw0fkOEs Dx3HIJqA7ozyZI+OBK/ULde5akL2Cpuh9BSeTkqNDJWRZ7wn8OR5j/KaBPyf9AyHxzXnNv E21lHVUVX2ue+srEAUXeRnDpLTfDWTw= Received: from mx-prod-mc-02.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-332-LXDEIfQnOtW5wME2WDqsQQ-1; Mon, 14 Jul 2025 01:23:25 -0400 X-MC-Unique: LXDEIfQnOtW5wME2WDqsQQ-1 X-Mimecast-MFC-AGG-ID: LXDEIfQnOtW5wME2WDqsQQ_1752470604 Received: from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.17]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-02.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 0E6FB1956089; Mon, 14 Jul 2025 05:23:24 +0000 (UTC) Received: from dreadlord.redhat.com (unknown [10.67.32.31]) by mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 673FE1977000; Mon, 14 Jul 2025 05:23:18 +0000 (UTC) From: Dave Airlie To: dri-devel@lists.freedesktop.org, linux-mm@kvack.org, Johannes Weiner , Christian Koenig Cc: Dave Chinner , Kairui Song , Dave Airlie Subject: [PATCH 04/18] ttm/pool: port to list_lru. (v2) Date: Mon, 14 Jul 2025 15:18:19 +1000 Message-ID: <20250714052243.1149732-5-airlied@gmail.com> In-Reply-To: <20250714052243.1149732-1-airlied@gmail.com> References: <20250714052243.1149732-1-airlied@gmail.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.0 on 10.30.177.17 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: kzaUQL0ErLES-DCFLK_wXjTwF9WnaFvkBOmEKV7VM1M_1752470604 X-Mimecast-Originator: gmail.com Content-Transfer-Encoding: quoted-printable content-type: text/plain; charset=WINDOWS-1252; x-default=true X-Stat-Signature: 8dhfpruf896ioqam5q79pbfx8wnk1swu X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 07966A0004 X-Rspam-User: X-HE-Tag: 1752470611-435821 X-HE-Meta: U2FsdGVkX1+X3GWJbdiS414Q5ZFPHxegOSVweZuYDXmi+h6W9+sdadFOlLqkkWL5qz8iF8ooNOr1sZQORKhtoR6rflj7gslZMPKSWXukzWl61N+r+jCWBcEaWZacnohw4dyZUZU2KmsXBhHKJWvx1tMSzIvxrL597lrHtq+pOrotXi+YLf3FuxPrkChTiNRUQp+U8+0KqVjP8+nTAettV0owqeL8CL42WT+XKhp+EIX/toazOTu1I29Y8zROF5eUbXM7kpZEDSLGLrr7f9M9ZGoWwjS72jVCPgB0eCMhfEeHUMWAOlbbnrKcjrLo2sDoZq6M9Kj2ggcKsb8zDaTC4sOcto/5zKhRR4Jfk0d5QMoitL0+u91s4Iisx7UhRny0qTzRLgHid7cL5i2nDXeY6HrKbPMBDbRRkfmT6KuNcprb5/CaDjo3UegRLIkfqqyUmZgIrVuCeblFVGF2XKPXQsx1kZX+zE5IDa5T58Sy5nGVzN0nNoLutZalpKAB+g7Jvf1lauHYVjDq4rUt7XqnSFYuHOqcmsVHqkGGdu6VPiD25NZ+1566m4thVfOttjRWNFMPZh/waIkXjS2aPZHTy9P+SEV+HPnIJl2I9ieoK6wpP6b2NKf03jqNoObsmmLK5W5bkJH3iZ48MRpkyIfoVF6ZU2BwhfDz7u/YAIvpmAzTBRqF06nytM/7pkKAU7hXzggZzEyxr5Wiy09L/mgO7Ujmy+WuFR88y1kvUICBeFQKQDnaUa+h9XrIECByhehbsaZur8jMyOGizOhMc2wHGafBnDmERV3uwpXF8YL9UbbZt/ABCPuSXrEUWQGBkuDQvJccA/AEL/i46UiPVEtu3PgIVBzW5Ej5vZN7V5/Lxo++uCKYwHcnLjkOITbbrNehRjrdodNdP9f8A82GAckVgfsJRnG48n7ZlnugRuFwNBMx52V9N6xE5FBQ7oxJNqzXj1Lc2mp4B7HyzLn8eYo lJTSM1Ps 04lwShlzcLdv2ZLoLcV8u6whrAg5bide1Kd/lszGCiEN1SxvsqrXiMRPm6eB2/PgITu5gN2Z+5dPfPenVJ0D2XBOsrR+TRpgO8B5E1Mj1AXmL7uZLYCvmf055pJIehfONTGv26T5ecVZ88OvV4dne7/ZKv1apTZGv+tBAj8zX4rlU5eUHrAKm3jhjODC3lpIc5mSzhRCqgMinXYu+9nlRlTKfVs0D2NcXgAkrMhQcwqUSB2fdmpDqulmAgYIkMfBc0H66XvP1PpwWxV53EMTUwQbFZZWoAOlVkL6rvXxIcbWi/Gyiz2F9xsXUtDDGrD8KmM+2IbhcqcRYRsYzc5AG2g9NDygmOuuxJ01aEwzHzqBPdldA81oDmtINzVLOlbTX27WcylIPIJ78ZwMCfWNYhl4Hq+JjbSCa1mZ3KMmFcgc4ZExjUqorF833N86x8ZUcnqjysVmCCrILGQfR6shDuRzFj2VO7IcgbzO9wq1bNLM2Yt1IcGrwJx1QlXcGFiy4HkgGjDLmXXpT52XKO1bJyl+7Mz1FaGnLqU7z/yFE/CxHgSDrpeji/9I5Z3MbPX4I8KWjYw3a0G9FhQrS3gTV1s4e4SawK9apE2wEV1b5TzucNVk= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: Dave Airlie This is an initial port of the TTM pools for write combined and uncached pages to use the list_lru. This makes the pool's more NUMA aware and avoids needing separate NUMA pools (later commit enables this). Cc: Christian Koenig Cc: Johannes Weiner Cc: Dave Chinner Signed-off-by: Dave Airlie --- v2: drop the pt->lock, lru list has it's own lock which is sufficent. rearrange list isolates to fix bad locking orders. --- drivers/gpu/drm/ttm/tests/ttm_device_test.c | 2 +- drivers/gpu/drm/ttm/tests/ttm_pool_test.c | 32 ++++---- drivers/gpu/drm/ttm/ttm_pool.c | 91 ++++++++++++++------- include/drm/ttm/ttm_pool.h | 6 +- 4 files changed, 80 insertions(+), 51 deletions(-) diff --git a/drivers/gpu/drm/ttm/tests/ttm_device_test.c b/drivers/gpu/drm/= ttm/tests/ttm_device_test.c index 1621903818e5..1f207fd222bc 100644 --- a/drivers/gpu/drm/ttm/tests/ttm_device_test.c +++ b/drivers/gpu/drm/ttm/tests/ttm_device_test.c @@ -183,7 +183,7 @@ static void ttm_device_init_pools(struct kunit *test) =20 =09=09=09=09if (params->use_dma_alloc) =09=09=09=09=09KUNIT_ASSERT_FALSE(test, -=09=09=09=09=09=09=09 list_empty(&pt.pages)); +=09=09=09=09=09=09=09 !list_lru_count(&pt.pages)); =09=09=09} =09=09} =09} diff --git a/drivers/gpu/drm/ttm/tests/ttm_pool_test.c b/drivers/gpu/drm/tt= m/tests/ttm_pool_test.c index 8ade53371f72..39234a3e98c4 100644 --- a/drivers/gpu/drm/ttm/tests/ttm_pool_test.c +++ b/drivers/gpu/drm/ttm/tests/ttm_pool_test.c @@ -248,7 +248,7 @@ static void ttm_pool_alloc_order_caching_match(struct k= unit *test) =09pool =3D ttm_pool_pre_populated(test, size, caching); =20 =09pt =3D &pool->caching[caching].orders[order]; -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt->pages)); =20 =09tt =3D ttm_tt_kunit_init(test, 0, caching, size); =09KUNIT_ASSERT_NOT_NULL(test, tt); @@ -256,7 +256,7 @@ static void ttm_pool_alloc_order_caching_match(struct k= unit *test) =09err =3D ttm_pool_alloc(pool, tt, &simple_ctx); =09KUNIT_ASSERT_EQ(test, err, 0); =20 -=09KUNIT_ASSERT_TRUE(test, list_empty(&pt->pages)); +=09KUNIT_ASSERT_TRUE(test, !list_lru_count(&pt->pages)); =20 =09ttm_pool_free(pool, tt); =09ttm_tt_fini(tt); @@ -282,8 +282,8 @@ static void ttm_pool_alloc_caching_mismatch(struct kuni= t *test) =09tt =3D ttm_tt_kunit_init(test, 0, tt_caching, size); =09KUNIT_ASSERT_NOT_NULL(test, tt); =20 -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt_pool->pages)); -=09KUNIT_ASSERT_TRUE(test, list_empty(&pt_tt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt_pool->pages)); +=09KUNIT_ASSERT_TRUE(test, !list_lru_count(&pt_tt->pages)); =20 =09err =3D ttm_pool_alloc(pool, tt, &simple_ctx); =09KUNIT_ASSERT_EQ(test, err, 0); @@ -291,8 +291,8 @@ static void ttm_pool_alloc_caching_mismatch(struct kuni= t *test) =09ttm_pool_free(pool, tt); =09ttm_tt_fini(tt); =20 -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt_pool->pages)); -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt_tt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt_pool->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt_tt->pages)); =20 =09ttm_pool_fini(pool); } @@ -316,8 +316,8 @@ static void ttm_pool_alloc_order_mismatch(struct kunit = *test) =09tt =3D ttm_tt_kunit_init(test, 0, caching, snd_size); =09KUNIT_ASSERT_NOT_NULL(test, tt); =20 -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt_pool->pages)); -=09KUNIT_ASSERT_TRUE(test, list_empty(&pt_tt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt_pool->pages)); +=09KUNIT_ASSERT_TRUE(test, !list_lru_count(&pt_tt->pages)); =20 =09err =3D ttm_pool_alloc(pool, tt, &simple_ctx); =09KUNIT_ASSERT_EQ(test, err, 0); @@ -325,8 +325,8 @@ static void ttm_pool_alloc_order_mismatch(struct kunit = *test) =09ttm_pool_free(pool, tt); =09ttm_tt_fini(tt); =20 -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt_pool->pages)); -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt_tt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt_pool->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt_tt->pages)); =20 =09ttm_pool_fini(pool); } @@ -352,12 +352,12 @@ static void ttm_pool_free_dma_alloc(struct kunit *tes= t) =09ttm_pool_alloc(pool, tt, &simple_ctx); =20 =09pt =3D &pool->caching[caching].orders[order]; -=09KUNIT_ASSERT_TRUE(test, list_empty(&pt->pages)); +=09KUNIT_ASSERT_TRUE(test, !list_lru_count(&pt->pages)); =20 =09ttm_pool_free(pool, tt); =09ttm_tt_fini(tt); =20 -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt->pages)); =20 =09ttm_pool_fini(pool); } @@ -383,12 +383,12 @@ static void ttm_pool_free_no_dma_alloc(struct kunit *= test) =09ttm_pool_alloc(pool, tt, &simple_ctx); =20 =09pt =3D &pool->caching[caching].orders[order]; -=09KUNIT_ASSERT_TRUE(test, list_is_singular(&pt->pages)); +=09KUNIT_ASSERT_TRUE(test, list_lru_count(&pt->pages) =3D=3D 1); =20 =09ttm_pool_free(pool, tt); =09ttm_tt_fini(tt); =20 -=09KUNIT_ASSERT_TRUE(test, list_is_singular(&pt->pages)); +=09KUNIT_ASSERT_TRUE(test, list_lru_count(&pt->pages) =3D=3D 1); =20 =09ttm_pool_fini(pool); } @@ -404,11 +404,11 @@ static void ttm_pool_fini_basic(struct kunit *test) =09pool =3D ttm_pool_pre_populated(test, size, caching); =09pt =3D &pool->caching[caching].orders[order]; =20 -=09KUNIT_ASSERT_FALSE(test, list_empty(&pt->pages)); +=09KUNIT_ASSERT_FALSE(test, !list_lru_count(&pt->pages)); =20 =09ttm_pool_fini(pool); =20 -=09KUNIT_ASSERT_TRUE(test, list_empty(&pt->pages)); +=09KUNIT_ASSERT_TRUE(test, !list_lru_count(&pt->pages)); } =20 static struct kunit_case ttm_pool_test_cases[] =3D { diff --git a/drivers/gpu/drm/ttm/ttm_pool.c b/drivers/gpu/drm/ttm/ttm_pool.= c index ee2344089d47..df6b81a43893 100644 --- a/drivers/gpu/drm/ttm/ttm_pool.c +++ b/drivers/gpu/drm/ttm/ttm_pool.c @@ -131,6 +131,15 @@ static struct list_head shrinker_list; static struct shrinker *mm_shrinker; static DECLARE_RWSEM(pool_shrink_rwsem); =20 +static int ttm_pool_nid(struct ttm_pool *pool) { +=09int nid =3D NUMA_NO_NODE; +=09if (pool) +=09=09nid =3D pool->nid; +=09if (nid =3D=3D NUMA_NO_NODE) +=09=09nid =3D numa_node_id(); +=09return nid; +} + /* Allocate pages of size 1 << order with the given gfp_flags */ static struct page *ttm_pool_alloc_page(struct ttm_pool *pool, gfp_t gfp_f= lags, =09=09=09=09=09unsigned int order) @@ -290,32 +299,41 @@ static void ttm_pool_type_give(struct ttm_pool_type *= pt, struct page *p) =09=09=09clear_page(page_address(p + i)); =09} =20 -=09spin_lock(&pt->lock); -=09list_add(&p->lru, &pt->pages); -=09spin_unlock(&pt->lock); +=09INIT_LIST_HEAD(&p->lru); +=09rcu_read_lock(); +=09list_lru_add(&pt->pages, &p->lru, nid, NULL); +=09rcu_read_unlock(); =09atomic_long_add(1 << pt->order, &allocated_pages); =20 =09mod_node_page_state(NODE_DATA(nid), NR_GPU_ACTIVE, -num_pages); =09mod_node_page_state(NODE_DATA(nid), NR_GPU_RECLAIM, num_pages); } =20 +static enum lru_status take_one_from_lru(struct list_head *item, +=09=09=09=09=09 struct list_lru_one *list, +=09=09=09=09=09 void *cb_arg) +{ +=09struct page **out_page =3D cb_arg; +=09struct page *p =3D container_of(item, struct page, lru); +=09list_lru_isolate(list, item); + +=09*out_page =3D p; +=09return LRU_REMOVED; +} + /* Take pages from a specific pool_type, return NULL when nothing availabl= e */ -static struct page *ttm_pool_type_take(struct ttm_pool_type *pt) +static struct page *ttm_pool_type_take(struct ttm_pool_type *pt, int nid) { -=09struct page *p; -=09int nid; +=09int ret; +=09struct page *p =3D NULL; +=09unsigned long nr_to_walk =3D 1; =20 -=09spin_lock(&pt->lock); -=09p =3D list_first_entry_or_null(&pt->pages, typeof(*p), lru); -=09if (p) { -=09=09nid =3D page_to_nid(p); +=09ret =3D list_lru_walk_node(&pt->pages, nid, take_one_from_lru, (void *)= &p, &nr_to_walk); +=09if (ret =3D=3D 1 && p) { =09=09atomic_long_sub(1 << pt->order, &allocated_pages); =09=09mod_node_page_state(NODE_DATA(nid), NR_GPU_ACTIVE, (1 << pt->order))= ; =09=09mod_node_page_state(NODE_DATA(nid), NR_GPU_RECLAIM, -(1 << pt->order= )); -=09=09list_del(&p->lru); =09} -=09spin_unlock(&pt->lock); - =09return p; } =20 @@ -326,25 +344,47 @@ static void ttm_pool_type_init(struct ttm_pool_type *= pt, struct ttm_pool *pool, =09pt->pool =3D pool; =09pt->caching =3D caching; =09pt->order =3D order; -=09spin_lock_init(&pt->lock); -=09INIT_LIST_HEAD(&pt->pages); +=09list_lru_init(&pt->pages); =20 =09spin_lock(&shrinker_lock); =09list_add_tail(&pt->shrinker_list, &shrinker_list); =09spin_unlock(&shrinker_lock); } =20 +static enum lru_status pool_move_to_dispose_list(struct list_head *item, +=09=09=09=09=09=09 struct list_lru_one *list, +=09=09=09=09=09=09 void *cb_arg) +{ +=09struct list_head *dispose =3D cb_arg; + +=09list_lru_isolate_move(list, item, dispose); + +=09return LRU_REMOVED; +} + +static void ttm_pool_dispose_list(struct ttm_pool_type *pt, +=09=09=09=09 struct list_head *dispose) +{ +=09while (!list_empty(dispose)) { +=09=09struct page *p; +=09=09p =3D list_first_entry(dispose, struct page, lru); +=09=09list_del_init(&p->lru); +=09=09atomic_long_sub(1 << pt->order, &allocated_pages); +=09=09ttm_pool_free_page(pt->pool, pt->caching, pt->order, p, true); +=09} +} + /* Remove a pool_type from the global shrinker list and free all pages */ static void ttm_pool_type_fini(struct ttm_pool_type *pt) { -=09struct page *p; +=09LIST_HEAD(dispose); =20 =09spin_lock(&shrinker_lock); =09list_del(&pt->shrinker_list); =09spin_unlock(&shrinker_lock); =20 -=09while ((p =3D ttm_pool_type_take(pt))) -=09=09ttm_pool_free_page(pt->pool, pt->caching, pt->order, p, true); +=09list_lru_walk(&pt->pages, pool_move_to_dispose_list, &dispose, LONG_MAX= ); +=09ttm_pool_dispose_list(pt, &dispose); } =20 /* Return the pool_type to use for the given caching and order */ @@ -394,7 +434,7 @@ static unsigned int ttm_pool_shrink(void) =09list_move_tail(&pt->shrinker_list, &shrinker_list); =09spin_unlock(&shrinker_lock); =20 -=09p =3D ttm_pool_type_take(pt); +=09p =3D ttm_pool_type_take(pt, ttm_pool_nid(pt->pool)); =09if (p) { =09=09ttm_pool_free_page(pt->pool, pt->caching, pt->order, p, true); =09=09num_pages =3D 1 << pt->order; @@ -748,7 +788,7 @@ static int __ttm_pool_alloc(struct ttm_pool *pool, stru= ct ttm_tt *tt, =09=09p =3D NULL; =09=09pt =3D ttm_pool_select_type(pool, page_caching, order); =09=09if (pt && allow_pools) -=09=09=09p =3D ttm_pool_type_take(pt); +=09=09=09p =3D ttm_pool_type_take(pt, ttm_pool_nid(pool)); =09=09/* =09=09 * If that fails or previously failed, allocate from system. =09=09 * Note that this also disallows additional pool allocations using @@ -1177,16 +1217,7 @@ static unsigned long ttm_pool_shrinker_count(struct = shrinker *shrink, /* Count the number of pages available in a pool_type */ static unsigned int ttm_pool_type_count(struct ttm_pool_type *pt) { -=09unsigned int count =3D 0; -=09struct page *p; - -=09spin_lock(&pt->lock); -=09/* Only used for debugfs, the overhead doesn't matter */ -=09list_for_each_entry(p, &pt->pages, lru) -=09=09++count; -=09spin_unlock(&pt->lock); - -=09return count; +=09return list_lru_count(&pt->pages); } =20 /* Print a nice header for the order */ diff --git a/include/drm/ttm/ttm_pool.h b/include/drm/ttm/ttm_pool.h index 54cd34a6e4c0..df56527c4853 100644 --- a/include/drm/ttm/ttm_pool.h +++ b/include/drm/ttm/ttm_pool.h @@ -45,8 +45,7 @@ struct ttm_tt; * @order: the allocation order our pages have * @caching: the caching type our pages have * @shrinker_list: our place on the global shrinker list - * @lock: protection of the page list - * @pages: the list of pages in the pool + * @pages: the lru_list of pages in the pool */ struct ttm_pool_type { =09struct ttm_pool *pool; @@ -55,8 +54,7 @@ struct ttm_pool_type { =20 =09struct list_head shrinker_list; =20 -=09spinlock_t lock; -=09struct list_head pages; +=09struct list_lru pages; }; =20 /** --=20 2.49.0