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 v5 35/70] x86: Remove vma linked list walks
Thread-Topic: [PATCH v5 35/70] x86: Remove vma linked list walks
Thread-Index: AQHYF957fXQogRVgFkCVx3m7VRYj8w==
Date:   Wed, 2 Feb 2022 02:42:13 +0000
Message-ID: <20220202024137.2516438-36-Liam.Howlett@oracle.com>
References: <20220202024137.2516438-1-Liam.Howlett@oracle.com>
In-Reply-To: <20220202024137.2516438-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?NwGlIs3UxQD/CO0+yfSjYu8GMjVl7DsdhW2ZxCOLgjn3h9Difl1t6NTIgM?=
 =?iso-8859-1?Q?hHyK4HhEcgiBmOBADCmNG6LxaL2l6anuaFYpeKVTznml4yaJqjL1/T/Z7J?=
 =?iso-8859-1?Q?76o+hKJl+e3gj+Yt8ZyewpjJHNtChFHppm0upt0mvJNxPDIh/1zpJuABct?=
 =?iso-8859-1?Q?XCW4c+k9MEJNXoJ96flYAt+yPIdF9NPQ96mLHPAygirs6xgbrWIoRaKyI7?=
 =?iso-8859-1?Q?8+FIE36xyEi+y9gzUPSXPgD++BBOMc+UONti1wVr1vg7zlf2p6pAzqnlaq?=
 =?iso-8859-1?Q?3AMn915+n79OLLo6rYwkIW3uPsWJQTdeHRtDRa5nUKuafjqUw/vs4OEPob?=
 =?iso-8859-1?Q?XODvBmzjM2Qu0QH0giXA5Ycs3Lx2lqBDjUl/qg14KKOmHcwIq8SwwMcq0t?=
 =?iso-8859-1?Q?2VJi+oDLyco8r0q0Ti5e1WHedCdjjlwYzfv/jee5wqmk2lhi1VyYVgQspG?=
 =?iso-8859-1?Q?MoSX1CXlfUSIsPDF26+dPTw3qy9eZPd5oIrAeYeshRSWO3DWvhinBlpqO3?=
 =?iso-8859-1?Q?S8wz46OqDAvs338Xum+X+lw3bM2KeYcGUlaNllooH5Q8CHWAik4oA1SLBR?=
 =?iso-8859-1?Q?hqtyVnIK5XfrvMLXRZZON1RMwhEOoah1oT93GI27YGoBleALSrPUyZuw2x?=
 =?iso-8859-1?Q?RR0eqyAaZk8+Hn9rmt4/bbRrGnojkvanoRXID54JMFriqftTcXW4hR5w3V?=
 =?iso-8859-1?Q?XEM2VoYObIJ3of/+fm6ysQb0ancttD09MCGOI1ZOaPc6Y8m6K+mIf6i8hF?=
 =?iso-8859-1?Q?XI5e32/A8egiB15X6+OXS23xX6YzFlLP/5DOZQQNTbVZPRMWyA72joFFFe?=
 =?iso-8859-1?Q?OPmDQvBqQMSQfjYC9bU+1JjDzJ1/2AWzerxBE31wbxJAekZNDdoLLHpJdq?=
 =?iso-8859-1?Q?57nCjF94h5XNz1DLFTeczl1k+ncjFtxzAHxdB0OgazeM/ce+TRrv2iGbPe?=
 =?iso-8859-1?Q?KbZYvnYxJLnyti6Lv9cg/ERpxMUpd/hVvlPO/bYO0ym+IyrCDVuZ3XUVUw?=
 =?iso-8859-1?Q?4vi9hIdiiFvw/sacPOloczVg5skfLQbzysmXdhWW4OKO57wBQht2X9piGC?=
 =?iso-8859-1?Q?geg+gLHTDhlQXvm/1A96IRITAaBQMoZcQV1srZQkaI/F45TorYubfefWQx?=
 =?iso-8859-1?Q?6+TGOELHXqSCho93Mg+dL4kRZ7i31yPqo+MAzNi6UaNbGeR4N10Bzyy/qJ?=
 =?iso-8859-1?Q?28WsYsmbhCThQN70yu5UtEzKck7DJZ6COeDKof1yJ3DNIxgpwEk9fhrbGS?=
 =?iso-8859-1?Q?x2Xa3ZGz9wKNt8pNMo0lxKxWQRo+fI6tUrkpkEV6q8rY+GkHZEfomRxtAv?=
 =?iso-8859-1?Q?WtMOm0CkqH/5gTVJZmtfZg/iaQLF9uTx0Fbg2X62vPVIZAakM4xRKVokyp?=
 =?iso-8859-1?Q?USW6E+ej43ky3wvBxKWmYC+q3Rbra+QDUBPh9s4IdesTkbCSZGTibU1NEp?=
 =?iso-8859-1?Q?uCD1Aw1UchR+2fms06NQMab+3+Q7nlbHOKsQE2LXIBU+5yhzH4GwAjl9yZ?=
 =?iso-8859-1?Q?306F/wdmsrniP3V4Z2njujZzv8ygMqGq6T9ShQPpV6coD2RjzsOuYUVnAv?=
 =?iso-8859-1?Q?31ksElBykiCJM0WxIuJHQ2DtGeVRwoKdYggV/RY+wVILWeRTEN48BCSOaz?=
 =?iso-8859-1?Q?qCEitq1dMGPjgSM69nPqXv3W5wf0kms+JdmXRQqvF0jq9EVuV60DZfcBBz?=
 =?iso-8859-1?Q?cm2BqAQ1K+s188ZCUH8=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: c0a9a791-756d-41c7-490d-08d9e5f5b1fe
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Feb 2022 02:42:13.2513
 (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: PkvyPuXjaRK7aAIW07Z/EOU+OQUroM04eLMlyrTC3BT5Kha6rh8nOdPhdy9+GmRjpqKbcXaN9QWb7f2dTbkeJA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR10MB4129
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10245 signatures=673430
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 malwarescore=0 phishscore=0
 mlxscore=0 adultscore=0 suspectscore=0 mlxlogscore=999 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000
 definitions=main-2202020010
X-Proofpoint-GUID: S7UsT13IZeQ0ZQdH-rcofvZrPoRa7pcc
X-Proofpoint-ORIG-GUID: S7UsT13IZeQ0ZQdH-rcofvZrPoRa7pcc
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 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.34.1