From patchwork Thu May 4 23:38:58 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mike Kravetz X-Patchwork-Id: 90267 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp37840vqo; Thu, 4 May 2023 17:19:02 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5VP5l/IoO60RYDDo3EQBQ0C7CeFcG4X2BL0JpMsGD55srWWjiuwHqKQ1h3aNzqMEAwhDfO X-Received: by 2002:a17:903:41ca:b0:1ab:c48:1de9 with SMTP id u10-20020a17090341ca00b001ab0c481de9mr6467088ple.65.1683245941992; Thu, 04 May 2023 17:19:01 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1683245941; cv=pass; d=google.com; s=arc-20160816; b=kuxtotSTpHSN8xir9IL9AnZvJOtnFLckMeIGsayLHJzbcVyia0pgW0FZDQosDJU4gJ jueKxQvaHINmmxJbbhhVwdOrQD9F+V1OcecUwUy9B4zcK3DTGQJ1uscaNUrd7N4/z2f+ EzfZ8l6zPICqWqBs1qcKjx7qSriT3ej6kZ2h1TcBIc3S9oH6DnUxM184zWgUrrg26yab QwpCFfimBlBayIIpxKRtusw+OI5bw3EfCY1znz8SudZ32GEimKiD0M0DPJXZipJ0agIQ rEYOpvi5F47WuPeZWw8QgNTja8k+R57JzVjOAu/+YoJ4CNork09cTUwprfkNKL5khGLS qJNQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature:dkim-signature; bh=GLq3qI2Yxv5HSwxpOHjg+O6jlpfkeBnZRt5doMWQYRk=; b=TJH91O7Enp5GYDmXjtQZQwDMU8DwbJ2+luRRl0jS2cBSrmLofnm0j7152LhWRtukx1 /G1KuzTa9dRN90GCkpxKXRU20O0+Z962r+/F2D/fzfuSP+DnxTh5qvgnXZhq0LJ03lJl A/1nleF3Iq588bGzCPlxXidk0hjD/dPPIx8GKnG2VtCo5i6Ctu65vSZoIrP2102Qt6IT 648lH11M3wCIgLqZzJoaO/4oc6GmvcazYxNXgBw3WglmaC10Q/A7liwj2k2flvHWBA8k qGWeCh+cLH7W6NP0EehO2tNPGzovh6q7O5B7+OZdsmesMhojEDebEQKGMY6BLMgI1RPm 2YKw== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2023-03-30 header.b=utKOeqcn; dkim=pass header.i=@oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b="n/5vQjJ1"; arc=pass (i=1 spf=pass spfdomain=oracle.com dkim=pass dkdomain=oracle.com dmarc=pass fromdomain=oracle.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id l4-20020a170902f68400b001aaecccb2c3si421200plg.7.2023.05.04.17.18.49; Thu, 04 May 2023 17:19:01 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2023-03-30 header.b=utKOeqcn; dkim=pass header.i=@oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b="n/5vQjJ1"; arc=pass (i=1 spf=pass spfdomain=oracle.com dkim=pass dkdomain=oracle.com dmarc=pass fromdomain=oracle.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229907AbjEDXjp (ORCPT + 99 others); Thu, 4 May 2023 19:39:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56966 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229879AbjEDXjl (ORCPT ); Thu, 4 May 2023 19:39:41 -0400 Received: from mx0b-00069f02.pphosted.com (mx0b-00069f02.pphosted.com [205.220.177.32]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F1EE0AD27; Thu, 4 May 2023 16:39:39 -0700 (PDT) Received: from pps.filterd (m0246632.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 344LxTvO027934; Thu, 4 May 2023 23:39:13 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : content-transfer-encoding : content-type : mime-version; s=corp-2023-03-30; bh=GLq3qI2Yxv5HSwxpOHjg+O6jlpfkeBnZRt5doMWQYRk=; b=utKOeqcnSKL7U0EsscMT8wurcwPnd08UQFxbEVN66NZRbXKMEJNskLRlplgPHTBnX9DL d5oAw2NCgH3ZxXKnWY+C6u5GLXXvHoHKaU/NaMU1f/THopVvJoMPCimXfsJ7Y7dtUcrc 9EcQzHsFOiqmZMSsYwKV51XEJwxf03L4hGjP/tuIgX1FjwNy01TeFhGXMxKJyEos+229 BU/m8zY/hM8xTeHGzb1yatUGDI1WTxujDL353Ocu/YR/q6GH3TrqYGmMJE0PAEilzNJ/ W1TD2Sw3154Nyb9hn68AjSsmgc9V/DjmFkh4XRnIWDpLkbHiR04H/+B65dzhOc8+Y5ms nA== Received: from phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com (phxpaimrmta01.appoci.oracle.com [138.1.114.2]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 3qburgbfh6-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 04 May 2023 23:39:13 +0000 Received: from pps.filterd (phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com [127.0.0.1]) by phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com (8.17.1.19/8.17.1.19) with ESMTP id 344NccHo020736; Thu, 4 May 2023 23:39:12 GMT Received: from nam11-dm6-obe.outbound.protection.outlook.com (mail-dm6nam11lp2169.outbound.protection.outlook.com [104.47.57.169]) by phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com (PPS) with ESMTPS id 3q8sp95wc6-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 04 May 2023 23:39:12 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LWkGwQEOBRHFeFOtb/1Bf31zgefqvdxfL8Kf74mD2RzqD/5tu+X+vOJ9n4lsXE5JxFbhlCQ3Zyo8Q8xTgysAwi+ffyvJfzkLBBiGASd+I77Vu/u8UbwdEO7gD59D09eIgoDWetuiCiSFC4Vl7iirC/laNORgSftWOzr/8hcWxoxNleMe13uTVRW9W9OhorjB/SrXJoljm9aMKib4koBJaRKt4PA9Se3b6KpOLWdc06tf/UYv5LbeY6vbFKTq7tNV0t5jgTEvnko0iU6O5AgRXeekF8GHLD+1c0F+41LgofqgYbuAEBH2RTY/Vr8xHxEbBpOqSi6bHf+U3Co5lvQyuw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=GLq3qI2Yxv5HSwxpOHjg+O6jlpfkeBnZRt5doMWQYRk=; b=SkDtycF2If6b4PvQM62M6Hv5evnAt3mzxIYDYOjacqMrFKCPMaixq93GPPZzl+M6vzfx2LpSYB+O8nd6dThbvupqxWjL18jiCl1dyxKAzAEf1GUI6sD1NrF3EPIRmqe94cCYnHCT+H4A8+Agxs7vhnjQ0ai5F66PzTT9277bEJ+8+4/JICx9HsEnbkPVoV1S9VnqmkSaFOLayRhQEq1962JfM1ZtLkBu7MAjQsxwfDYpmUmWd4S0eKIjpf6BWc/fRxkeepNcahyIMiM7oO4zn0XV7fOUS4e/XswD7606TSBATyHHe/1azYX0buXOThQ8E3CXVmVUXHpxyLo/5WhKww== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=GLq3qI2Yxv5HSwxpOHjg+O6jlpfkeBnZRt5doMWQYRk=; b=n/5vQjJ1vYQNRoUBR92v0oqaqWOVm6i7JNdQjVKDFUuYCrGJmMjxCZpjOLA5sSyVAKS2Ftw8mAgVqqCh1XkwK9Z2C9GUTO5KGxqvdBD68Pe4sDvVLdhohR4JrefSYA0h/ai3QH3LbPT1hJLuSC2JVbagBU9e1YxXvfWt75dGIFM= Received: from BY5PR10MB4196.namprd10.prod.outlook.com (2603:10b6:a03:20d::23) by CO1PR10MB4706.namprd10.prod.outlook.com (2603:10b6:303:9d::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6363.26; Thu, 4 May 2023 23:39:10 +0000 Received: from BY5PR10MB4196.namprd10.prod.outlook.com ([fe80::ffb:de39:b76b:52eb]) by BY5PR10MB4196.namprd10.prod.outlook.com ([fe80::ffb:de39:b76b:52eb%3]) with mapi id 15.20.6363.026; Thu, 4 May 2023 23:39:08 +0000 From: Mike Kravetz To: linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org Cc: Matthew Wilcox , Ackerley Tng , Sidhartha Kumar , Muchun Song , vannapurve@google.com, erdemaktas@google.com, Andrew Morton , Mike Kravetz , stable@vger.kernel.org Subject: [PATCH 1/1] page cache: fix page_cache_next/prev_miss off by one Date: Thu, 4 May 2023 16:38:58 -0700 Message-Id: <20230504233858.103068-2-mike.kravetz@oracle.com> X-Mailer: git-send-email 2.40.0 In-Reply-To: <20230504233858.103068-1-mike.kravetz@oracle.com> References: <20230504233858.103068-1-mike.kravetz@oracle.com> X-ClientProxiedBy: MWH0EPF00056D13.namprd21.prod.outlook.com (2603:10b6:30f:fff2:0:1:0:16) To BY5PR10MB4196.namprd10.prod.outlook.com (2603:10b6:a03:20d::23) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BY5PR10MB4196:EE_|CO1PR10MB4706:EE_ X-MS-Office365-Filtering-Correlation-Id: 07263c40-6768-45e5-2fd5-08db4cf8c193 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: GB629ZNoFrq7qVvKqPaVP0nwmRjEGnS5jheF9tJ9YsLp0qLnb9EUHt8dA7u3oMknVhZfEy2TboboveX0+g4l0EEhlJbeS9orMvmcb1QNuDV8Jb5F/uHkTnpEbqf2giovbl5TArjX0Z0eEUB01hb9Udgq4XL8k1UHED6aRxhEaMDgjIYHJvrJ60NFi7O4VesbNHnz/shVk97G6l1cqnlGWVgeC+Lctm18/eS1spYP+OOoor1ZRFmVTXJdQE9TLKAAUNYP02cPYKjzEQQNgr1tD19NvDcyqBeE4wkGnIwgSP5Ry/xKPTc71xoVGz19v6YegVWCv0rUxGUYzn1ac6ESHjyPNq97ARirqaSgQHCcEBkMt9VB2CM+BmdqDDIr7aGMYlaPuv9GFDnEWg6h5JPlhYshNJXwOQ/4F64JMI8wtgYs8SvMfwncJcanBCOX6yZfwTRCYkPaehiHj7/YfhWLN4srVGlCU1OjpXYaXOIaNMHVqpQDH+4dmYFgzXy9SlHGBwDL3aaDm04fJjug55e2rOcWXnex2ioviK7JyEe+Hus= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BY5PR10MB4196.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(346002)(396003)(39860400002)(366004)(376002)(136003)(451199021)(66946007)(54906003)(2906002)(478600001)(5660300002)(316002)(8936002)(4326008)(66476007)(41300700001)(8676002)(44832011)(66556008)(38100700002)(7416002)(86362001)(6486002)(36756003)(6666004)(83380400001)(186003)(6512007)(26005)(6506007)(1076003)(966005)(2616005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: OaPoHvgFScspsVPcEZY8R30HvPiouIZNHdny4QCqPYFqGMkNTOr7f4reXwDCTQAlmtwNf/fXl452kuBx/ct4U7MCfalIBLaxxhYGm/1czmYlRJdRI3wNlLCK5gZKFpIFFY3xqCdsWUwMUxWm4afdGB2iewCHBuKsXuVK2UoL0Vu9s4vW7BbJ7NtiVyUS9RVSC/wcpU0Gy5O4XjvLpuaudexRsggqo+AcKsP2C0jcfQZ4ZZp8BRubH9v9ikRsCVe0sMZwqoCvRVW6UA0ACrZStfJEeX4TCUzRz1kCaqCAIsB1Jaouw9nE2AeiDm2wscmYD9Av4v74SAmunuva/MpbMs6U6jBqjklplTxEIl15SgbPNAYgjSXWV3d8ky04nUrrnpYov1EEannHZclMhcoMV3p+X17t+QmTvWmeTTfXcWDgaCDkHODk/AZ5M7Uvhjn+IxCKDCd7sav7UCoRLP+nQmRJb2AgwbOgjqpYFgkcBlioKAUuZx/6/y+McKNV785dZ7m/uMeVNcnuQeRcknFEO+LxMmXgT8f6JTNx+3CdjGbUjiGBe+zRjXsrE22nU06HG11lRxnKqNa+eNDT0ZCGsoVSTbU1nIDiqSRMlTUra3FkJVj9wJMBATkdDGfKV5n28xUusvaGe9uNjBmzxVhW4X71Qke6lK3xXyUdkyZfOILeHmh3oTMn876c51wFjmfGU/nkXoo6hsM7Y3xc1/pIW9XOhCAcUBBIOnbx2O2TZarDo/AhKEG714+EDSqtKEHFp+IlSxUZIYWduU0ZmqmzX7+V6bxeY572NKNVQliFoi/ctafbV1VvTnjHo9FBDSgjswr8C4mBnxDGs6Zn1WahZ4h+/LjH+nR32ObJU+Cu99SR7lJPn5XqIAjcI8jngpOCvTlCefLYcVSNm/PjYhHRKtNWvbTL4hDYsjtoVxSiKIkDE8SCKHRKEoz1hUKMr9gYqxlOzoUM2l7cY3TQ3DR38lS0Gk12knjzuC3pvDUfOwqlImYOioEz0WAZP38i8SYtkXA0Ax69zPKYZjKkFHT5sGrMYUhHhXddjkT82NB/WRCJiR63dJ4FN4UAyh88veGqF6WJ6IC+Pc1oTBCmPHQNQC8zgk/EZ6mZm/oQ0VanMzfxATeW+tW9XkR+v1tsP0iO56s+bjkqX1/ZNpAEYPxuAYaNH12MEIvflBxAcgFM5FanukDMXWje6Wkyk+Dkkwjd2hDhm7+8IPxZxU4p3GX7hw48aIyR52F5JYgF0myLLLPFUlK/eEt/V0r+kj1UOpu5ZxucJ52j5vsgmZ89iq/0wZpL1D8oRIjKD327cXyvcSQs2fg1D3WWzI3DIZgZRiJEtai74QpnAwla/+tOIWWQH3Bbs/fAvHAZliZ8M8E1SE/U3GjIQ6ni2PrRwvlMUZkqzy2M4F748UTHWCRAi03KL9tQEHVOA13A7XcIdoEQzA97A9HCJMhuvHRTYB/BXlzQyv0x9F8ns6P9a5MHx6WJQj0mwqpzSgqBZx8KUhVMBfisKSNb+nMYGqxVMINzn0cZ7QPxGXMPsDc5njDTxiaTKK5mdmfoEzEb+giGxMHqFIqvS6PixQazIF1xmJhpXOcY X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: mQMjqMTrnDUicjk/2ib1pbS1BjCsRfnvjav8+pR6daJPdUDE/JmzG1lbnC9Nho0uZXBhRPwYtxUgR6pfqb1F8ZhD3ivlhW2/XpLBiYc9s0viuVVr1nWwB0gx9/KPkFgWi0RDj5faOw4G44Ip4Y9lblA4ZmRuOTO+2oeM52PLutgH14UIji/x6F+OscAV/NAYXMwgKrYAm4Jrmwlg6yBIBzXT9zLxFyNSZV+QytKHqtkS7E+GvjfkUwl//XtQXN6J8vRG4dmhoAMiJIxkNCW2mwMbtYl57eSpc306qDAKPe9yvGEjm+botQeBgUuDFSv/cTm31slUg3pViA2Rj+e5QqVs668wbdaUAIXVhm314os5TDXzRGFzm6+MKjcqcXXYncPHJ2YkmvohX87lOX5TBN/oNZkxhGF/JlW6v6LJUYjniEzuqGTPMipD+KggZ7CXZDD08B5HmFv16pZ9CgkkyrYd/CEZYCwq3PePn27M13/tS9Cvd/AVBMCQ6jE6G0zUKu8iWkuLlllUQt4ZmRgqhYrrBiwXW/VEJaSErPGVrK8JAajwzrdF/5L3q6nHv7tdjWbSPh93Pff1WN1yhag/NCSlZ6TlzAGFctZqxOZ6jDuBpwMEp7GXwCA44Ibun4rC2RmadwKD0Psw7IcILPD+HJvBd73UY5sIxQwt2ZcEbtofHsF+yA3RShQ837Bn/+b3VATOTdbo8zsQqUsXVy53LMoHEds2pgaUTaEcIhoMPUwo12KrepLRDM2HuSeOjlIih4Nr1xUA/8uPEjn+8W8RhU4tl3jer2nAgyWbKzLs96TNz5xfGwlVTrbG2d+zWY0EZjkx+iyXznGe6LM1xfDqlhqsqJBfhCsTLPbR25X78Wxz9acvbCP4sks8EBsC+EPhlsPPVpidDli5TM+g3G4Q2UfN5df2CfMfECLfgeDAeWTysS67RZOv939yPwe7w4bo X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 07263c40-6768-45e5-2fd5-08db4cf8c193 X-MS-Exchange-CrossTenant-AuthSource: BY5PR10MB4196.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2023 23:39:08.7440 (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: auXNcPTzhvyzpl7J7SllsZpfgfCljkrbvqHMcP98ZCh+Qr1ikCXB7u2q7+0MsDjLp6aULkSmskkU9YN0kQx/fQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1PR10MB4706 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22 definitions=2023-05-04_14,2023-05-04_01,2023-02-09_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 mlxscore=0 spamscore=0 mlxlogscore=999 phishscore=0 malwarescore=0 adultscore=0 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2303200000 definitions=main-2305040190 X-Proofpoint-GUID: DlpToA6PPbKGyarX5I0qZeoGjBI21U_K X-Proofpoint-ORIG-GUID: DlpToA6PPbKGyarX5I0qZeoGjBI21U_K 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: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1765011297022893014?= X-GMAIL-MSGID: =?utf-8?q?1765011297022893014?= Ackerley Tng reported an issue with hugetlbfs fallocate here[1]. The issue showed up after the conversion of hugetlb page cache lookup code to use page_cache_next_miss. Code in hugetlb fallocate, userfaultfd and GUP is now using page_cache_next_miss to determine if a page is present the page cache. The following statement is used. present = page_cache_next_miss(mapping, index, 1) != index; There are two issues with page_cache_next_miss when used in this way. 1) If the passed value for index is equal to the 'wrap-around' value, the same index will always be returned. This wrap-around value is 0, so 0 will be returned even if page is present at index 0. 2) If there is no gap in the range passed, the last index in the range will be returned. When passed a range of 1 as above, the passed index value will be returned even if the page is present. The end result is the statement above will NEVER indicate a page is present in the cache, even if it is. As noted by Ackerley in [1], users can see this by hugetlb fallocate incorrectly returning EEXIST if pages are already present in the file. In addition, hugetlb pages will not be included in core dumps if they need to be brought in via GUP. userfaultfd UFFDIO_COPY also uses this code and will not notice pages already present in the cache. It may try to allocate a new page and potentially return ENOMEM as opposed to EEXIST. Both page_cache_next_miss and page_cache_prev_miss have similar issues. Fix by: - Check for index equal to 'wrap-around' value and do not exit early. - If no gap is found in range, return index outside range. - Update function description to say 'wrap-around' value could be returned if passed as index. Fixes: 0d3f92966629 ("page cache: Convert hole search to XArray") Cc: Reported-by: Ackerley Tng Signed-off-by: Mike Kravetz [1] https://lore.kernel.org/linux-mm/cover.1683069252.git.ackerleytng@google.com/ --- mm/filemap.c | 26 ++++++++++++++++---------- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/mm/filemap.c b/mm/filemap.c index a34abfe8c654..60875d349a7b 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -1760,7 +1760,9 @@ bool __folio_lock_or_retry(struct folio *folio, struct mm_struct *mm, * * Return: The index of the gap if found, otherwise an index outside the * range specified (in which case 'return - index >= max_scan' will be true). - * In the rare case of index wrap-around, 0 will be returned. + * In the rare case of index wrap-around, 0 will be returned. 0 will also + * be returned if index == 0 and there is a gap at the index. We can not + * wrap-around if passed index == 0. */ pgoff_t page_cache_next_miss(struct address_space *mapping, pgoff_t index, unsigned long max_scan) @@ -1770,12 +1772,13 @@ pgoff_t page_cache_next_miss(struct address_space *mapping, while (max_scan--) { void *entry = xas_next(&xas); if (!entry || xa_is_value(entry)) - break; - if (xas.xa_index == 0) - break; + return xas.xa_index; + if (xas.xa_index == 0 && index != 0) + return xas.xa_index; } - return xas.xa_index; + /* No gaps in range and no wrap-around, return index beyond range */ + return xas.xa_index + 1; } EXPORT_SYMBOL(page_cache_next_miss); @@ -1796,7 +1799,9 @@ EXPORT_SYMBOL(page_cache_next_miss); * * Return: The index of the gap if found, otherwise an index outside the * range specified (in which case 'index - return >= max_scan' will be true). - * In the rare case of wrap-around, ULONG_MAX will be returned. + * In the rare case of wrap-around, ULONG_MAX will be returned. ULONG_MAX + * will also be returned if index == ULONG_MAX and there is a gap at the + * index. We can not wrap-around if passed index == ULONG_MAX. */ pgoff_t page_cache_prev_miss(struct address_space *mapping, pgoff_t index, unsigned long max_scan) @@ -1806,12 +1811,13 @@ pgoff_t page_cache_prev_miss(struct address_space *mapping, while (max_scan--) { void *entry = xas_prev(&xas); if (!entry || xa_is_value(entry)) - break; - if (xas.xa_index == ULONG_MAX) - break; + return xas.xa_index; + if (xas.xa_index == ULONG_MAX && index != ULONG_MAX) + return xas.xa_index; } - return xas.xa_index; + /* No gaps in range and no wrap-around, return index beyond range */ + return xas.xa_index - 1; } EXPORT_SYMBOL(page_cache_prev_miss);