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 DCF35D2A559 for ; Wed, 16 Oct 2024 20:00:29 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 52E156B0089; Wed, 16 Oct 2024 16:00:29 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4DD796B008A; Wed, 16 Oct 2024 16:00:29 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 330B56B008C; Wed, 16 Oct 2024 16:00:29 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 0B7276B0089 for ; Wed, 16 Oct 2024 16:00:29 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id C6AC280AFE for ; Wed, 16 Oct 2024 20:00:19 +0000 (UTC) X-FDA: 82680532158.19.B6734DF Received: from mx0b-00069f02.pphosted.com (mx0b-00069f02.pphosted.com [205.220.177.32]) by imf02.hostedemail.com (Postfix) with ESMTP id 06C6480006 for ; Wed, 16 Oct 2024 20:00:07 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=ASdb4gVT; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=p0OAdhVB; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf02.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.177.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com; dmarc=pass (policy=reject) header.from=oracle.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1729108680; 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=ChEUAvslQT82XQhpA6VkV3t+NAdEM5VW0BhjpuzoClU=; b=4ahddF2HHhM+de/OT/cPcwTpRaEk1CTBIDs/vklgCb4h1BVBoBaAdKGKSOWFHgXQaSpZMJ SUgW1DEPi1p/vyDkMaI/vW5vOHWpCHsNaynVJqAJan2fd3hc1sL2S5OtPCOChGUflxsHtq I7SZ8Wc6w9+Zl+pC6pjXPMZAdFaan1w= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1729108680; a=rsa-sha256; cv=pass; b=IHO7k6uAB9SFTFGeMUxwr/oNcvINer9SNDP2BwjnunGGOEqwtqbRrz+YjUjfJcAl6fv+Gn +lErMYC7D6+OGs8SKa/B+NzlufvKbfRhJCZklN/PjMR0HISh72iU7yneP66h2SV+vDJY4T sfythOVrNcBE5Yvp1twswoX1XP4mIqs= ARC-Authentication-Results: i=2; imf02.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=ASdb4gVT; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=p0OAdhVB; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf02.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.177.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com; dmarc=pass (policy=reject) header.from=oracle.com Received: from pps.filterd (m0246631.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 49GHte6C004874; Wed, 16 Oct 2024 20:00:12 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=cc :content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=corp-2023-11-20; bh=ChEUAvslQT82XQhpA6 VkV3t+NAdEM5VW0BhjpuzoClU=; b=ASdb4gVTen/Pl/LuiDrvNYy6Ta60xSPDn0 R8VF+xV96AYEOlJbUM6GrxUvXf64lMX61mRN9dmelwB5sarjv28gfDSR5JRM0YBh oUyW8CsVI/t3PXa3Eg8S0j1sqAhcLmYYn24L73a4E67Jqy1fOTRTkpolnq3Fa+uA v+9ocG5SM7z3dnQrfGvbFZKNyiWTWKWNrLVk30pfemU03GYWtKF70exXyuwfRfcg uXRSGtNyskdsCmI54iU8ypwOF6pO6WIzXdbMwWP0XmqXTxlVBH7dMNJcbDrGAiaR A1LybB66vxsuIj0xFsMjHfNyUcPx3hAEqbKI+z1N4WZUlXrod1LA== Received: from phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com (phxpaimrmta02.appoci.oracle.com [147.154.114.232]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 427fw2mvte-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 16 Oct 2024 20:00:11 +0000 (GMT) Received: from pps.filterd (phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com [127.0.0.1]) by phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com (8.18.1.2/8.18.1.2) with ESMTP id 49GJ3tZm019983; Wed, 16 Oct 2024 20:00:10 GMT Received: from nam12-dm6-obe.outbound.protection.outlook.com (mail-dm6nam12lp2176.outbound.protection.outlook.com [104.47.59.176]) by phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com (PPS) with ESMTPS id 427fj9arja-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 16 Oct 2024 20:00:10 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=nQEpFLV5puoj1TIFKWcCIZZBySBwFax/Y7KbDFutbWTek0ZIj+r318uPLBFqAMq0Crmpg7nDgqOjCDNy1efNN39QR0Ghy0UJoYrfPKivUQ6fagS+0AtGbEEt8trOUpCx2TZKUUMFMzwAgs9p0KCviKsbZP6s9pRDUoP4sxAbs9JXaM8zJEd/Hj4yJqckq/N99fAWnj8529YuBTegPBsvSrvY6odjAFHJdrgc8xVqAF2HdxAG3DGOgw+cZSLIGS0XhLWMMyi+g/8zAyAxEjIy/PZGlBrqYkPI8lxVx31my987/nwdx2pVJUCPiLm8YZZJL2wPGJ5P1NS5YJoiv2YiwA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=ChEUAvslQT82XQhpA6VkV3t+NAdEM5VW0BhjpuzoClU=; b=EVjPAqYNUcl3LiVlypeULwAJ/HSV1gE/2F/2WpSVp17fneOLs9dMPWJYHEm9Cy7uJGs7bi7hjwme9Qhe+2hY1ENqZZt4x8+NZsbo/ezVGJPxCPCrnC5PeqkSrP6OtxNfHqJ5u8QMWCnRFY4TGQEwx6gzI7Wf4QOO/5ZqtrEqr0IgtiZGYMhKTS3kzYnHZHfaV2d6fH3+1TEMQN9SyxlcggQ2MtZRFRZuQXjc/2nLT0ORfW68L4+wTasyXiG8prIXP9X4SLti9boOwWT+c41GFahhngUAb44v8JaxydOb1IG3BxVv/UJwbxW8hF7DCsOYhCbHgq76JjCGH3VwtBmFsA== 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=ChEUAvslQT82XQhpA6VkV3t+NAdEM5VW0BhjpuzoClU=; b=p0OAdhVByw36B+NQd19wN5UdJYlTGdc25mmuH0D+PZVtoh1xJhlTejCYRaseUAH2fxEGWV1jAmzV5lFVDkGF7kNdPtLTg3Tpui/8kCUlRpiEeEm+UNx5lyZt1HxHcCLF6o/MuWz7gm41yu1GBwlgBF/AjmWVnytap+P8OWLI5Fo= Received: from SJ0PR10MB5613.namprd10.prod.outlook.com (2603:10b6:a03:3d0::5) by CH0PR10MB4986.namprd10.prod.outlook.com (2603:10b6:610:c7::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8069.17; Wed, 16 Oct 2024 20:00:07 +0000 Received: from SJ0PR10MB5613.namprd10.prod.outlook.com ([fe80::4239:cf6f:9caa:940e]) by SJ0PR10MB5613.namprd10.prod.outlook.com ([fe80::4239:cf6f:9caa:940e%5]) with mapi id 15.20.8069.016; Wed, 16 Oct 2024 20:00:07 +0000 Date: Wed, 16 Oct 2024 21:00:03 +0100 From: Lorenzo Stoakes To: Christian Brauner Cc: Christian Brauner , Shuah Khan , "Liam R . Howlett" , Suren Baghdasaryan , Vlastimil Babka , pedro.falcato@gmail.com, linux-kselftest@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-api@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/3] pidfd: extend pidfd_get_pid() and de-duplicate pid lookup Message-ID: <9294a556-345e-4766-9b54-1a429bfb917e@lucifer.local> References: <8e7edaf2f648fb01a71def749f17f76c0502dee1.1728643714.git.lorenzo.stoakes@oracle.com> <20241016-beinbruch-zeltplatz-4bfdedca1ee8@brauner> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241016-beinbruch-zeltplatz-4bfdedca1ee8@brauner> X-ClientProxiedBy: LO4P265CA0303.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:391::14) To SJ0PR10MB5613.namprd10.prod.outlook.com (2603:10b6:a03:3d0::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR10MB5613:EE_|CH0PR10MB4986:EE_ X-MS-Office365-Filtering-Correlation-Id: 7ff2ea36-f579-45f4-4c79-08dcee1d2216 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|1800799024|10070799003; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?YQiWXbl+k24QIAXLLY3O83Gcq9zH7/lLHH7i0RtOC91EksJ8o1NfaVmAP/UW?= =?us-ascii?Q?ftKVAKnVy3JoqjDx8PgaIMDhevadnz4VxkKZj/5dah0Y33rfmpvjEGUa0nV1?= =?us-ascii?Q?D5kWu7aT5GvbY3+++u9qIBgUTWNTdtuDuVdVwGjvoMIwj8Gr0Sj8zxkrGY0c?= =?us-ascii?Q?bYWu4tTkDx0NnOoWmowwA7VpCVXkfIqVe7AHbKPEgvL1q417ClY3cpNtxuo3?= =?us-ascii?Q?bapYDvhnTNH3iFQRRnHaMmTSHBmIOAJ772jFhjz1fv1MQ//P9JuHd1qeCB67?= =?us-ascii?Q?MKPls1eeMHySIKdIWMS1upIQ2krjX7wzrs02R/UE+IWogepyRvneYTF+Z8/f?= =?us-ascii?Q?qp2XpsmYweTks0DHHvMF1PH70OvlDhbFaHEQhIi/Aq/GWTGVhX5CZHymSdxX?= =?us-ascii?Q?mS2LyePNnJrAHmEXzyRbeEcbqcQHShVR/egrXOLOxHwbOFQSlqRwktRd2arO?= =?us-ascii?Q?0XEu0EY2XulTr6QMlQyBRntT4MBPEH9xzFMvU5BIWdClaDXUarsvC1EICLh9?= =?us-ascii?Q?OHaaYYrwcZ5qkhETgRQ7mDRSB/xEVWcnvBb5IPeEzJ6lfQysEpwPEHaBl3Yl?= =?us-ascii?Q?zXiE1DpxiLdpNeMYvDBP0110gw3rJr+98EID4eRbz8t6JCDV9xkNE1C7D3xD?= =?us-ascii?Q?xV4awI+XKh+IIXkxQ/WqHgge+F2at0i7Hjdhk8g9n3822NfpENIccFlU0Unx?= =?us-ascii?Q?hieZE1Z2Hqkew2+nlS15+58W5UZ9EwymtX95p500NjuS2ybjlg50Myou4DqS?= =?us-ascii?Q?0FqKcZZrhKxx/fnBwnOkb250bHr2eBrQVxKL39HA8vFDXFRViv6hrWr8HGGe?= =?us-ascii?Q?Bsj3Ti+8gninOXu5yxIFVH59eIJVVhY/TKLAxX80ylT6q9iyC7LrbBKGfexR?= =?us-ascii?Q?WFOsXF3i/iFhsTImpRB7sb9N5tvVHEVSdawJOYvy9LDVw3uRGtBckDyGIkeF?= =?us-ascii?Q?KQS4hjPLnHKw+gGk9NOsCakvp6Dn3y+8kzXcMr96tIJdNdyZMsJt+qnJGYGP?= =?us-ascii?Q?DlE94t3AxjW6z63Py8R3CmwEWFoE/A3KbcpBsMTCzaq6MzA4cb7yqwwDUW0A?= =?us-ascii?Q?tuaXENVl0xsLPBLTrhglHWxsQyPug/XyzI0YDdrwOKJh9CbzZqsMXTBtS62+?= =?us-ascii?Q?e0cmiFRYGutT1hEhTDK+VKGXlTUreOWWy0Tj5732q6gzR7M/x74dCwk0YV1c?= =?us-ascii?Q?dRaCf0pniK55JafzRMjYgRxXtfMuiXX7JpRDS1cMlw0IyHYxaHy4TjYzSr1U?= =?us-ascii?Q?oPbrO3/iH3gseHxLSAfxK/4DmM0/Dj4zR8kWiXrkaw=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ0PR10MB5613.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(1800799024)(10070799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?V60CyV7VsT0dP3tojy+Rjb+yDxY8AlIwvvKn27SKcXxKs7M35AOl26V7BMPP?= =?us-ascii?Q?G1ElXT577+r+LIa8W19v2TLVqAJxikiSbbIPyoRIjQoeq7nrCAVRzouqrwDm?= =?us-ascii?Q?JBKf0xClAk2M+UaHVlA3deizE/HhWu+/C2rCJDhdzmSsjpHW5LUI2NYJzcJD?= =?us-ascii?Q?H8DnPoDj5rPHOY1OpnJJ0xzbcAYxHhnLuKBnLbFXR1/sWqype9QrsvokAck2?= =?us-ascii?Q?jYAEsGqgoAqxTaTssDO/QcFhvOi483GPd+8KurX0Hf9Sx+OxzIQPrukIvfP5?= =?us-ascii?Q?2lWXidnZaP0pvRUaI/uWijBKM7IO2krA41nst4ZdXODgLtiqPjIaLvp8btKf?= =?us-ascii?Q?on4yMD3GXqyuh8rc3BF76LYqZS+38LX/ea2//MrHk26A8mFlZFaI6QGfWuV7?= =?us-ascii?Q?fgEMAn5Mf9hbGpuP3sk4Z/icFWOOghFh/Ki5EF4MGH1Ldmvp490NNVZiQ/sE?= =?us-ascii?Q?rmm4wc4jbzHPpGDJuZQhT1Wcr5RVgsycGYxrh1pXiikZWdXJRpfAhWa6a/jX?= =?us-ascii?Q?t9van4Hd4z0Q37CkXFFU683iDi9n/g8A4FX8TqRn5tUB0xH8QygL3XgtWO16?= =?us-ascii?Q?Dbo/HYw4aoN4bhr495whPu1NUn3G7IY/HBZORuIiOrzyGLLU8yNG5FUa0FYz?= =?us-ascii?Q?3L4Uv4+iIp+TqFDwUcrvnL9ia5xFDPj4OV/URL8r+tIoqyBxqjwQe5V/2ifJ?= =?us-ascii?Q?qa6U4gins0H4X7l6uvoA7pHFQvtIht/yUqheNIr98GvZLhF6esE/Z8wUMpBz?= =?us-ascii?Q?r+84mG8Sn9goancOjAXeg9lK1bBr3uk0YTkNT6LcWrmPD/rHNwFDMA5YzbpX?= =?us-ascii?Q?evitFpabDenKivSgpf50aZrrhJsbo1p8ZSO0bMPzgWnw0xkCr3Znl0noiZCK?= =?us-ascii?Q?kAj7R/GwkMZxKDD7z/9vIR3pJny/gOZtxSAv9Qp7Dk5OJ48dZjRPzEvU03mk?= =?us-ascii?Q?aXhCsUCBO2n2qOyosbzVihqmia3Xh3x0A8jFbJvo76JXTh17uGPj4F8lI7HY?= =?us-ascii?Q?5R1PSmUpcF+C6MV+Ag1bAu8rlxAwHmC2Ha0pS5/dXD6ug2Fasvdl9muhNHZS?= =?us-ascii?Q?pl15ZuFugro0V5PXE1/fDGvXC7RrnBFXz/l30QLUz0h0P5bXBqdm8xmsjhPx?= =?us-ascii?Q?taCbHNirPqrUhFMg4puUbcg6xK6NaIBFiF2vUCsagDr9CB5FaAN7ulpQWBh5?= =?us-ascii?Q?HkHLfA3Nc/ol2Y7D84NvfbYc8a/f/kVb8IhXit5RCucR2P4SY5UQBuBg8a/M?= =?us-ascii?Q?WlTHJt9V0JjSzKrDQ3azBoQStUXgt5ik8+i7wwy7m6urhTWqD4Uhechc1+uz?= =?us-ascii?Q?gkjHGHUxHmHBSm8ySC1ddbRDX4+iiUWhxjjJsPjmt+UJcwXnDSdZ4cy+wPmT?= =?us-ascii?Q?PQ5B3Zdkt1hzI0uM4wAKPyA29Xmmia7g016mQjSGw/5MqKarLtWvMCf2fb4Q?= =?us-ascii?Q?cXHAWsZfFErN1icv19dC7+xEnTpURrr5Ae4BbKUH/dZZnIlikQsvaw46/9yP?= =?us-ascii?Q?AbqJw4aUCInj8zKxhJsscIxeaxKohXIK5L5D6S/xvMCAaXRippyZEKjTM6AH?= =?us-ascii?Q?N+z4P2Da/PO7z9gSZQsleCzioUQ8+ByrgLrkLPo6k1mpzYDtX5im2dVS1eMh?= =?us-ascii?Q?MwHRqyrkZEjxu7RH2Cy3P950/YTwgZ2kUx2+JJIfSs1q/qzIZqHD/sM4bbfc?= =?us-ascii?Q?bsiMvQ=3D=3D?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 0yzLvoef5SiBEM9p8c9LgANlaBvp3TCXqLXw64EKoQDesnAQrSToLkpuj7Vi4/MBdcbqvs5g2XnGiAoSgzRgkSQcltD/mH9yUaON0NeRxe5wwUAZY2UqasLZjkwsvktPfXRtRZUlQwJx3YHbUp+SEG+cfSgnRrsvB4AAUXMIuub7GyE3o4WXxjoV6HIqu482dgnpvLkixBtLAHDczF7AnAD9tupNnRcz5gfxWe06moGNqXoObz3peRC2zdL0U+JVnpsmN0WTRMSzqFYse2/2DY6k+5V77fONruKCSTrrKKeEPj+xHjaGPDbjr91OGduXcPi2vg+RofpYdEyhRP7TSIHPHqtvtO+XI6rhk6XEnhSFJZ0vSfwfTt3IARHtv82+yrzYtICgfic2GpuhD8XXbVwdFKdRTd2XJ9M8RT45NRZZhCKChfRLsAyGWLdh4UXSl/tum9dsFqd2O3wlIp5m+s+9OrOteXUNKwg1coV6EJ8ubqqaDMkosNIM6RqpHHmh+sISVut4fdfP/M4xeVJwNiQK81tT4NgtqUv+ozUI0hiyRPaWPly3VoDj/lzLXS59o/jF/SRMU6s0h3MRibn05AeznrttfLzGbwyylfTHU1k= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7ff2ea36-f579-45f4-4c79-08dcee1d2216 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB5613.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Oct 2024 20:00:07.5918 (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: NxDk8aIeIS1mDSpK4GimX+tvfxEdy4ajcutpzmNtp1V+FJZVpEg1qgXSMtuhZFwXPhob92xUMd1N/ookQDM8j6xLy0tAHInL2U6DHtzBJ5Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR10MB4986 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1051,Hydra:6.0.680,FMLib:17.12.62.30 definitions=2024-10-16_16,2024-10-16_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 mlxscore=0 phishscore=0 adultscore=0 bulkscore=0 suspectscore=0 spamscore=0 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2409260000 definitions=main-2410160126 X-Proofpoint-GUID: Rkh604OcEQphwIlsT625AIyDvYrjdCFv X-Proofpoint-ORIG-GUID: Rkh604OcEQphwIlsT625AIyDvYrjdCFv X-Stat-Signature: 5hd7p5od4ubs4sb7yifu9wwddqionq4o X-Rspamd-Queue-Id: 06C6480006 X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1729108807-63851 X-HE-Meta: U2FsdGVkX18b0HCvPosT1Zn6X5XtR0TGyM+2Cy3YPYsZFZP2MVm+kApXsKTmGumnpb++GZgJDT96mzm7yA5Q/cF9QmHFOfym2heqE8w1vuN0qSCuG/6XU6nTfq/NAhGsS+m+Kl6yAmTTXZl4wifzEJPZGxjWp7YIeNYNAM18QU3fZOFxFFdrXz0vBFdqDpiUPpTb0Q3peOxsJc5ohqhm20djLGzhmtDkfq/qhcY4nIOB6K5MXVtU1mZEkDtdoTuNPuoWcSzMc+Ov4U583oMvt8NIxlYQY3gdQNDpDGCd5KDSIyNSHvmun49A3GuFhdcKsjjb1nNQjoxj6hEtrUGwC3sBRFgux4qTq4MqcdOa9HYG1DPHUKDl7vSj7MhExZ7G5tY7++WHL+jZE0Hfv5d1BI5Wr2MlWFgIfBPRRdjxQV4qclflJ307Qf/zMh6MCAK2X1N8JIPXANI2kHRT3ron9lZ7LPyLb/osHAx0o6acuWnfEtgrGIA8l2yQYvNFFfMYdk4rI0Tn1R9Er9Sv1Tb3NEhsCcDUyHqCB0YvKGLYQFHMOlgGOPAD8I8br9gDrw4QIp2G6QLPDIzgvdxd56C7WdRax5Et+9AChkZiI9Rk6xRje325bRlTuBa+P0STfJ51rFibv/B6ltGBPjQKYyrxeGBEAdOoFpIFe0xP/msH05xnX4c72mptTrYyrU2CAYOYIpgwRYY3zrySInqFgegWM1dNtf9nRio1eUEiedsILGqdTBrRT0FSxQzr22nMx3ecCcQqbtZwnEXDJbUw5JB0xtg+CFwjA1DTKh9FxKhwxW0eUffnoi61tQ/q6TniLbRVtKmeSiImopEzxZc/R2q2ULsPxZsk+dGEOn/3ylq1ghfnTMNveyWnXPTToi7Qqu/W1YidbuhRkU8GHt6B7yUcGDhGhMjNGVM9cs1MAYVp3WCk7UWdIKq2F29eDyWUwJiKB5SBMEey7lG/8BjGbag JU7zpcah zKOMYSe2zBAYjmMu2RzEJQc5PT39bfeBZ4ijQBS7onc1XQZ8FCMywJbba4YWJI/lFkWhCtDEBZroqAcvXLnQmbrShS9WUvAvziaqpaQ9l6Qn6WZDYB3wRTABnMeiucuagB8IeCnGGbv20A2fAuZzRwWDpUuGCQfahxQ2jRosXuiT6YqC+V1Xx01xgX7ERjdXWXKcK0bToqP2DIy4n3G5JIszCTGCG0leNhyn6eejLDGDypEeQvjojAxWrwN15o25TWf3ZQOIMihQyibr31Zw+1aovVQC9vgjihGMMa4tyqSDWiGqMQPCwLmiZbeXRAPB9/wKvJ6xzJcfHNppCZxdEgOcPSU2hEInBIfUE5FX8XQLzshTp5UtlqUFFgY47X438Tq3+ElEwqypy9I7TP/L0pxPZQekLIiRy6wbiUeNO9YvYTi0i9cfKk3wqIaWopFDJ5t4cPC6wsElHN8CuqHwpRh0Uci4lKqpHYh6pViczZpX12s9ZPQ8FzBBi/4RfQeToWsK1uKSVD8GgJzZUQWE5BNv5CLvogttAJl7Q5dx9riLPFwnn3UzeaWlq+a6DKbn+fSKsh4xx5bZecZ2bkspC7CkKji4JEAq7GpgiRJu9rn/+J2czh0tYQjcBQGI1sUgrJrAWEVkkaVCyYgQ= 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 Wed, Oct 16, 2024 at 03:00:55PM +0200, Christian Brauner wrote: > On Fri, Oct 11, 2024 at 12:05:55PM +0100, Lorenzo Stoakes wrote: > > The means by which a pid is determined from a pidfd is duplicated, with > > some callers holding a reference to the (pid)fd, and others explicitly > > pinning the pid. > > > > Introduce __pidfd_get_pid() which abstracts both approaches and provide > > optional output parameters for file->f_flags and the fd (the latter of > > which, if provided, prevents the function from decrementing the fd's > > refernce count). > > > > Additionally, allow the ability to open a pidfd by opening a /proc/ > > directory, utilised by the pidfd_send_signal() system call, providing a > > pidfd_get_pid_proc() helper function to do so. > > > > Doing this allows us to eliminate open-coded pidfd pid lookup and to > > consistently handle this in one place. > > > > This lays the groundwork for a subsequent patch which adds a new sentinel > > pidfd to explicitly reference the current process (i.e. thread group > > leader) without the need for a pidfd. > > > > Signed-off-by: Lorenzo Stoakes > > --- > > include/linux/pid.h | 42 +++++++++++++++++++++++++++++++- > > kernel/pid.c | 58 ++++++++++++++++++++++++++++++--------------- > > kernel/signal.c | 22 ++++------------- > > 3 files changed, 84 insertions(+), 38 deletions(-) > > > > diff --git a/include/linux/pid.h b/include/linux/pid.h > > index a3aad9b4074c..68b02eab7509 100644 > > --- a/include/linux/pid.h > > +++ b/include/linux/pid.h > > @@ -2,6 +2,7 @@ > > #ifndef _LINUX_PID_H > > #define _LINUX_PID_H > > > > +#include > > #include > > #include > > #include > > @@ -72,8 +73,47 @@ extern struct pid init_struct_pid; > > > > struct file; > > > > + > > +/** > > + * __pidfd_get_pid() - Retrieve a pid associated with the specified pidfd. > > + * > > + * @pidfd: The pidfd whose pid we want, or the fd of a /proc/ file if > > + * @alloc_proc is also set. > > + * @pin_pid: If set, then the reference counter of the returned pid is > > + * incremented. If not set, then @fd should be provided to pin the > > + * pidfd. > > + * @allow_proc: If set, then an fd of a /proc/ file can be passed instead > > + * of a pidfd, and this will be used to determine the pid. > > + * @flags: Output variable, if non-NULL, then the file->f_flags of the > > + * pidfd will be set here. > > + * @fd: Output variable, if non-NULL, then the pidfd reference will > > + * remain elevated and the caller will need to decrement it > > + * themselves. > > + * > > + * Returns: If successful, the pid associated with the pidfd, otherwise an > > + * error. > > + */ > > +struct pid *__pidfd_get_pid(unsigned int pidfd, bool pin_pid, > > + bool allow_proc, unsigned int *flags, > > + struct fd *fd); > > + > > +static inline struct pid *pidfd_get_pid(unsigned int pidfd, unsigned int *flags) > > +{ > > + return __pidfd_get_pid(pidfd, /* pin_pid = */ true, > > + /* allow_proc = */ false, > > + flags, /* fd = */ NULL); > > +} > > + > > +static inline struct pid *pidfd_to_pid_proc(unsigned int pidfd, > > + unsigned int *flags, > > + struct fd *fd) > > +{ > > + return __pidfd_get_pid(pidfd, /* pin_pid = */ false, > > + /* allow_proc = */ true, > > + flags, fd); > > +} > > + > > struct pid *pidfd_pid(const struct file *file); > > -struct pid *pidfd_get_pid(unsigned int fd, unsigned int *flags); > > struct task_struct *pidfd_get_task(int pidfd, unsigned int *flags); > > int pidfd_prepare(struct pid *pid, unsigned int flags, struct file **ret); > > void do_notify_pidfd(struct task_struct *task); > > diff --git a/kernel/pid.c b/kernel/pid.c > > index 2715afb77eab..25cc1c36a1b1 100644 > > --- a/kernel/pid.c > > +++ b/kernel/pid.c > > @@ -36,6 +36,7 @@ > > #include > > #include > > #include > > +#include > > #include > > #include > > #include > > @@ -534,22 +535,46 @@ struct pid *find_ge_pid(int nr, struct pid_namespace *ns) > > } > > EXPORT_SYMBOL_GPL(find_ge_pid); > > > > -struct pid *pidfd_get_pid(unsigned int fd, unsigned int *flags) > > +struct pid *__pidfd_get_pid(unsigned int pidfd, bool pin_pid, > > + bool allow_proc, unsigned int *flags, > > + struct fd *fd) > > Hm, we should never return a struct fd. A struct fd is an inherently > scoped-bound concept - or at least aims to be. Simply put, we always > want to have the fdget() and the fdput() in the same scope as the file > pointer you can access via fd_file() is only valid as long as we're in > the syscall. > > Ideally we mostly use CLASS(fd/fd_raw) and nearly never fdget(). The > point is that this is the wrong api to expose. > > It would probably be wiser if you added a pidfd based fdget() inspired > primitive. I think we can actually probably just avoid passing it back and pin the pid instead of the fd, which keeps the scope as before and simplifies things generally. Let me experiment with that!