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 ACAC6C4167B for ; Thu, 7 Dec 2023 15:43:31 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4CE4F6B0096; Thu, 7 Dec 2023 10:43:31 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 47D936B009B; Thu, 7 Dec 2023 10:43:31 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 345556B009C; Thu, 7 Dec 2023 10:43:31 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 27DDC6B0096 for ; Thu, 7 Dec 2023 10:43:31 -0500 (EST) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id D8C5FC011D for ; Thu, 7 Dec 2023 15:43:30 +0000 (UTC) X-FDA: 81540441780.06.F9C7787 Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by imf08.hostedemail.com (Postfix) with ESMTP id 8E9EF160019 for ; Thu, 7 Dec 2023 15:43:27 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=arndb.de header.s=fm1 header.b=enYVfsBH; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=h3uVDoUl; dmarc=pass (policy=none) header.from=arndb.de; spf=pass (imf08.hostedemail.com: domain of arnd@arndb.de designates 66.111.4.27 as permitted sender) smtp.mailfrom=arnd@arndb.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1701963807; 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=kKDmIidL/tnOKURnMGeoV9D0OqdgpEUnz7ab3Y6ER9I=; b=va9CRELYpwwfDMstKU+CpquGnHAguaMrWuIhF+NsLoIPq0WPi6f/NnQHNLkZAt1NgTx03r Lj/ZQ+vtW86TTdTAgU3eGADImBIBVaFZwSjLFFoz6ST+cB3epbtJUZm033TwJuDcEYZbSD +skytdT3DsezgwBDwpgrjVh2lt6lfSo= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=pass header.d=arndb.de header.s=fm1 header.b=enYVfsBH; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=h3uVDoUl; dmarc=pass (policy=none) header.from=arndb.de; spf=pass (imf08.hostedemail.com: domain of arnd@arndb.de designates 66.111.4.27 as permitted sender) smtp.mailfrom=arnd@arndb.de ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1701963807; a=rsa-sha256; cv=none; b=vrAtAu9O0cVEj8Ty3C6PY+JMkJb3OMaHyRG0JHYttdEIZHLcn36moEfTdCfJtJlflxuo8Z F+pxDml8HZNnaJuXZAap7SAMsnTs8glYxN29UvSr1mI5o/+CV2pGKe9gJgEqcsQ/SlTuA8 FvhPbbfJyXYB21v/v9YAjd+OVNzfrMM= Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id E103B5C010F; Thu, 7 Dec 2023 10:43:26 -0500 (EST) Received: from imap51 ([10.202.2.101]) by compute5.internal (MEProxy); Thu, 07 Dec 2023 10:43:26 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm1; t=1701963806; x=1702050206; bh=kK DmIidL/tnOKURnMGeoV9D0OqdgpEUnz7ab3Y6ER9I=; b=enYVfsBH/ajHQbRk+Y 748qP1SgSEffOJ5hAXxyjXL79fyI6TSkDBSLYrNCmcVCq9v5FqtoqXQwOw5lasHn FdjfPgvi7JpLTfg/u5bs72AFRKuCXSYkzrte+625THB8YGyrU0JihPN0miNheqoS D0NH44LxD7N/1sobNw0Aqy1s6ZHn/UN0gbmPjh9mm7d5+vbrOSLLAiLUK0S0P4rf 34jHD8JOJaD+kmA/JtJS8HNws3R7fKWcZFWhqXHa5i0qhZKDLXx+5RDFDb7aZd6p SE7Vks6G0laiS/g2n6jj4Eq+j3nwhVdbsv3hWpdK6tutO7HMCyl1A6bJi8zSJbIV rYFA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; t=1701963806; x=1702050206; bh=kKDmIidL/tnOK URnMGeoV9D0OqdgpEUnz7ab3Y6ER9I=; b=h3uVDoUlmP2wNbOIFulx31xrQpwen CLulfcqV4LLs/ZfG09NAuLrxZ20DgU994sXssFbwQFZNC/gutGZcX4WrybKiHidS npwYhQNy0OBZ7+XdwKQh+l37P9khQTd03fIAZgpnBU9YnkzQ7my3EiMVcs/UckaZ zfibqKZf3DUibCygSZs3dSQmqWfZowxwFWsakCyLAUwzQ5KaAebIO9ZsDMafwrDx aoI+Q3fr1PTB3EkNdddVmXIk7K2I7r2gVEPZImHXO3Cetg68FaozmclIfX+VFBy4 vndBJwldaNRMEz1o/rDNBtt4Ojs6bmGByaTe+V15v5B8pYnx/V9404ysw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrudekfedgudelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtsehttdertderredtnecuhfhrohhmpedftehr nhguuceuvghrghhmrghnnhdfuceorghrnhgusegrrhhnuggsrdguvgeqnecuggftrfgrth htvghrnhepffehueegteeihfegtefhjefgtdeugfegjeelheejueethfefgeeghfektdek teffnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomheprg hrnhgusegrrhhnuggsrdguvg X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 3BDB4B60089; Thu, 7 Dec 2023 10:43:24 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-1178-geeaf0069a7-fm-20231114.001-geeaf0069 MIME-Version: 1.0 Message-Id: In-Reply-To: References: <20231207002759.51418-1-gregory.price@memverge.com> <20231207002759.51418-8-gregory.price@memverge.com> <67fab0f1-e326-4ad8-9def-4d2bd5489b33@app.fastmail.com> Date: Thu, 07 Dec 2023 16:43:03 +0100 From: "Arnd Bergmann" To: "Gregory Price" Cc: "Gregory Price" , linux-mm@kvack.org, jgroves@micron.com, ravis.opensrc@micron.com, sthanneeru@micron.com, emirakhur@micron.com, Hasan.Maruf@amd.com, linux-doc@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-api@vger.kernel.org, Linux-Arch , linux-kernel@vger.kernel.org, "Andrew Morton" , "Thomas Gleixner" , "Andy Lutomirski" , "Ingo Molnar" , "Borislav Petkov" , "Dave Hansen" , x86@kernel.org, "H. Peter Anvin" , "Michal Hocko" , "Tejun Heo" , ying.huang@intel.com, "Jonathan Corbet" , rakie.kim@sk.com, hyeongtak.ji@sk.com, honggyu.kim@sk.com, vtavarespetr@micron.com, "Peter Zijlstra" , "Frank van der Linden" Subject: Re: [RFC PATCH 07/11] mm/mempolicy: add userland mempolicy arg structure Content-Type: text/plain X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 8E9EF160019 X-Stat-Signature: uwadfarqmbikbaoxf7zuwwfju6n97mf3 X-HE-Tag: 1701963807-966928 X-HE-Meta: U2FsdGVkX1+N/BFTiaVG8bSnlwoKwJ9nXsEsyfaPF53hrGynBCPfhT9WlF06Kz2r9cfPT9+Aplkh8ZqbJZu94hBoP0KqS1LUMYn9WhabSwr2MDFSmn8Ecaw1DWOvQ6vSEoL94DyOzjSl8Hw2z45UsiDKkDxuIJDbIUBzh8kLd5NKgdybR1WD5cZJjza5YiHjEjaaJVhXgD/oJrvmEhN2XWgdYEO412rX3e1pKihyCWT12ll4dfx4HEyWr/nPu9BLswT6Af9voBEYl6iEKpDFOAkh5s4roJxDrBGyM2YVprcFIhG7hYkLZcnu2f4W5bVA5G8RtyZ4qpRMhSJFPqS6E3VbK0VcvlY/6dvcx5xFNj4JxP8cXMTRfIbfOCt64Ky8KSVukzBr8kW1A0ev2LZV9baTwIZWhL4zpJIOFPUG+m20BFA1dfDuObQb8gsh7MpGO0kH6PM9upnIGYtnkLus7kbB80gZURfC/kAfc+nh6U/XJIaIN65VjDv7h1WGUxk7/FQ+xd7Aqe7PnznmSvBgYzBRiybN01Dc8nCb5b5pm+3RRzYbuSMjqAv5V2NSGYJjk0e6dEPZ89/R1mCzoWFirY67y7T32nTcryc3a3DEQTsV7FJoUVGZVf4K0YthJ3AHLFZM+SQuwe9skbI7mYyaH3VnoPklV/4V3fv7QySrlly3fpWqBl4pbu8CgyUPckQR2GqYZO8L9qruoCBJmZIRm/UK2T8qvJlMBoz9NOqDV1+3vE/dG18yXSyTjBZIYTSJy+sM+aXRMuUrUB8KDxNhzsyiuQND3C3HdqIOZi9rGnqbpofUlHU9x6GwnWvX3F2D20YNATF2laV6pl9kPzSzjXogye/8xQN60OI7lmUBdNCAbKzwIbjgd3GkNoW0M7Lq75Qaqh3hPm9k1WkqNl02+wMreN5q3udgk9HX3MZqjU98SWom0Ica9KqG4z3wKMMpMYINxPoUMidwQKE2SHG FresEpxn qdeuLsGJVqOFTGimzgGn7NRb0c5PjM1GWks9cfCgK3g384HqGtG8zy042U+uSpsnEXse0axjFvPpa2Z39hiUJJVllmBxB2Ia1/zrgI0d8yJ6AXZ+LHizvr2u1Im84bqYWfuMrSS+1ZiiE60YtbtfmIJNM+Itifgfc5z1LKoBHFdnoE3p9yPIuBe6l2X+3BOaa5zVbDVN19JwpKh+gXTI391iCaQahuEG/d5qDyoNLavSfy6o= 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 Thu, Dec 7, 2023, at 15:58, Gregory Price wrote: > On Thu, Dec 07, 2023 at 08:13:22AM +0100, Arnd Bergmann wrote: >> On Thu, Dec 7, 2023, at 01:27, Gregory Price wrote: >> >> Aside from this, you should avoid holes in the data structure. >> On 64-bit architectures, the layout above has holes after >> policy_node and after addr_node. >> >> Arnd > > doh, clearly i didn't stop to think about alignment. Good eye. > I'll redo this with __u/s members and fix the holes. > > Didn't stop to think about compat pointers. I don't think the > u64_to_user_ptr pattern is offensive, so i'll make that change. > At least I don't see what the other options are beyond compat. Ok, sounds good. I see you already call wrappers for compat mode to convert iovec and nodemask layouts for the indirect pointers, and they look correct. If you wanted to do handle the compat syscalls using the same entry point, you could add the same kind of helper to copy the mempolicy args from user space with an optional conversion, but not having to do this is clearly easier. Arnd