Received-SPF: pass (google.com: domain of linux-kernel+bounces-171770-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Tue, 7 May 2024 12:06:05 -0400
From: "Liam R. Howlett" <Liam.Howlett@oracle.com>
To: "Edgecombe, Rick P" <rick.p.edgecombe@intel.com>
Cc: "x86@kernel.org" <x86@kernel.org>,
        "keescook@chromium.org" <keescook@chromium.org>,
        "linux-sgx@vger.kernel.org" <linux-sgx@vger.kernel.org>,
        "luto@kernel.org" <luto@kernel.org>,
        "linux-cxl@vger.kernel.org" <linux-cxl@vger.kernel.org>,
        "dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
        "debug@rivosinc.com" <debug@rivosinc.com>,
        "io-uring@vger.kernel.org" <io-uring@vger.kernel.org>,
        "akpm@linux-foundation.org" <akpm@linux-foundation.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kirill.shutemov@linux.intel.com" <kirill.shutemov@linux.intel.com>,
        "mingo@redhat.com" <mingo@redhat.com>,
        "linux-s390@vger.kernel.org" <linux-s390@vger.kernel.org>,
        "christophe.leroy@csgroup.eu" <christophe.leroy@csgroup.eu>,
        "linux-mm@kvack.org" <linux-mm@kvack.org>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "sparclinux@vger.kernel.org" <sparclinux@vger.kernel.org>,
        "hpa@zytor.com" <hpa@zytor.com>,
        "bpf@vger.kernel.org" <bpf@vger.kernel.org>,
        "peterz@infradead.org" <peterz@infradead.org>,
        "nvdimm@lists.linux.dev" <nvdimm@lists.linux.dev>,
        "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
        "bp@alien8.de" <bp@alien8.de>,
        "Williams, Dan J" <dan.j.williams@intel.com>,
        "broonie@kernel.org" <broonie@kernel.org>
Subject: Re: [PATCH] mm: Remove mm argument from mm_get_unmapped_area()
Message-ID: <6luuf6tsdsrydtgzqpdggpsyyw7z4vcsv3q7gcvo5yufnrms5u@7we4w2mkhbcm>
Mail-Followup-To: "Liam R. Howlett" <Liam.Howlett@oracle.com>, 
	"Edgecombe, Rick P" <rick.p.edgecombe@intel.com>, "x86@kernel.org" <x86@kernel.org>, 
	"keescook@chromium.org" <keescook@chromium.org>, "linux-sgx@vger.kernel.org" <linux-sgx@vger.kernel.org>, 
	"luto@kernel.org" <luto@kernel.org>, "linux-cxl@vger.kernel.org" <linux-cxl@vger.kernel.org>, 
	"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>, "debug@rivosinc.com" <debug@rivosinc.com>, 
	"io-uring@vger.kernel.org" <io-uring@vger.kernel.org>, "akpm@linux-foundation.org" <akpm@linux-foundation.org>, 
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, 
	"kirill.shutemov@linux.intel.com" <kirill.shutemov@linux.intel.com>, "mingo@redhat.com" <mingo@redhat.com>, 
	"linux-s390@vger.kernel.org" <linux-s390@vger.kernel.org>, "christophe.leroy@csgroup.eu" <christophe.leroy@csgroup.eu>, 
	"linux-mm@kvack.org" <linux-mm@kvack.org>, "tglx@linutronix.de" <tglx@linutronix.de>, 
	"sparclinux@vger.kernel.org" <sparclinux@vger.kernel.org>, "hpa@zytor.com" <hpa@zytor.com>, 
	"bpf@vger.kernel.org" <bpf@vger.kernel.org>, "peterz@infradead.org" <peterz@infradead.org>, 
	"nvdimm@lists.linux.dev" <nvdimm@lists.linux.dev>, "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>, 
	"bp@alien8.de" <bp@alien8.de>, "Williams, Dan J" <dan.j.williams@intel.com>, 
	"broonie@kernel.org" <broonie@kernel.org>
References: <20240506160747.1321726-1-rick.p.edgecombe@intel.com>
 <tj2cc7k2fyeh2qi6hqkftxe2vk46rtjxaue222jkw3zcnxad4d@uark4ccqtx3t>
 <8811ab073c9d1f0c1dfdb04ae193e091839b4682.camel@intel.com>
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <8811ab073c9d1f0c1dfdb04ae193e091839b4682.camel@intel.com>
User-Agent: NeoMutt/20231103
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?iso-8859-1?Q?PgWQoAPUa8SnVWvxJbrP3PNfc4c7BN3VE5ySJFsL4/Y8dYKnc+pwm5+Ff2?=
 =?iso-8859-1?Q?mvItF7JCwFtvdh9D7CTvufjn3xZZDq0Qi1LB4cGMLOp9Lwuhj8jBcfzjnV?=
 =?iso-8859-1?Q?qlCZ2kDW883cp3e9cTogqS5XD0TJBZtZUh32Euk++o1nOGCrkI13C46fMa?=
 =?iso-8859-1?Q?d9hXOuziKdOLZPTqs7R7RMO3p6gKU+Sliu8by7DQO5nbTlOWyiTuK3OYj4?=
 =?iso-8859-1?Q?1Sne6rB/pbOd+CoCxZmB8lArnUDN/cJ2f2Dq4/A5x9eTB0cvbd3U1WXC0J?=
 =?iso-8859-1?Q?xhfmJKWVZEhuutOvDxbA9b3O9TUodu+C2BLV9evh6K/LtkWrNxD8UVeXut?=
 =?iso-8859-1?Q?do/nCv1ANiIZStRY+x70ZE2aDsIvmw4hrHtTcgJ67DG6RR5VD7+A1U1+oU?=
 =?iso-8859-1?Q?v7IApYuDvUsh3nuuKeUOqMytQDwZSVOClP6Vk5Mm86/B9KDq37cGaz7Ara?=
 =?iso-8859-1?Q?oQwCfEAMIAskLDLkr/kpq34zQP1lv2GuLipEtq1Kj5JoNIVZAgcWWM4al9?=
 =?iso-8859-1?Q?8R6dTzCLax70cW79A/b2XWQ7gzCE6pGOacbk7ZdGOE8GcO9VWzkRDEfdqB?=
 =?iso-8859-1?Q?NWT2zRhjDcrmk3wM+682ZySe9m74qYa4L1aMHxkf/B/0Q1uX18TnVdrYSe?=
 =?iso-8859-1?Q?z2T4Xg9TgmPSO2I6SXIs2BOv4I8uXr21rIkZmQDrr7ALdQrpnrO2lH6aPV?=
 =?iso-8859-1?Q?KwB3HQviSHOLPvVkKmsBVqy3ZeJ7Gup8mc8moQKQw4nNPPnhn8qKWrJ5Zd?=
 =?iso-8859-1?Q?ALYbheJZTqcL25YYAZFEfhtHqhGUFZDNW7kyezE9jOj5JAlQt5Aj5NB7CI?=
 =?iso-8859-1?Q?ud2nO2GlZwsYKFY+PGK9DlGdi2POTWBZOPSlP8f7ikZLY/DW8qJGDIzSgr?=
 =?iso-8859-1?Q?x2hV8OphUW9QrfbugcYbLUaaQPsIN0yCzrbRE+BtxNJOIVvowbCCMPmLHj?=
 =?iso-8859-1?Q?IWMJr0A0jo2UTSyiUOB5aHPDfyKK2+B3DwiM/1RfUm7IYwxPje0KvSUtXs?=
 =?iso-8859-1?Q?wPRHvd/Rkdr5qUpNuNZ7+YhBNXXvm1wCJmvtG1bCNlC3YDQ12vWf8/nnh+?=
 =?iso-8859-1?Q?xxDATCMWTnmr88VqMhSpnTJk727RjxbnTZwmJqdrs9xR78ZWJkBORjAHd7?=
 =?iso-8859-1?Q?Deor6jyKPKheNhp6bWtWghm4TEBmNMBTWEASQksZrbt7hYZ6Bhxf9aPZFD?=
 =?iso-8859-1?Q?fHabtowYDPe4NucNKEIijVz9e3QM8i6m31jVcCZ83p+vb2FjDxvWxXh8s6?=
 =?iso-8859-1?Q?0+SOAVUldYGRGJAXBxIVEhI7QOmCm4Pof8KHFkfyT+jGr2fU1QUNRgCEKD?=
 =?iso-8859-1?Q?xJataU/fZNLS7IucSaz2M9SXzTyjHE2y2KmH5Hu4RZVkFXKxi7YyK8pFeV?=
 =?iso-8859-1?Q?F5lm/4q7ctpgmJm7yiWUwn82/WKDK6ghUPBCIxXouZ8Ss6gsbHr64lsaIG?=
 =?iso-8859-1?Q?x4hzgtP6bzslewfNdxDYkaZgSnSi8jtInw4xvdjl2Zq1fBFCq3dA7v2WOn?=
 =?iso-8859-1?Q?erdHzbu+qrfrjzCWZE31ALRTn1fslD5t8iB9QZwJeWfpwkmujkprIBIUlh?=
 =?iso-8859-1?Q?MUKh0/gvHPxHJCCqZ6CCqZHYROzXjqedznlsd0Low3mYEHkvdrtcQEozAF?=
 =?iso-8859-1?Q?YHPYB0lc4ysMsbU/uPjxrJunuUKOBYAlVfAH/V9ZPOF0nZTryHEOPYwg?=
 =?iso-8859-1?Q?=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	ciAE2b1RrYt21IVjIgN6ARnOe73ZNv/Qr9RVN6JTeNXoXfZdhAeat8uwBxS3d9nrF5ndFVlXFfVCz8FolDS962+7c+cAJbVHYtgYZmgDcdABMi42p545KYfLK2L3b/AQUVjrfr3ZT8V7iVpiY7tqhBCD0iZYPn9gVDNwqp5bTnSC/I1z8yEss7OYrssZVgUdb9sk1mmMUbVoxbRbaujjMPmu6Ok9IWxvWoAuIGbJT8IxCkQsk/RaHqA7UlDIU5Zj4HpUMxdxlhH7zif5Gb9GjoJtICfjhM0ms4NnLGxiTRr//5IuRuSfs4hf5f9/sM3PPU9j9htjtL1huPg2mH0QYRQviVgVkyF/DIXXfOJFPfThWm3c3ULzB4NvJ26l/OdlkCPCKS3d7grIakbyYKW+urcx/k3b/2uuTgMhMBM4Hg6Q1gb1uleCe94d93oo9idOslECO2TKNIufXEXqeqtot0wKqgTlgQ4ELOe5wu15M+Eo2mf3mePyGbGRqsYgsTI6sKwpgAbysUz4IsbKadSP6cf7sNqPLDksImx7w1Tz/s6p5usCKM8aGzJUjdvDBodFNcoCOvd3QCm5r3l9CpGJsT7cagEipmYHsYs3nwEX6MM=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 36f30636-045a-497c-1e6c-08dc6eaf9b06
X-MS-Exchange-CrossTenant-AuthSource: DS0PR10MB7933.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 May 2024 16:06:08.1111
 (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: CnXJ7x8cMMoc6GFVKLy/QvqqFxud+x1Af/471R0A3pNB0vjRa7wpLCa3XL5o7ML8/td/qIZ2qYnl69Y4J3PllQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB7007
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.650,FMLib:17.11.176.26
 definitions=2024-05-07_09,2024-05-06_02,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 mlxlogscore=999
 bulkscore=0 suspectscore=0 spamscore=0 malwarescore=0 mlxscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2404010000 definitions=main-2405070111
X-Proofpoint-GUID: n59kw3vf6GcarDRHJAmCfClyFBwRPs11
X-Proofpoint-ORIG-GUID: n59kw3vf6GcarDRHJAmCfClyFBwRPs11

* Edgecombe, Rick P <rick.p.edgecombe@intel.com> [240507 09:51]:
> On Mon, 2024-05-06 at 12:32 -0400, Liam R. Howlett wrote:
> >=20
> > I like this patch.
>=20
> Thanks for taking a look.
>=20
> >=20
> > I think the context of current->mm is implied. IOW, could we call it
> > get_unmapped_area() instead?=A0 There are other functions today that us=
e
> > current->mm that don't start with current_<whatever>.=A0 I probably sho=
uld
> > have responded to Dan's suggestion with my comment.
>=20
> Yes, get_unmapped_area() is already taken. What else to call it... It is =
kind of
> the process "default" get_unmapped_area(). But with Christoph's proposal =
it
> would basically be arch_get_unmapped_area().

unmapped_area(), but that's also taken..

arch_get_unmapped_area() are all quite close.  If you look into it, many
of the arch versions were taken from the sparc 32 version.  Subsequent
changes were made and they are no longer exactly the same, but I believe
functionally equivalent - rather tricky to test though.

I wanted to unite these to simplify the mm code a while back, but have
not gotten back to it.  One aspect that some archs have is "cache
coloring" which does affect the VMAs.

The other difference is VDSO, which I may be looking into soon.  Someone
once called me a glutton for punishment and there may be some truth in
that...

Cheers,
Liam