Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Liam Howlett <liam.howlett@oracle.com>
To:     "maple-tree@lists.infradead.org" <maple-tree@lists.infradead.org>,
        "linux-mm@kvack.org" <linux-mm@kvack.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        "damon @ lists . linux . dev" <damon@lists.linux.dev>,
        SeongJae Park <sj@kernel.org>,
        David Hildenbrand <david@redhat.com>
Subject: [PATCH v10 50/69] bpf: remove VMA linked list
Thread-Topic: [PATCH v10 50/69] bpf: remove VMA linked list
Thread-Index: AQHYhbATR62Q/2COW0OT7XksoSBRKw==
Date:   Tue, 21 Jun 2022 20:47:09 +0000
Message-ID: <20220621204632.3370049-51-Liam.Howlett@oracle.com>
References: <20220621204632.3370049-1-Liam.Howlett@oracle.com>
In-Reply-To: <20220621204632.3370049-1-Liam.Howlett@oracle.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?FsMtO7Ni6VqQ0t5TaZbxrhkDcKmZ53ITrCwDX03u0Kzp5Nb4CcINaCnOd1?=
 =?iso-8859-1?Q?QHdYXAWanfoO7dYqFHMgQbBcqRchBkuYG8HC60hXNIlk0a7VQVOIJuajA5?=
 =?iso-8859-1?Q?FNWbGDOX4PBPBcJO7VmDydjCdGQd/6JikfnWaRY0jNxR/PNLPLEWxaVh50?=
 =?iso-8859-1?Q?CJ8bB6SRwLQmB8GG3NdYvEiwvUBHbjE2+GrIL3NembhrXiGcoavp3fWeiB?=
 =?iso-8859-1?Q?TkwpSO60yek/ziaoErLzBA1XdB4JTjyeNaVd1JcWxHu/SW8ut00+ECE8Gv?=
 =?iso-8859-1?Q?wV5/WFA8pWXaeelKhW8qfZz1cf1YcGQm2UTb7WlVifZagnrAppxZp9y4TV?=
 =?iso-8859-1?Q?/OrGyr+H7nkttQxJo/iCPSeDajSiCUqH9A9ZF8DF1VUNhdmOPurnArtNfq?=
 =?iso-8859-1?Q?TC3EPAxDeXYMqkWlChgL2FwR6EBhmmbcfMuXAph5v6rNg/CdZlApML/RJ5?=
 =?iso-8859-1?Q?uMI1TvXOVtv3x7iQwMBSuT3PqJmMw1OCPJYeSC0vLLBMKT3/AIjnTl7kQU?=
 =?iso-8859-1?Q?YAAoH+69l/uWF1i7McG1NsVbSW8EcTOZRo5vxj5Z8MZMrv4FYgQAT4gZfO?=
 =?iso-8859-1?Q?4gxlDLFf3bvOepOUsmaXX4lbnWkb8xEZyVEItyrmf0ZcgN4oyucF9fBLGB?=
 =?iso-8859-1?Q?9pgwmD+ViGE8umQhDTbAFwfIl2CqVU9LaNjNfiPvKPp/zfHoe2ev6WqZ1A?=
 =?iso-8859-1?Q?BWRMS0hvip37/jg7D6lVIK42RuNG+ASVdgWsDujv7pxhnWAFhY99EoDA8e?=
 =?iso-8859-1?Q?dApIqPIsOOw3igKI5cuVhZd7IjSfkKrlglywWPUkQ+6dwh1pEwDf2mRpSN?=
 =?iso-8859-1?Q?yMB/LqNoVLMNGIOVV1Ewek3JcCz/S1C6ySx/kSIVnoB4u24d5ie/xy4VeJ?=
 =?iso-8859-1?Q?sDeC0RzPLsG1sDUO4xLyZ9POk7v+PyNK8kFlevatcNV1hph3LseJV8axRv?=
 =?iso-8859-1?Q?EI5IZK4oDko8g1wKqqnC7mZh5VtIc1jax6lN3u8/0XLAYXz8pI4MKBlBte?=
 =?iso-8859-1?Q?PmxIt/gZhUwRIB/+etbl7HaLNZty5yY68BurWJqemq8t2R8qBZFNRAGiGT?=
 =?iso-8859-1?Q?jvOuqykugv7meFhm4YNLb9wgfOSo7HBjYv8i31IFWrHzRb0rYZtMsJGuBb?=
 =?iso-8859-1?Q?ZVcwqmHDo7BfngyJ4y1ZInmVnuCAD6HA7ntkJqNW7DNt9PYadQ7jNNV7jE?=
 =?iso-8859-1?Q?C2um/SL+Pu8FT5+QtBnFR0HPEXZ+gZI+yxHDclQV9iQzI4GYCp7QbGrobw?=
 =?iso-8859-1?Q?zrUk/ANZt8YVH1Mmvsqms77qqTtKRRh6UF9odM/LuARUSkdsjdosi21wwo?=
 =?iso-8859-1?Q?fpqwD3W0AfuwJ3UNX9mKK4HiDQ5Rni8MghWw9w21Y96zU6JfvMInMYE1QB?=
 =?iso-8859-1?Q?H14iFxOsXtcStsTId/se4dAY2bUlh7HM2IXTg781X6zjW6HeYe9YqT2qpp?=
 =?iso-8859-1?Q?SfusfI5ocmJmLgxGJeqyDbEZUtuJJbD3h4hZiRMpjkw0/lStOMQt5qFL/3?=
 =?iso-8859-1?Q?4LMV3Dy+YcFZF/2NghO39LAk3Sa/b7Q5JDqqQHe6kbpkk7pftyaOJQoRxG?=
 =?iso-8859-1?Q?hqK+umxihSDxdT20kE8LIusFJWodQ/YtJbzjMq8Je/RAULJ+zxwlP/u6cz?=
 =?iso-8859-1?Q?upU+TJxg1miwJGv/4ZaUOq+Xf8Om+oj0epj5kT9wHJFLA3ePhfvXxbGXnz?=
 =?iso-8859-1?Q?DT0MX1/krMgwOwoMT3tl5zvnrGIGAAVkCTPm62IUP+R0hYFZgABKgW4eya?=
 =?iso-8859-1?Q?pmO76Vf5Ls0Ty+vyuB8RS69dhCTDeLQ2gNSIQ5epNLlMfEtFwmFQON7ZLv?=
 =?iso-8859-1?Q?Ev9r/xm1y+s8C38r4KlqP0Mas2NK3Xg=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SN6PR10MB3022.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: da87cd77-2c38-418a-8ffb-08da53c74fb9
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jun 2022 20:47:09.3315
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: JIPgGPFRvdwUPgd/urecATlMX9zkChlp974bh130oaYQEqwjVgMsG6jtaI89T9U2EN5pL5Exe4VGgyo5c7qXgA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR10MB5392
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.517,18.0.883
 definitions=2022-06-21_09:2022-06-21,2022-06-21 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 adultscore=0 bulkscore=0 phishscore=0
 mlxlogscore=999 mlxscore=0 suspectscore=0 spamscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2204290000
 definitions=main-2206210087
