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]) by smtp.lore.kernel.org (Postfix) with ESMTP id AA087C4167D for ; Tue, 31 Oct 2023 17:02:46 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 35F7E8D001E; Tue, 31 Oct 2023 13:02:46 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 2E8378D0012; Tue, 31 Oct 2023 13:02:46 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 1626B8D001E; Tue, 31 Oct 2023 13:02:46 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 01B738D0012 for ; Tue, 31 Oct 2023 13:02:45 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id AE4E2A0321 for ; Tue, 31 Oct 2023 17:02:45 +0000 (UTC) X-FDA: 81406375890.11.1FE1E42 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2040.outbound.protection.outlook.com [40.107.236.40]) by imf21.hostedemail.com (Postfix) with ESMTP id CCB1D1C002D for ; Tue, 31 Oct 2023 17:02:42 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=memverge.com header.s=selector2 header.b=aCJ9c8fR; dmarc=pass (policy=none) header.from=memverge.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf21.hostedemail.com: domain of gregory.price@memverge.com designates 40.107.236.40 as permitted sender) smtp.mailfrom=gregory.price@memverge.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1698771762; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=lFgf2S8ET76HACcqHN0EKVE7QG+sXKNnVNdN2ze0pRM=; b=wGBer+cDiNchSZ5uBH5kOxDEYtPBelVDWQXQzblUG375yVMqjLWBPS8aO7jUW3uzg7HKaw jRcGuW0mvymgEBe8K6czc4X93hH7vNfDKRN0voRBpLJ5BIvJVtugKwjD5SqaguhQw7IMMJ 3XAlRo9u7OGW1N+24hPvXLAa9VsOnoI= ARC-Authentication-Results: i=2; imf21.hostedemail.com; dkim=pass header.d=memverge.com header.s=selector2 header.b=aCJ9c8fR; dmarc=pass (policy=none) header.from=memverge.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf21.hostedemail.com: domain of gregory.price@memverge.com designates 40.107.236.40 as permitted sender) smtp.mailfrom=gregory.price@memverge.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1698771762; a=rsa-sha256; cv=pass; b=HMAz/Jjz0X+G8XDRbsf8Pd5k3dibFdc89zelo52C4tuJJa+3PFl7ERQie46SyT7H4/7ZbV zqDnrgNDBNiypgmj/m8qAFaG4co9EWJU8M5nO19svB/co2tMebB/LScqLCRKsqNMcDAftr 6z5IaQNNy7nlK5XyacPfmVZimyWkTn0= ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VZbNFKUCNvN7UywkgQFgOEyBdk5Ykv9D6ke62xcxUeHsPiY4ZVD3w03uYAGiNmwVckOd3scHX9svAH+mlZabVTJDH+/fxGpCwQtkm3CSLIDhaF1MS/Xm00zwCYYGFW4aHxTgz8xgdwthNBlLeXu/AxYWXfT2VLZJAyHEI6NuQNLB2EbDfqpES5cfhkxPYZ/cTzOxBqqUeTpWQf5s+zd8nsKB2y/jXdHkv+6KLUwbE5nyo4u1YGkhLJd5e1b3dfCmDwvW3DE19iXt3tS3zWxT+7TAtx+cWoFrDs/bFVqsD5xbyqHunmh1N1QfTj4YnveKcr7gGKV2TIMzsBk7GWhIcQ== 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=lFgf2S8ET76HACcqHN0EKVE7QG+sXKNnVNdN2ze0pRM=; b=bHvcQzJxe680qoCO5ibKyb+Nn8QZM2vXHKDkNwK9bb+OMyVX1PKbWas3ULc4xyWVqOuKiQU3Ps12DSmcCtx2ka1qqsHJJWmj55zJ5BtgIgNs2/1zDBCaOttdaJR/qVQYOU3dTnEPi6JrQwl16gp13bJbW5QZAnb/3lGRXU9N9esvMDaAsProVKvKfvUEeKexegzzEPlaQ7Fv8QGyb97wRHIoyiB4lsMZumOQAHcO92GeuT0Ip0M+Y0GIObuQ74pdO8CG/ELwYPNusv544O0V99hTsQjDWUP8hVX7Jb955I36m83eorYAMLDtWZboXZGvPVoBx7V8vjymws8fUL/5WQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=memverge.com; dmarc=pass action=none header.from=memverge.com; dkim=pass header.d=memverge.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=memverge.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=lFgf2S8ET76HACcqHN0EKVE7QG+sXKNnVNdN2ze0pRM=; b=aCJ9c8fRvdBOVhdUjA7XDE7AmNPzU5rCUaMHbD6lVfmzDwn3qI1l9BXIF1/nqniu4PZR9LmKUvu+Tw/9Sfk7OhlJje18yADRzi0AbSMqfeXFMpVunnr3l0Rz5CBNEojUBI57N+HkdXsaiBdrRzvryyDcpo5FHgloeAEFXpCDHT8= Received: from SJ0PR17MB5512.namprd17.prod.outlook.com (2603:10b6:a03:394::19) by LV3PR17MB7214.namprd17.prod.outlook.com (2603:10b6:408:1a4::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6954.17; Tue, 31 Oct 2023 17:02:38 +0000 Received: from SJ0PR17MB5512.namprd17.prod.outlook.com ([fe80::381c:7f11:1028:15f4]) by SJ0PR17MB5512.namprd17.prod.outlook.com ([fe80::381c:7f11:1028:15f4%4]) with mapi id 15.20.6954.015; Tue, 31 Oct 2023 17:02:38 +0000 Date: Tue, 31 Oct 2023 00:34:29 -0400 From: Gregory Price To: Matthew Wilcox Cc: Gregory Price , linux-mm@kvack.org, akpm@linux-foundation.org, david@redhat.com, vbabka@suse.cz, naoya.horiguchi@linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] kpageflags: respect folio head-page flag placement Message-ID: References: <20231030180005.2046-1-gregory.price@memverge.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: BY3PR03CA0021.namprd03.prod.outlook.com (2603:10b6:a03:39a::26) To SJ0PR17MB5512.namprd17.prod.outlook.com (2603:10b6:a03:394::19) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR17MB5512:EE_|LV3PR17MB7214:EE_ X-MS-Office365-Filtering-Correlation-Id: 910f2cf3-d8d4-4957-e1ce-08dbda332fca X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 65oHS9RaCaQTB7gMRCA0Fj0REtLsY2Q002/f+NP/cDSgIKkDy/uQ7ZoxcAYwC0HlFbh+WcekUmLOElHFLnzu5qXQgIF7P9HMOzY+UdEuXzuymunifXY6Q/DqLOo9YM4XgSriE7joASvdrmBZv7y/n99cxDXOj3RkbXmQnygZaFBDqCtFOQAn6sc2ZI1cYO6c4U24+Xtb+KyXCHCTajq3lvu97nLop4S5Z6s4sSGc9HYGkRJeR7trPZhgzGa4KdNAneqwxFJ/QJVfiId1mmOELTQ7UhU/lRkYSKtwZ64hiN4B73abHIjcoPTtEScMu0Y6FsoY9CvhTpS0eJY53FixqJ6+vyj+dCRyCDc3PttmVfM1uwE6Cxz5GnTKOeBkrMZxC4h1dVtSe6JW4fBydTFrjwaS9g8h37pJ238JWWCuQpL8wlwMQjkUD/NWdWYw884Toj4cWLcIQnSbSgI5LBwXR2ryQ99ITmkho4Cau4povqWSFa5LKUFTm5ft/FPSyOrNqZEsOc6LabkP1nqpo8uLeO+kpUx5Axwq7hxl4aaB6FpqvYBK3jwBxZf8/xKb1+LluK/vPGudwEieYCRlG61dkHAK+cIxbhmmdHfw0V486AFjFCr06a6SZP30fVvf7yB0 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ0PR17MB5512.namprd17.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(136003)(366004)(396003)(346002)(39840400004)(376002)(230273577357003)(230922051799003)(230173577357003)(186009)(1800799009)(64100799003)(451199024)(26005)(83380400001)(6506007)(6666004)(2616005)(6512007)(38100700002)(86362001)(66476007)(66946007)(66556008)(6916009)(36756003)(316002)(6486002)(478600001)(5660300002)(2906002)(8936002)(8676002)(4326008)(41300700001)(44832011);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?l1PxBFa28dKcznTdDYEIgYn+WFC2QMp26ZmsPp58n9ZPphc8xM5XVaoeOP4M?= =?us-ascii?Q?1qImVrPszwRKJ6t2wk8UuFuPLnZZghGSTBd1GxEfvdGA09I9K3A9YhX+JciG?= =?us-ascii?Q?gUlGc8T9VITGI19Boas7F/PyazR7lNY5QooVvWelhGFOiK3RMLjfoK5QDz84?= =?us-ascii?Q?Cr2zgNQacfRkNCt0olg3HpdMz6RgcR7BNjVN0S+luIZL6/Eo/K4WYRy8KTxe?= =?us-ascii?Q?Ne79jVA6xpFgHQog+A4LHjrwyUIKCQ0VfulSH5dU7tzpNxRx3DDDMspy0w8v?= =?us-ascii?Q?B2wkccfqsVt1bFw9CR3+zJXPgLgRZlo012DgOxq6Dq9PX0vyUBipaaFiif+F?= =?us-ascii?Q?CKgBwmNZ1pipj0aRZ924krYOh97aInaUep0h5r9lBuGSj8TKMTHGX064v3SL?= =?us-ascii?Q?/MSAjlGSw4PreaUus6s39wb6eYB3/oFdE5Nh8E83J96GMpEszIGvBekSJ323?= =?us-ascii?Q?H9C5GyJRFIh8TG/ARDlG7suNz9EWV9kbGbOFNBAtFgAUt5MVOkcWf3pdhVml?= =?us-ascii?Q?2rLbg8KQzJ7e7Z1oq1fcG5//wj3V7pdGtUs5kp1wJdo9XdOv3UVeCy4cobtp?= =?us-ascii?Q?dgDdTGK53TRz5g9PeBcLbvzZ8f41IrA1mGUCbVu+y9jmR52uyfubDbU2kHK2?= =?us-ascii?Q?fkfPK280SITbbl9eYuucgGRgU+synu+NCsbKXsxFUntTckXmxT8Cm9uMg+Ev?= =?us-ascii?Q?dUQ+ACTlSRWdU01c5m1/TYgyjsSh0kdAnv/LJB9HKvu7DxDc6W+d/mOqwdQC?= =?us-ascii?Q?bPcxPal11ZtH8N7yi1CPOqiV/R3l39dSuB2A7Xbm15im8o20YUsZ8b4//Ok6?= =?us-ascii?Q?rOrCJ7Z1HrZNiP3qu8OSYwcxQtsmO1RT+bfUxUeD03POKlB622YVcQxfa91i?= =?us-ascii?Q?87BsqKVekbNxCFKZFJtBE7diuA24RCHV3iwoZyCCS0ZHzh6ZLWRJ9wCl7OXh?= =?us-ascii?Q?QVlH6+qx+fy7+KYBYCJgQ1ciN8wyIpxB6FUl71ySL9WKBWkof7aFH+7oVtY4?= =?us-ascii?Q?1mkbtwav2+stOvR+irvPPE7grt8v61xp/PwQilSGElu0RsN71xKJ+uBtYMbJ?= =?us-ascii?Q?uoICRdXaN8MT6w5jPcFesh834mkecPaHoRGv7QVWnFjN20jlqmnYpwZkGPFW?= =?us-ascii?Q?HRMhieW+T5Dk1Ir5asN9uwgi/PzX88tPIpJxAueICTdKFUXzYNPWFwp8XY81?= =?us-ascii?Q?swpsctZ21PBLmNuzC2xIzxvw1BwOYOupihHO2E3lpSuZita1cYhsT/jG334F?= =?us-ascii?Q?dXDXSrbqVX/eCCJryJ+SrJbqyrBMK7TjwdU+q2JGG9EAuZmmUuQJjWtRA4Mh?= =?us-ascii?Q?ZVJcG90DaJXs/wMm0PBNpYpbt9LTddFfjQ/pHfAx9fHhywbWY8GR7sFeY+f2?= =?us-ascii?Q?dEUGQBV93mAs36MsefWtsmfUSk/Sq73E61C51dlfmy7ZaCubOh8jPMf1WWOY?= =?us-ascii?Q?aJ6XEi8HEqYLv2jY4ZJk5VdZhFT7zmKTPV4grAAMMZg0YSX8E2fV+c8b+fo3?= =?us-ascii?Q?5ZMWx086/u9XfUX5Ybw9IpbTGW2ZIisXEi4s2kEOMXG8SD7S2a2IT6duqzut?= =?us-ascii?Q?Dr5smo/DFZ20gR9UF+1gRm7U/DYbOgwqjspMmGK0EMkay1wWiI/YfrSpsAGY?= =?us-ascii?Q?zg=3D=3D?= X-OriginatorOrg: memverge.com X-MS-Exchange-CrossTenant-Network-Message-Id: 910f2cf3-d8d4-4957-e1ce-08dbda332fca X-MS-Exchange-CrossTenant-AuthSource: SJ0PR17MB5512.namprd17.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Oct 2023 17:02:38.5739 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 5c90cb59-37e7-4c81-9c07-00473d5fb682 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 7zVY5LF7w3ZO8uEyLwnDxn/FXVVeGuP5oXLUtGNHgSLkNzvESAWRz4pLzZej9pUZaxjDYSvoihNv70fB+WbPDrvxZErJE5JOzO2WaYEOxEI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV3PR17MB7214 X-Rspam-User: X-Stat-Signature: c4g1gsf7p8oshbanmjopmpc85nen1fsn X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: CCB1D1C002D X-HE-Tag: 1698771762-764936 X-HE-Meta: U2FsdGVkX1/jZxKi6GqkzFILfZK3z+byOiRr5bsxAf1ddYtxJy7cEvYK0jYviDl9lqG1unUi4QTi4oi/bKWhQpyo8CBvSWgr7gRqRHkafjDrp6VPgxBZGqmmumWUYoYYhraMnVtx2VXG609q7fW7xsDMTJCayrHK1Nv6A6dKqDmPJ4pQcoPusigSAnb8FDnmUEihtgo3+OP5abY7MQcVtV1FEnIrVnf5zzlOuZzePP6wf5Y1K9DweF1wXGc1RYWz+7OYAnD6JjnT/W7x0NDpwgpOwQWW4kWv7oLFBrSmypSreM9nxcAP0pkL1OZl6cWL4LlxnoJu4jUP9ZnsZF5EpETPE7UrwJwAfewOS6fNEqzcrgicGijpV7JSJDPI/CHoGAwWxV38sVI250qXZL21SEYQWYJyA4V8vM7ykErRJj0saPHWS38sQYT2vkp3r8gesP+7GYNlow01zDxhO4euaeXw7+oxp1kgrdasbKJ2lijOHChWHrD7Dza4PG9+0nbevGmYEeYVSIqjyBOwo6aTlFwMpPSCWdJZK5iGnPLuYnt6nQ3ENcC2Xoy3hXNAovoPe5bxtdxD+XPGXg0qqm+6r+Sgz1k4wJKI2ESM4ZPSepyiZh6J90as9zjIH5ZiBJxhs75dFxcZcjeA3nkqlOljKWYtvt9JMr7UDKVY71+9jBZmHzRyOyeMOFuBQgZEzLai1/CpiiM9W/RZbNrn7PHI6AvrPBH+TIuZC8jHNnpwo9pZwMBAzkXSO6cszcLNbyZDNIj2NC0+U6x2QsCROWg9ptx2oKeA3Cq7lfKR9lzhKWAwcsFIJLQ6bkUD6iAc3Xlm0ERL7sT06OrEUTH0Sg5wwWf8FzeBUnDc5RaiyEeHu+Ano3Z9j2JmF7Hf2D1dfjWnusoR63A1UQvLzfLt3zHKeY2WqxWe+O1V7YzcuuP/gp1nH9ouNcc5O/rzcGM/gC/yDPMA53Cpf0IjDby9KQH od0w3oLd qwQuMS2twfIDX3F877uCMCRjsA9Oocl9zOXDY5MUdxPE3dGX6d5VZHiOF+qzbtaqTpVn42h7fdqnDhhn+aXfZCMVJs4az8mXWqxLKOYXH+U7qHyUgcBV7P4KzqFKkoXchGxyLxDJZ2lobCb5vSZXe5hdiR2YWKW6Zx7246fA3s69AZdWyKaikuxlSvttbKvzC5FT8+lwBwL2DEj7Tqgey5pjWbgZL4S3l6foc3VIEGoUTJ4hbrABbOIc/6HSmwS9H4E874AG977xc7+epiSgznNSVib0S0jCd8G3Z X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Tue, Oct 31, 2023 at 09:13:44AM +0000, Matthew Wilcox wrote: > On Mon, Oct 30, 2023 at 07:41:23PM -0400, Gregory Price wrote: > > On Mon, Oct 30, 2023 at 11:22:18PM +0000, Matthew Wilcox wrote: > > > On Mon, Oct 30, 2023 at 02:00:05PM -0400, Gregory Price wrote: > > > > kpageflags reads page-flags directly from the page, even when the > > > > respective flag is only updated on the headpage of a folio. > > > > > > > > Update bitchecks to use PAGEFLAG() interfaces to check folio for the > > > > referenced, dirty, lru, active, and unevictable bits. > > > > > > But uptodate, writeback and reclaim (amongst others) are also defined > > > only on the head page. > > > > > > > Ah yes i was only looking at the things defined w/ PAGEFLAG defines in > > page-flags.h. I'll give it full once over can collect them all, my bad. > > > > (also i forgot to update my commit message) > > > > Quick question here since i have your attention: any recommendation on > > what to do for ONLY_HEAD flags? If the provided page is not the head, > > should the flag report 0... or whatever the head says? > > Thinking about it some more, really almost all flags are per-folio, not > per-page. The only exceptions are HWPoison and AnonExclusive. So > probably the right way to do this is to make k = folio->flags, and > then just change a few places rather than changing all the places that > test 'k'. Funny enough that's what i originally did but was confident it was correct so walked it back. I'll take another crack at it. ~Gregory