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>
CC:     "Matthew Wilcox (Oracle)" <willy@infradead.org>,
        Liam Howlett <liam.howlett@oracle.com>,
        Vlastimil Babka <vbabka@suse.cz>
Subject: [PATCH v13 35/70] x86: remove vma linked list walks
Thread-Topic: [PATCH v13 35/70] x86: remove vma linked list walks
Thread-Index: AQHYtji6yK3zROAHp0+KBXUHJAPK2g==
Date:   Mon, 22 Aug 2022 15:06:16 +0000
Message-ID: <20220822150128.1562046-36-Liam.Howlett@oracle.com>
References: <20220822150128.1562046-1-Liam.Howlett@oracle.com>
In-Reply-To: <20220822150128.1562046-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?9XK84wgT3jT69lZ5XZxeaoD9eVbvmQoMUez6CJuIkSbrBx73prrGRIV5+4?=
 =?iso-8859-1?Q?p1uRTSG18V2GMG4jDs9beZxBSfIhkEHHEfsaPZMhTVW+aNuH6YSicQ4VrZ?=
 =?iso-8859-1?Q?Mp9Y28pUr0JyGgL1GQtpakSjP287/NJYZ/t4URmopeQw2t3oUTshh4sC+F?=
 =?iso-8859-1?Q?gcWqofqkjInkb9vkagTSiyocr/XvBqD/sVlYPx7XN929a8e424eU7Mxktq?=
 =?iso-8859-1?Q?m3YNNrNMMVHI6qEeQZ7km7G5hEEkZYufKCn1/FMjVHwbWxOiPpR/l2pUJs?=
 =?iso-8859-1?Q?GgOXRwPfcAsYZE/1iavat4/WZ1+2htwRIoCgVWalxafefLItOV8BFCSFPD?=
 =?iso-8859-1?Q?eqT25V6KcEi2Xg6pYTHaw7PNHzOHITXOtDU9A8mKc8w6m4GABPKyEBsNyp?=
 =?iso-8859-1?Q?l7ZxKX/HKdkU/Sn6g+iAI1eO7ExIazlBAnESHso7b66fTnWO3TLegSAgsU?=
 =?iso-8859-1?Q?rJZCT3keIqe/BxO9v02ttay7eYNUrABg07u7IUdwwU352BtV460vaF1d/m?=
 =?iso-8859-1?Q?UDzf0W5ulKVm06XPnmdBf9sV4B9gS+kw+4BxKc1+bM6EZ7z3JpjlArU4le?=
 =?iso-8859-1?Q?9J0PxR2XT0SeoF4QgbFIICgY2BA1Rr5Nl41J7os+cpFu9vYS5ASFP9Ofjq?=
 =?iso-8859-1?Q?ghsqWHLkr+HJ0l92SRVaq+cw5uFKLfZYYNsPBpzc8pxDAibz+OV3GVAHbw?=
 =?iso-8859-1?Q?lBLyEntYOrizXVQwaZ/KPvvmAbvz9Xa7aUUfyVLyfHNCjTRYFrpw3vGr4v?=
 =?iso-8859-1?Q?uwTUYE+CdTY4cjn9uQp9ZFqMj2nUSrXdJRfFIEnxKuFE9O50/BWGlL9AmF?=
 =?iso-8859-1?Q?faA6qHlokAnCfq26qGVZiFiRp631Yi8QDLg7nqu9ZHw/hPuoR3RihCsDFK?=
 =?iso-8859-1?Q?r1gYY+TW5zRVJLwft64cimfPp2aUx4ErQWjiNjrRFa7s3L4dLAQD4qK18z?=
 =?iso-8859-1?Q?Tz/+I8Db4fvn6N4DiSaDuYwVryIGhPcRbeZPtYbUq17Xbd+ZUYeel+Ae/8?=
 =?iso-8859-1?Q?lo42iaZ0GQ/5uNDDcxjnW6VhJd+FjvIGne0qOpHzD7FqzJj70PbSGPPXKB?=
 =?iso-8859-1?Q?0dkg54gynA/EKk37vs9I5h6KuQBpiCcMiFbPE8cEysiqYftZrwXHoXZeYN?=
 =?iso-8859-1?Q?rfzKhDYDHjJwXkYeAcLcNZVuXVeFEdcvCxaXztoa6sR0SttTs9/3si1I/5?=
 =?iso-8859-1?Q?QlpOFR+XKOZQvCW0v3ZTQl4cHsHk0bP+5EJhoRLK1r+pXPLgsVlJPZTiJr?=
 =?iso-8859-1?Q?g+G0yH3PeLMEoMQkY+Lgog4UITludI+pGJVEL+kLxjt3XUW3F9lfLfbyQ/?=
 =?iso-8859-1?Q?bH0ptvwRe5zkl8JJLpDUu035z7i8dND94RSqWEaGJYNJ/v5vpt7nLByRV4?=
 =?iso-8859-1?Q?WvHa/rvcsjZ89TTmBluzkbJIC6AqAruqaYaWDA7WOtDGquQ0XXTy6nk6RE?=
 =?iso-8859-1?Q?hiJTd2Kq9Y+Uq3qbHnXlV+e0hb83k8cdiwC670nCP6LwgMQgj0aSk7ISEW?=
 =?iso-8859-1?Q?wAJ2bxoCn6X5WzSpevbo3PwQn9TkSCoq7QXfLupGA1Mcc+dOhwOfoG7p4O?=
 =?iso-8859-1?Q?D6RT2nW1u+QweGN5iiJ9v/u7rj2aiV82io2HxAfG8nFvz1kp4109GZqBZh?=
 =?iso-8859-1?Q?2Q1VaWzs1WnMJzVo3Vc4wTj/EpkBlQoUYGS2y9JCGttf+PuGmmDkP3mw?=
 =?iso-8859-1?Q?=3D=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: c8007260-45c6-4b4c-a6ad-08da844fdca8
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Aug 2022 15:06:16.5200
 (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: ujF5M8ib1NgErkiHbD5A7J75SfVVRrZQGVUvXRIJldIWyoTQy24PAgF9P+M4GMgvp9+xDbOwK2YY/Jr/k3hu+w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ1PR10MB5931
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.517,FMLib:17.11.122.1
 definitions=2022-08-22_09,2022-08-22_02,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 mlxlogscore=999 adultscore=0
 spamscore=0 phishscore=0 malwarescore=0 bulkscore=0 suspectscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2207270000
 definitions=main-2208220065
X-Proofpoint-ORIG-GUID: QqO0kEgpCvsA_juBRjAlmeENKWr5d9rx
X-Proofpoint-GUID: QqO0kEgpCvsA_juBRjAlmeENKWr5d9rx
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: "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 1000d457c332..6292b960037b 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