X-Proofpoint-ORIG-GUID: 8iXB4D9ZFR_1wKxKyGSPWNkodoj1w5gV
X-Proofpoint-GUID: 8iXB4D9ZFR_1wKxKyGSPWNkodoj1w5gV
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: "Liam R. Howlett" <Liam.Howlett@Oracle.com>

Use vma_next() and remove reference to the start of the linked list

Link: https://lkml.kernel.org/r/20220504011345.662299-35-Liam.Howlett@oracl=
e.com
Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: David Howells <dhowells@redhat.com>
Cc: "Matthew Wilcox (Oracle)" <willy@infradead.org>
Cc: SeongJae Park <sj@kernel.org>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: Will Deacon <will@kernel.org>
Cc: Davidlohr Bueso <dave@stgolabs.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
 kernel/bpf/task_iter.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/kernel/bpf/task_iter.c b/kernel/bpf/task_iter.c
index 8c921799def4..1c8debd42dc9 100644
--- a/kernel/bpf/task_iter.c
+++ b/kernel/bpf/task_iter.c
@@ -299,8 +299,8 @@ struct bpf_iter_seq_task_vma_info {
 };
=20
 enum bpf_task_vma_iter_find_op {
-	task_vma_iter_first_vma,   /* use mm->mmap */
-	task_vma_iter_next_vma,    /* use curr_vma->vm_next */
+	task_vma_iter_first_vma,   /* use find_vma() with addr 0 */
+	task_vma_iter_next_vma,    /* use vma_next() with curr_vma */
 	task_vma_iter_find_vma,    /* use find_vma() to find next vma */
 };
=20
@@ -400,10 +400,10 @@ task_vma_seq_get_next(struct bpf_iter_seq_task_vma_in=
fo *info)
=20
 	switch (op) {
 	case task_vma_iter_first_vma:
-		curr_vma =3D curr_task->mm->mmap;
+		curr_vma =3D find_vma(curr_task->mm, 0);
 		break;
 	case task_vma_iter_next_vma:
-		curr_vma =3D curr_vma->vm_next;
+		curr_vma =3D find_vma(curr_task->mm, curr_vma->vm_end);
 		break;
 	case task_vma_iter_find_vma:
 		/* We dropped mmap_lock so it is necessary to use find_vma
@@ -417,7 +417,7 @@ task_vma_seq_get_next(struct bpf_iter_seq_task_vma_info=
 *info)
 		if (curr_vma &&
 		    curr_vma->vm_start =3D=3D info->prev_vm_start &&
 		    curr_vma->vm_end =3D=3D info->prev_vm_end)
-			curr_vma =3D curr_vma->vm_next;
+			curr_vma =3D find_vma(curr_task->mm, curr_vma->vm_end);
 		break;
 	}
 	if (!curr_vma) {
--=20
2.35.1