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>
Subject: [PATCH v9 50/69] bpf: remove VMA linked list
Thread-Topic: [PATCH v9 50/69] bpf: remove VMA linked list
Thread-Index: AQHYX1Q9vQsGKESn1UKSfp/Zbw/+Yw==
Date:   Wed, 4 May 2022 01:14:01 +0000
Message-ID: <20220504011345.662299-35-Liam.Howlett@oracle.com>
References: <20220504002554.654642-1-Liam.Howlett@oracle.com>
 <20220504011345.662299-1-Liam.Howlett@oracle.com>
In-Reply-To: <20220504011345.662299-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?q2eW/EDQ54aGIQL/MAFvQW4ZptUeDC3AAYjtzIQJ36zm96sSh3DOqkqWAL?=
 =?iso-8859-1?Q?Qg3dXUgOhEXMncbDnCSUrVsKArO46pOjSemEJE8U8lXjAUnU2FRGUHB6A1?=
 =?iso-8859-1?Q?msPJ8ZyozaX4XiY4vVBMilQ/pZhipxIjH8yjaCTi5UyLlreSyQ09682ltr?=
 =?iso-8859-1?Q?Zx7hwQhrYzhakeYdfXJZwIATXL/RD393BeNli59p4zQU7W9hoVSbYtJsPS?=
 =?iso-8859-1?Q?ldr2QMMb5/xYCfjyxxPciAzi/+TgSmtkjS9WkinemME+g2N5TL15GE++zB?=
 =?iso-8859-1?Q?tgKqBwNqmNJVd185b8JjI4EKmWcqppY94DJ43xZNjvdQbsmWc+m9r2THBq?=
 =?iso-8859-1?Q?6lOdqpyMmXmg2jI7XiOq1rcWE+FsZyBxDzb9GeOJbRDw6o1KIuKsPA4YtS?=
 =?iso-8859-1?Q?5bCL+ppHmzqFPh6wG/MMNUPKj6/EV4al5eI5qSdeCSxnuTBYA9Kfflj79H?=
 =?iso-8859-1?Q?q6pErpJChTJzb6wILabogovU9X1JNW2G8Mw2C6UGrbqxQixp3b5wF9g9vx?=
 =?iso-8859-1?Q?M5elF8MPwl9Bcp9YWCsLrqNG01RvZDTvyBLlmmwY844Y0zEPDeq6u1v7Uf?=
 =?iso-8859-1?Q?R9YcIBW+uieXBxBvxcHqAJlbzHG4N9jYeWg4LRlYzPio2z1ayaDhb/blYP?=
 =?iso-8859-1?Q?Aynf3LlUaKajbFE7BEL8vdpNoepEehOZJp0hwQTO3DGwPEtxb9g+KjxLYx?=
 =?iso-8859-1?Q?ioB95Rgz6co7gjZn+LVd8IEWvMkuFPujDBBQu2RvCpAfGyWqd/9RuqHvrj?=
 =?iso-8859-1?Q?Oke5mNgdX8GlmQHttELjK3m84xZ96xJ9zGXLqgRGBHHAAdfu+F5+0vmZ/3?=
 =?iso-8859-1?Q?Wvb2CTqBFT+nSLfSszgvJ8wSrcFnbD4QXSbHUxlf8WF9h2iOaYBq3EbdUf?=
 =?iso-8859-1?Q?eW6EyMF6N9xoCqEAxppksC9YYRe6rZHxf2UmOTmZGwzZHfLTbu4Okweu4l?=
 =?iso-8859-1?Q?tPtqb1pwJCrv+ZfIccSyyo3E23ja95ZIr4/hHk5Z7F6Ik2szocMSJ+p6fG?=
 =?iso-8859-1?Q?e5afn/xrX3eoZ0T+4AMAp3Kl/znLSJjlOENobw7uEU2NtAJ+t5xTYerhtl?=
 =?iso-8859-1?Q?w2zJ9V2wG2jzmf8BPXk1t7Ai70GQsBfYfoCXfqzY+VDtgOCsrtueE4Ggjd?=
 =?iso-8859-1?Q?fBoittmSUsvLUJd79XWk7Nfs0/BGI+r921CD9dhgxAd9dYoeNM714aREZs?=
 =?iso-8859-1?Q?HGF0TgJk3U6kZXspZnLDBd7SA65sNlYbqB+8WshMB3Wg2hs2PZ4KrnmNgo?=
 =?iso-8859-1?Q?SfjTV6z8Lj5IGarXTXlq6SQN9fe5ClW997FPDMB5p+1IVeuzeWtg6g9Tzt?=
 =?iso-8859-1?Q?xhczRm/Dfr8rN3cUZqa9916kGvZ0VbnbjR4Ac14Ugg0Kf6tDNCrmaKgt43?=
 =?iso-8859-1?Q?eoZs7iFLPU2nidXVyate5Ly6aqxGq2Q/AtmAoCZSEUuvab+XQR/o2PBw+V?=
 =?iso-8859-1?Q?+pnIQTbmqcyvLy0y1ird49ymDxpK9n/mY+MdxWbunb4xF+0RWOV2eqZBKW?=
 =?iso-8859-1?Q?gJAJUawrEkCEB90MK3xkp5iHKqT+a1N13r6ZRIsOL6mg+rn4hsUQsIS6iv?=
 =?iso-8859-1?Q?dEYpkYSvtpiBxe722ntS+EI2lmfKilB3wGn1oHungrRfoDYxTRfEppO7Nx?=
 =?iso-8859-1?Q?IPCQYP8boGWmTlCUpgC5tx9oA4u+dP8qK1im/mfAp0CICd1xkZS/LGt1DM?=
 =?iso-8859-1?Q?Zewq8fXWiCY27pidwMd8gloWDK7j+y4dyHgVBizyX670hw9+BH5irupvRp?=
 =?iso-8859-1?Q?dzFbQ4SQwoAAWD5oQPz58Rg4lkP2Cw0wgG2mm7qESDagmnAgFMGdlNT0Y0?=
 =?iso-8859-1?Q?d/P1EE50MkE3F6UHwN/qAJB3BDisScc=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: d2a96818-6ba3-44ec-1d12-08da2d6b74a8
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 May 2022 01:14:01.7045
 (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: 4JrVFhBdtlqknHBIAWtg98WDLCkohuVzGP/4TdlErvHlqSUe9Lad90ZtnFpB+YZoDecwZjnQ9owbx91+MN5BvQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR10MB3728
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.486,18.0.858
 definitions=2022-05-03_10:2022-05-02,2022-05-03 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 phishscore=0
 mlxlogscore=999 mlxscore=0 spamscore=0 suspectscore=0 bulkscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2202240000 definitions=main-2205040006
X-Proofpoint-GUID: 5g6BGiViF4zOq5wocdPjTS31ODB1PKpB
X-Proofpoint-ORIG-GUID: 5g6BGiViF4zOq5wocdPjTS31ODB1PKpB
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,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

Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
---
 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 d94696198ef8..9a0bbc808433 100644
--- a/kernel/bpf/task_iter.c
+++ b/kernel/bpf/task_iter.c
@@ -300,8 +300,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
@@ -401,10 +401,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
@@ -418,7 +418,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