Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
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>,
        Yu Zhao <yuzhao@google.com>
Subject: [PATCH v8 36/70] x86: remove vma linked list walks
Thread-Topic: [PATCH v8 36/70] x86: remove vma linked list walks
Thread-Index: AQHYWX88D5AxfxbA7UKZ5+DppX12IQ==
Date:   Tue, 26 Apr 2022 15:06:41 +0000
Message-ID: <20220426150616.3937571-37-Liam.Howlett@oracle.com>
References: <20220426150616.3937571-1-Liam.Howlett@oracle.com>
In-Reply-To: <20220426150616.3937571-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?9dt7W3F+d60fWENef5t0mwFg2u0qBr7rkbjUen/GyoTrBnIS4vp/06e8V/?=
 =?iso-8859-1?Q?j+KyePQiXTySBePsUkOevY1bP42YIONw6JRctZzYDGO/pZD866o7sG3pUJ?=
 =?iso-8859-1?Q?Vs1//BAO1rrO+vKC3ycwRQez2G7iSo4tDAzyJ4JNEI7kzKdcFIJDZnNshl?=
 =?iso-8859-1?Q?c3XFYYs8NI3mXKnjIo5MyFBKjOQygFQo6IwQnu1Hd2PbkPIz4I8LpzwY56?=
 =?iso-8859-1?Q?KSmJug8pxRiWMFi5oKi4Sj2Qs9XCtrez42ACli7aHQk+czGDVamo/geAgM?=
 =?iso-8859-1?Q?TvCONhjjuYPpc9avRrU1Sd4Hx0SPx0tZmxWiqoYagfzY6MwrhGQdXjor5g?=
 =?iso-8859-1?Q?4pppKczT8Y7jr/VAO2jp5q4xW3BgcTzLFOTieGODTBFoa1b0horiB8/BWV?=
 =?iso-8859-1?Q?7Gqu5hR6Y4ATTVDesRZrGwRtREp87KxvhcAcAWi4v8KL6WXGIk5V1W+RKf?=
 =?iso-8859-1?Q?z2bOFcLOwzUx4cCWfls891BttqwWY+TGs6Kgg42O0S8wrZncgw0ibRMO7y?=
 =?iso-8859-1?Q?yKVxWXGokCZBWID/sneDMu+GJvSDXU0KI2Fr5y6r9jjkpg7PqC5diTPVDo?=
 =?iso-8859-1?Q?Pszl5qbutJ63S9RrnRKYTInKABNIpuGsRxtfhbWgAp6uX1jk1jbnjfVFlR?=
 =?iso-8859-1?Q?0GQfOvH6DACsGID5rEup5ToZqEBFE5fC13fz2HM9zyd2dCJrAeHjZ/iNS+?=
 =?iso-8859-1?Q?IRfhBG6qbIcclIrRbFMCbJ2d+PSM4Qa3yy1uvduwBore1y5izFQ6oG+DAC?=
 =?iso-8859-1?Q?c7FfunhHvJghCfUQ08QNqxfMTuqY+fyHjeKnmgS90vZEYFk5wIt7PjxN5j?=
 =?iso-8859-1?Q?AYyvpjgPdZtQjfs/wu3lVVgO+QHwvMQX7eD3x5t4aSTuHbbkKP3JEOTf6x?=
 =?iso-8859-1?Q?maM7jswwG+cOJrdSVLyA84ZLyWDl5uOo9nddEP1f77zJ5QlLfyiaP/WMLa?=
 =?iso-8859-1?Q?RIgVgvuj2Arvo9n6r1XwZhxCLfEuEw0WDoofmwqHh6sg1u229WlWQumbbc?=
 =?iso-8859-1?Q?t5+wGENGfuTQwaXve9PWBEPjESx90bGPoGqndChcjCK5t67MWg3YqhgPsf?=
 =?iso-8859-1?Q?66smDaOYcbQy0X877QSymtI8S6GMm1IRh9pR3SkqHf8mC0SnDsTVWl4YEv?=
 =?iso-8859-1?Q?4H3XuTVYbNr8eC5J8I7TFlbcaB6XwptnVCpfklLmWrrDYHZbi7zvD/L43x?=
 =?iso-8859-1?Q?2GpUVORynHbmWyKuAEDFTZuIKvaZEte02ZfUucwJsfHzJlH+3qjw/bSI32?=
 =?iso-8859-1?Q?BtWd0GcT0QmGpjGpf6M8ouXkRKFGBIk9zALu4jlLfE6hXClnMmtxdUJX/E?=
 =?iso-8859-1?Q?+ejHrGpasIIbzD+9G5ufL6dhQ1PqCozPGTf4hzbMEnc2wyVa3FoPrQYDU7?=
 =?iso-8859-1?Q?u4PZbAwAc2COTCyOWT0ZfmuKyQh4CFs2tX+Z+3UUOMr3maPgdrhGR/ot1a?=
 =?iso-8859-1?Q?/kHT73wHdsdK1M04hpGBvAlugltFvpGFRUW6cBtxwOKjSMuS/WyVbX6Wus?=
 =?iso-8859-1?Q?tuOkaCUF6wrqUR1AjUzJ+8D4F1GjKlY9JDD7XHf87+rksRS0YO6UMX8wTq?=
 =?iso-8859-1?Q?kM3xj0yMpjlmt1rfaK96lNj0uUd96Z482WQYmEesJOzL3nUSpEaI4x5vvq?=
 =?iso-8859-1?Q?yC4NASOSJOCxIsD2SSBC8bnuY58HwfwfXTqiciEy/L2Ky608ofcwURkYnx?=
 =?iso-8859-1?Q?NVC5vLuaOfkgtkD9Rql6dgCPSjncHXaPHMmFQrXJBo0QNNi9LrO1YBG2bF?=
 =?iso-8859-1?Q?2N1JVGFeSqjl9p6Bo8ntaaIg3Z6rm+OmOYdSgRIFWPfOIUZkd7HxPgMAUf?=
 =?iso-8859-1?Q?WaESg6ib14bnntBQNtL8n7wcWDdt6Ik=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: dfb71ea1-ec45-43a9-a564-08da279672b7
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Apr 2022 15:06:41.1616
 (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: 2eMF7O2OyF08w4VGbj8Eo2CRbjlf5cCZS+99U9s0igLZN0ePv76JduHMf+lYlgQSmeV4dpO3u83wqbOb3MhDFg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR10MB2929
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.486,18.0.858
 definitions=2022-04-26_04:2022-04-26,2022-04-26 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 spamscore=0 adultscore=0
 mlxscore=0 bulkscore=0 suspectscore=0 malwarescore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000
 definitions=main-2204260096
X-Proofpoint-GUID: 9nMB0R4m-GvLAC8oKqhtM5ipytFDq6ne
X-Proofpoint-ORIG-GUID: 9nMB0R4m-GvLAC8oKqhtM5ipytFDq6ne
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE autolearn=no
	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: "Matthew Wilcox (Oracle)" <willy@infradead.org>

Use the VMA iterator instead.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
Acked-by: Vlastimil Babka <vbabka@suse.cz>
---
 arch/x86/entry/vdso/vma.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/arch/x86/entry/vdso/vma.c b/arch/x86/entry/vdso/vma.c
index 235a5794296a..3883da001c62 100644
--- a/arch/x86/entry/vdso/vma.c
+++ b/arch/x86/entry/vdso/vma.c
@@ -127,17 +127,17 @@ int vdso_join_timens(struct task_struct *task, struct=
 time_namespace *ns)
 {
 	struct mm_struct *mm =3D task->mm;
 	struct vm_area_struct *vma;
+	VMA_ITERATOR(vmi, mm, 0);
=20
 	mmap_read_lock(mm);
-
-	for (vma =3D mm->mmap; vma; vma =3D vma->vm_next) {
+	for_each_vma(vmi, vma) {
 		unsigned long size =3D vma->vm_end - vma->vm_start;
=20
 		if (vma_is_special_mapping(vma, &vvar_mapping))
 			zap_page_range(vma, vma->vm_start, size);
 	}
-
 	mmap_read_unlock(mm);
+
 	return 0;
 }
 #else
@@ -354,6 +354,7 @@ int map_vdso_once(const struct vdso_image *image, unsig=
ned long addr)
 {
 	struct mm_struct *mm =3D current->mm;
 	struct vm_area_struct *vma;
+	VMA_ITERATOR(vmi, mm, 0);
=20
 	mmap_write_lock(mm);
 	/*
@@ -363,7 +364,7 @@ int map_vdso_once(const struct vdso_image *image, unsig=
ned long addr)
 	 * We could search vma near context.vdso, but it's a slowpath,
 	 * so let's explicitly check all VMAs to be completely sure.
 	 */
-	for (vma =3D mm->mmap; vma; vma =3D vma->vm_next) {
+	for_each_vma(vmi, vma) {
 		if (vma_is_special_mapping(vma, &vdso_mapping) ||
 				vma_is_special_mapping(vma, &vvar_mapping)) {
 			mmap_write_unlock(mm);
--=20
2.35.1