From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 74A931076387 for ; Wed, 1 Apr 2026 17:33:55 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id C7F456B0089; Wed, 1 Apr 2026 13:33:54 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C2F2E6B008C; Wed, 1 Apr 2026 13:33:54 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B45A06B0092; Wed, 1 Apr 2026 13:33:54 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id A09246B0089 for ; Wed, 1 Apr 2026 13:33:54 -0400 (EDT) Received: from smtpin16.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 4846E1B6C84 for ; Wed, 1 Apr 2026 17:33:54 +0000 (UTC) X-FDA: 84610684788.16.8CFB6D7 Received: from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com [148.163.156.1]) by imf07.hostedemail.com (Postfix) with ESMTP id B90504000B for ; Wed, 1 Apr 2026 17:33:51 +0000 (UTC) Authentication-Results: imf07.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=o1DVKJC0; spf=pass (imf07.hostedemail.com: domain of sayalip@linux.ibm.com designates 148.163.156.1 as permitted sender) smtp.mailfrom=sayalip@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1775064831; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=rSqaC2HS6F3jm0Ha+Zxiv2wIM9/dauwd7UVnyK3ZjbE=; b=z13bYdNrxx84jxLB9yZSquQ59XXCyDbKj8QyxBdXhJIgRV6mTIDSuygduDO21soA+dGU5b hUYIIQ77eJXLVVScDIyE/jLvEQ5UTtde7NQLkOHa5xt8aXZp1EP667NGm6DCPHCumzsO4p qpTd1XFdI4ZTz/rroWGEBb/RPGRbk88= ARC-Authentication-Results: i=1; imf07.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=o1DVKJC0; spf=pass (imf07.hostedemail.com: domain of sayalip@linux.ibm.com designates 148.163.156.1 as permitted sender) smtp.mailfrom=sayalip@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1775064831; a=rsa-sha256; cv=none; b=SwBlJdaje0zk9VZJqJkLZUx1yG5xhXh/pS0+nlZ/GVgXUiS5n4rtwYcQ0YaLCGfyoBv2uE pETFNruXrfH5EC4W2TfuxyjAVetDIj/J2jyGpZdeZF3WOU2yrxX/VCWF+ET5V83ctlYHgv P21QXY7qYxDi7ejyUMl2/PWWnJC7bvE= Received: from pps.filterd (m0360083.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 631AVPX0505649; Wed, 1 Apr 2026 17:33:45 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=cc :content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=pp1; bh=rSqaC2 HS6F3jm0Ha+Zxiv2wIM9/dauwd7UVnyK3ZjbE=; b=o1DVKJC04XxVqWAtIzvm4Q LvBKhissxWZgbfjOMM41uOIN40Vbf7jf8rD5RxfPQpQ8rxnwmMJCTWMrjuTec8f4 O+LXOFaX5MtNt9OwkYdW8m5/6Tlsy7A6TacUJeiTkHJtHOmepgg79JpyRxZTRkq7 22pxvXhguFyk2/LGs0FlKOYyGjBQAG75lFktzcBkT0rVrvhXrU/5NgrEGB9s+np1 YtSPbsKbjeZaQHj3bQ1K5izuMlXdtOlQP6iru4ZV7/lJbr2hksjo5YyawNOt3rud Ux2zjYfchNL/r75AR4k3CvwZBrqhXbNZ/YjQ8/6tx+1qqoA3TibcvbJ+gjo8d+hA == Received: from ppma21.wdc07v.mail.ibm.com (5b.69.3da9.ip4.static.sl-reverse.com [169.61.105.91]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 4d66g21fg6-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 01 Apr 2026 17:33:44 +0000 (GMT) Received: from pps.filterd (ppma21.wdc07v.mail.ibm.com [127.0.0.1]) by ppma21.wdc07v.mail.ibm.com (8.18.1.2/8.18.1.2) with ESMTP id 631FYUVo022165; Wed, 1 Apr 2026 17:33:43 GMT Received: from smtprelay06.dal12v.mail.ibm.com ([172.16.1.8]) by ppma21.wdc07v.mail.ibm.com (PPS) with ESMTPS id 4d6tan6pjp-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 01 Apr 2026 17:33:43 +0000 Received: from smtpav04.dal12v.mail.ibm.com (smtpav04.dal12v.mail.ibm.com [10.241.53.103]) by smtprelay06.dal12v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 631HXhgG26673892 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 1 Apr 2026 17:33:43 GMT Received: from smtpav04.dal12v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 1C00458052; Wed, 1 Apr 2026 17:33:43 +0000 (GMT) Received: from smtpav04.dal12v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 36C5558056; Wed, 1 Apr 2026 17:33:38 +0000 (GMT) Received: from [9.39.18.42] (unknown [9.39.18.42]) by smtpav04.dal12v.mail.ibm.com (Postfix) with ESMTP; Wed, 1 Apr 2026 17:33:37 +0000 (GMT) Message-ID: Date: Wed, 1 Apr 2026 23:03:31 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: Sayali Patil Subject: Re: [PATCH v3 11/13] selftests/mm: fix double increment in linked list cleanup in compaction_test To: "David Hildenbrand (Arm)" , Andrew Morton , Shuah Khan , linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, Ritesh Harjani Cc: Zi Yan , Michal Hocko , Oscar Salvador , Lorenzo Stoakes , Dev Jain , Liam.Howlett@oracle.com, linuxppc-dev@lists.ozlabs.org, Venkat Rao Bagalkote References: <4c47a2ebbc41f90c7a0cb3f8b34106b2b64296f6.1774591179.git.sayalip@linux.ibm.com> <5684cab6-d5a6-4de9-9545-f8cae775d825@kernel.org> Content-Language: en-IN In-Reply-To: <5684cab6-d5a6-4de9-9545-f8cae775d825@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-TM-AS-GCONF: 00 X-Proofpoint-Reinject: loops=2 maxloops=12 X-Authority-Analysis: v=2.4 cv=Fdo6BZ+6 c=1 sm=1 tr=0 ts=69cd56f9 cx=c_pps a=GFwsV6G8L6GxiO2Y/PsHdQ==:117 a=GFwsV6G8L6GxiO2Y/PsHdQ==:17 a=IkcTkHD0fZMA:10 a=A5OVakUREuEA:10 a=VkNPw1HP01LnGYTKEx00:22 a=RnoormkPH1_aCDwRdu11:22 a=iQ6ETzBq9ecOQQE5vZCe:22 a=Ikd4Dj_1AAAA:8 a=VwQbUJbxAAAA:8 a=VnNF1IyMAAAA:8 a=T6nULJMaX99D9d00DIAA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 X-Proofpoint-ORIG-GUID: Bxer0GwKb5xpt3tAX3oiBerSsEtjsxnZ X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNDAxMDE2MiBTYWx0ZWRfXzEISyrtqStlA hNkcbo7yzW2GQIQlG/T++H5n5nbP9R4jBNPLST6vVhS0DrgKsrqn9FT6HkVY/ip1FchAa3/SLMl KtPoBJcHbTL157FOIpjzDMq0ytao+4spNvl9bCbLTdzKwtVyL4VgqLI2+WScV1BTmtyMjS9L6Vj kwe44k4cuZ4BTzzlv2FQdD9ytLMhoDWFzrc5nqQyHpXhEEN2Gyih5VhWoe26aHMR36iVWy6dsVQ zZiWJpH/ckXojBJZwOoLcpOYp2QLtRj3IYONhTkJ+8olYpEUpFpFaO85uHSBxLD9MDiRtPze2oP uR9WdL9QrlU+cSU0/fLvpR5MDQpt1kl9Puyp10hbaLabexZqJYewwFd4GzE9Hlj4Z0QdAONKp9k FZOHRpiRtOXRbJZ7TZtr3cxKjuDHfHv2/mJD3kY3SJTbAtJxbPEpBj67f66RlDs6/2VUSQ+P/CB V4OX2hHq/vbPwVps/jg== X-Proofpoint-GUID: aqf5EXqq0VS2xdQNbf1Bhqhvgccm-WRq X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-04-01_04,2026-04-01_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 clxscore=1015 bulkscore=0 suspectscore=0 priorityscore=1501 adultscore=0 malwarescore=0 phishscore=0 spamscore=0 lowpriorityscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2603050001 definitions=main-2604010162 X-Rspam-User: X-Stat-Signature: n7he8fiti8akm6pykdct7ts77qaxwfsk X-Rspamd-Queue-Id: B90504000B X-Rspamd-Server: rspam09 X-HE-Tag: 1775064831-868092 X-HE-Meta: U2FsdGVkX18TgNdUyrjRPq79APYr9Jh044KsVvITLx19Tq8Bk2jd6evCx34tD8E5RZlBN5glN0KXAaAkIyGe5FbTh6Z+PGFdbGHIoCSTfANMO7xCgncFHe2xgfcW1e2Sz3hA1I7OeI0KWEHtAgay9k4p4bf6Ntb3cO70bXjcmJz+gJx0hki/FFFbwRYSBQQ6BuA2bta+ozXTtIwkoarOvZiFdmKEmVol7EqvYikQh3Z0GUw0JCPfDZOOLO6ggRuL5mnTT3QQptC9RKqHRE0ulU71PX956MWuZnrLTiGePnzxDK0uXKwF2iZyFmOcCepjbk3ixVng/MJzmpmiM3+1gowSA2gfaXRvG016au7mtSApJeDuCqPf64FAJMxm3mV4rhVuwmTO6s8eQLEVlWxIVnuxle9d0MT5wwc7MpwMqKvPzXmazfK8AE4s8b5L1Wup3+4KMCi/Cf1+josfDXrTyqUh31RVwFJYAbc/JcnIhH489S/8psHMQm4QCmXDT6qBc6wxVD7bfjjr7x1WbxNIDPR6lxSjVXSBL1o/YiuH9Npv5t/FkzmNQ+fCSKEHm+RZQUdjjZF3un7lOlGkkPCS7sPvgbrPTVr+SLTYH014ziNUzMDaPE+FVHWVxZ7/HqFv644KJ3QSb8u3FDTUMwON0+uUFOyswf9vgL0JkcjYqB3793UiM6w6joxq2Le32r7v8jyWnwwV6rMN0n3dHWUlJ8DAnOPzagT2RMv5eBts1IRoRedBeSaXxQB/p5XDhTUL/QPQcqFdazDMKOTsBqF3ls+hT7vsnV7PB0rQXWv82j4j8qFfUHG79Ai/dq09HDLNwqhx62Z/Yrvu+z5TwbB71H8lrmTkCUr1SOjQ9V73hdVIx8Tv8bkQZFXzV3cg/ea9BrprbEXpXPSUFIqOs4WjGEKzHNBG/hgHgthvXvKzFdWuyQnG/DwheeRE81klp2LRW6CGiB6yA9NQGq6w323 nzBofgxT sqp5uIBV2oNK1IVlMI5HNoZXsxPj6SiidkPVBuVds9bOWZtbBZIiW/D8dlqXeXLe4HyNS+COfKi2nMkduhzn9LcDnxXLwvZ6AlgNJtXNMK2vr6AktS9rPlTIwDjpriydd1ELEYCjpldgVJ4GpxjguGXnTKu7pvbeGRT9b9QFGtPF46WWq+UTd7DMe/r8vKmADwv51XSfPkUSIWuh/9Oo/266pi1JuThypN24UURltyRE/xGTJdpFzMf2YcvarQv22D+O6vN0eDMbee+khaX8fGx3E0z36OBi3iPBQYsNMk91uxJ921Et3DufDxTJCOvr72FqY/krDn+wJWSylLoRNBYl20bptDMlCCyZJ9cr48R6e7CVz5rOXMFWhbUNdWP4xjUQFfQAwj8U74VWFVU+woB7NolOeVFt31FEhESKH81N8fSrOQrHGTa9XIyOY3ambNJZu Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 01/04/26 20:09, David Hildenbrand (Arm) wrote: > On 4/1/26 16:32, Sayali Patil wrote: >> >> On 27/03/26 12:46, Sayali Patil wrote: >>> The cleanup loop of allocated memory currently uses: >>> >>>      for (entry = list; entry != NULL; entry = entry->next) { >>>          munmap(entry->map, MAP_SIZE); >>>          if (!entry->next) >>>              break; >>>          entry = entry->next; >>>      } >>> >>> The inner entry = entry->next causes the loop to skip every >>> other node, resulting in only half of the mapped regions being >>> unmapped. >>> >>> Remove the redundant increment to ensure every entry is visited >>> and unmapped during cleanup. >>> >>> Fixes: bd67d5c15cc1 ("Test compaction of mlocked memory") >>> Reviewed-by: Zi Yan >>> Acked-by: David Hildenbrand (Arm) >>> Tested-by: Venkat Rao Bagalkote >>> Signed-off-by: Sayali Patil >>> --- >>>   tools/testing/selftests/mm/compaction_test.c | 3 --- >>>   1 file changed, 3 deletions(-) >>> >>> diff --git a/tools/testing/selftests/mm/compaction_test.c b/tools/ >>> testing/selftests/mm/compaction_test.c >>> index 30209c40b697..f73930706bd0 100644 >>> --- a/tools/testing/selftests/mm/compaction_test.c >>> +++ b/tools/testing/selftests/mm/compaction_test.c >>> @@ -263,9 +263,6 @@ int main(int argc, char **argv) >>>         for (entry = list; entry != NULL; entry = entry->next) { >>>           munmap(entry->map, MAP_SIZE); >>> -        if (!entry->next) >>> -            break; >>> -        entry = entry->next; >>>       } >>>         if (check_compaction(mem_free, hugepage_size, >> >> Sorry, this change is not valid. >> >> The goal of this test is to verify the kernel’s ability to compact >> unevictable (MAP_LOCKED) pages. The loop is intentionally written to >> unmap every other chunk, thereby creating fragmentation with locked pages >> before check_compaction() is invoked. >> >> With the proposed change (removing the double increment), the loop ends up >> unmapping all allocated locked pages instead of leaving a fragmented >> pattern. This results in memory being effectively unfragmented. > > Ahhh, we should really make that clearer in a comment. I missed it myself :( > yes, let me add a comment to clarify this and send it in v4.