Received-SPF: pass (google.com: domain of linux-kernel+bounces-157101-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Wed, 24 Apr 2024 10:40:40 -0400
From: "Liam R. Howlett" <Liam.Howlett@oracle.com>
To: Andy Shevchenko <andriy.shevchenko@intel.com>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>, linux-kernel@vger.kernel.org,
        Willy Tarreau <w@1wt.eu>,
        Thomas =?utf-8?Q?Wei=C3=9Fschuh?= <linux@weissschuh.net>,
        Andrew Morton <akpm@linux-foundation.org>, willy@infradead.org,
        Namhyung Kim <namhyung@kernel.org>
Subject: Re: limits.h in tools/
Message-ID: <whrf4yfvi5oea2kw7mw6rjmuxh4vg45wi7ucg7j4iowufp2tav@mnqgaq6lpd64>
References: <ZigEvxn0-70og0H1@smile.fi.intel.com>
 <ZigKWZKY6kq-w0Tp@x1>
 <ZigMG8ZsBryO57zz@smile.fi.intel.com>
 <ZigMa2GqF_HN39oz@smile.fi.intel.com>
 <t2n5aj7orslsewknx6vmlf7t4wbnj3u6cabrjjfaptexb64fzn@rbjhcj3uwrkg>
 <ZikU5TWvL7zEfr5B@smile.fi.intel.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <ZikU5TWvL7zEfr5B@smile.fi.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: 
	=?utf-8?B?TUl1QWdqT2hSR3R2TC96OEpXUm5LK2ZiUlpoYWN2aEVEWTl1WnJKRGVUOWdE?=
 =?utf-8?B?VXhTNzJNMldzeEFYS2psMUhsSlJxTUFDTEZBWVpzVmxRZ2YxWWFiOW9qaS91?=
 =?utf-8?B?MEpvL3ZUaGNjNUVzOVNoRFRMN0RndTBGN2I3SnQ3dHBxZEo5d29NYkJacDRi?=
 =?utf-8?B?K1dHWTZNUjROTzlKMFY1bzNEWFVBUmM3VVNiT3h5WG9sTHh4R3h2WVlZakpQ?=
 =?utf-8?B?Z2V3djg3RzljeU91TEJteGRnWnVEUXB6bEtmeVUwY1laOTdDL0dyQkl2ZS8v?=
 =?utf-8?B?WUc2UmRjZEZTTHJTSEVJV2l1elNGeGV5d3NJSmQ0cjJqYXZGUFh4d1RXakhQ?=
 =?utf-8?B?ZDM4L05RUU5hcnFyTzhvZjF1OFlrQlFhVWtYNlpVMXZhaEZjUm5pcDIreHhi?=
 =?utf-8?B?VUpXdWU0RTFxR0xBUVRlNUtXV3cwRWpZbFRFWWlwempDdUllYkxnOTRVZVJ5?=
 =?utf-8?B?TWtKK0sxNGhFc0FFalR0Vm1NeEhXcXdFZDI3SUxobUhTVkVrcUNZNTh3RVAw?=
 =?utf-8?B?YzNrV2FCTTY5VW5xenRIVUxlV0RpRm5PSmtGdWVUZlZmN2tKNXdYMXR1SmJO?=
 =?utf-8?B?NTIyRWM1a1NFK2VsU1NnZ0VNcWxiVW42NW95L3pWbnEzVFIvSzZqeDRObzl2?=
 =?utf-8?B?MTdSaXgzeDA5TkZxSFhpQlMzNkxhZm1PMkdad0lxY280QmZ5VjZKTUI2N1ln?=
 =?utf-8?B?TmMrYjdDb2pESUdjSkgwekFuVS9zbThBM3JGMlc1cWF2ZzJYWFdrdm5CUWgw?=
 =?utf-8?B?eTA2NGRqVWNORU14akVXNUNtZTNCdVVMMHpnYVBicEJySkRMVEsxMUFyMHFk?=
 =?utf-8?B?VDFja3FwMzJMd1c4eVpJQWRiell0SU45L0xseGlVZytTOUs1dGRsWFJ3eTcx?=
 =?utf-8?B?RVNLb0doUVNIeUNEYkRYUVJnc0FOa1NENFhNUk9qL2dZVVZGZzl3elVaREFG?=
 =?utf-8?B?ZkswbFRCcGV0Y2UzendWSFFsOTNwSkVRZXlrUmV0aHZVdzdRWUF2UTlZb3dS?=
 =?utf-8?B?YmhVVVplU1NrdlpaOXpPYXZZbmFocTQ1Qy94dVlhVDFvNUVzck91enVsUHdO?=
 =?utf-8?B?bTFEZlhaWkxtQm5iRFRXSENUL2E1OXpOZ29EZWdkbW94NWhvVllUMWxIb2JH?=
 =?utf-8?B?N1AwYTBiN09kR2F6SFdqWWt3S1ZpeVJHMlhCRFJDNWtYUjlvSFRzcFR3VklR?=
 =?utf-8?B?QlZvLzJ1R2sybDV0NitEMXJENHlnNzI2NkJwNy83eUV3VzRRejRQZWhHT1B6?=
 =?utf-8?B?R25wMnIraDBSVnp6Sm9lckhtdWNmQnlxSzNkTTlURGNwbFJOZjkvT1VrUW1w?=
 =?utf-8?B?blNaZG9BNkxaa2VtRHo0aE8zYzRNUGdkQ05KQmxDSWtoZFR5TStaNDNCSk5D?=
 =?utf-8?B?VDQxVlN2WkxDdHlyVjlpVkZaVzlxM3VqQlhhRVR4SFlHMVIrY0gxZWNIcFZV?=
 =?utf-8?B?Nit0SUhocmZUbmV1NFNLTXQ3aWdaMlRyQU1rRktHUXlxdHNldW5LSmxpcUVU?=
 =?utf-8?B?cm5CcXdDcUpMbjNxaTlwSGQ2U2lXTGE0QzFaV2VDQVVEOWJ1Q1ZRZHBwclZo?=
 =?utf-8?B?RXZtVHVwLzhmWElaeFpzSEJKOXZpRXZtMm1wYnczUExxRE95UFo5VDFhQlA3?=
 =?utf-8?B?RTd6elZ0ZUI3ZUYvWGlPRzZyZG1Ycys3cmVlbnJUSzlwdXh0VXNrMEhlTTRz?=
 =?utf-8?B?QTJFUTArUmsxVTVyTTl3TEdjL1lONnAyWWRkWXdENE91MS9jMjFNazQ0ekVr?=
 =?utf-8?B?clpGY1BpQm9QSGtRdVh6UUdYODV6dWxINjZQb05nbTdpSUVWU2o3anRHaUFS?=
 =?utf-8?B?bVh1ZXdQU3EvMWFsMDI1SzdsNk1ScVFGQTFqNzBJSVk1c0JzTGRZYlQ5cndS?=
 =?utf-8?B?R2xUeHp1VWtWKysyQThISEZnbEVvTDdxRHRMODJBZERBUzh6K091TXZtSUN4?=
 =?utf-8?B?VDRyMElORmZtUmdkeE51dkFtdTRuVWl0ZzJUTUYyUXFHZUg4RERCZ2s1dVE1?=
 =?utf-8?B?M2x0bDU3emFiakFoSUhaL3V6a0pIeW5hTEhSOTJDY2U3UVRELzFZRWZyQ2o2?=
 =?utf-8?B?NXZReUc4K3FsL3hGbGZLWHRHZWJ6WWhic0hJaVFSL2dOOTlQZXhyVXVzbWFI?=
 =?utf-8?B?dW14STJXNFByQ1NrRmVmVG5UMmw0SC9VZkNBdnlRMFBGNUJRNWgwUzVTaVNh?=
 =?utf-8?B?R0E9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	yGyM2nwafJaUXudvJtF07Y1CFl+8oRhFWDQdoYL+CWWvqX/X+bhvuSwWz6JeH2A44vAdRQgaw1QP3vmZN+/N3v6VoQGc5FogwhXrvM1UimV0tZYJYSPO9GEf3ZkogeHy6nBz7dB1ysmyO2G+fbtH7/tlE6nSen4/soHShX/MDSltn2aBlniN6AhAzbI2CeHRdCAOb/WGTrWxkHnLxH8H4k+z04UUcqGjAuDi98Yol+t63756lyzkRpspvEIS/77tQuCO/7j4m2fJpQY1usRYx2eAg+4VN1iXyCvCCVn8bMgttuTmS9Y0CaMWfzZwSDYd973vB4uCpc6nQMI7P0HIgGhvb455YzplGBQqmv9Liu2Q9ftN1hKvY1dPL5RZvbXAidPtbZp4T8OmITq5C3mpwlp6HPfGqfV1GKDd8h94BsxOp9E6Gga140MDDiRdDnHRKkE3vFoeJDimMh1aC4/DPIzpKfH5aKxKc0Oy00LkCApPbsDOpKLZZyyZX5rEPZvLaMoOzLJQnoHlBc06Sl8LLzp070ZjsAAlrHVr3DQrjrdZMuxZ/64/q089iigbhzFDVX53IamC4IPzri2d2ZQ5Vphl/Z4BA0ACC1/CDZQ1VLU=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2604dcf3-4845-4628-cf52-08dc646c847b
X-MS-Exchange-CrossTenant-AuthSource: DS0PR10MB7933.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Apr 2024 14:40:42.2905
 (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: HAObFf5cLRnTSDr36ZozPIU8blFTxqa1tvSY+/YxJMLa/2i3q+SnH0MC0BYDx6tkz+c5pd6lC2YX6D7k4EjinQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4588
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1011,Hydra:6.0.650,FMLib:17.11.176.26
 definitions=2024-04-24_11,2024-04-24_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 mlxlogscore=999
 mlxscore=0 phishscore=0 spamscore=0 malwarescore=0 adultscore=0
 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2404010000 definitions=main-2404240054
X-Proofpoint-GUID: fgWsdVPJY1f2ASMYuQNeVuv_TmcxGSgt
X-Proofpoint-ORIG-GUID: fgWsdVPJY1f2ASMYuQNeVuv_TmcxGSgt

* Andy Shevchenko <andriy.shevchenko@intel.com> [240424 10:19]:
..

> > > On Tue, Apr 23, 2024 at 10:29:31PM +0300, Andy Shevchenko wrote:
> > > > On Tue, Apr 23, 2024 at 04:22:01PM -0300, Arnaldo Carvalho de Melo =
wrote:
> > > > > On Tue, Apr 23, 2024 at 09:58:07PM +0300, Andy Shevchenko wrote:
> > > > > > It seems tons of the code in tools include linux/limits.h. But =
I haven't found
> > > > > > where do we copy it. Any pointers?
> > > > > >=20
> > > > > > Based on the discussion 20220603183231.15159C385A9@smtp.kernel.=
org.
> >=20
> > I was unable to locate this discussion.
>=20
>   `b4 mbox 20220603183231.15159C385A9@smtp.kernel.org`
>=20
> works for me
>=20
> $ b4 mbox 20220603183231.15159C385A9@smtp.kernel.org
> Grabbing thread from lore.kernel.org/all/20220603183231.15159C385A9@smtp.=
kernel.org/t.mbox.gz
> 8 messages in the thread
> Saved ./20220603183231.15159C385A9@smtp.kernel.org.mbx

Interesting, I usually use the web search for the id and that failed me
in this case.  I use b4 for patch fetching and reviewing.

>=20
> > > > > =E2=AC=A2[acme@toolbox perf-tools-next]$ diff -u /usr/include/lin=
ux/limits.h include/uapi/linux/limits.h=20
> > > > > --- /usr/include/linux/limits.h	2024-01-31 21:00:00.000000000 -03=
00
> > > > > +++ include/uapi/linux/limits.h	2024-02-03 16:18:02.652000641 -03=
00
> > > > > @@ -1,6 +1,6 @@
> > > > >  /* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
> > > > > -#ifndef _LINUX_LIMITS_H
> > > > > -#define _LINUX_LIMITS_H
> > > > > +#ifndef _UAPI_LINUX_LIMITS_H
> > > > > +#define _UAPI_LINUX_LIMITS_H
> > > > > =20
> > > > >  #define NR_OPEN	        1024
> > > > > =20
> > > > > =E2=AC=A2[acme@toolbox perf-tools-next]$
> > > > >=20
> > > > > And in the places where I test build perf I saw no problem so far=
, the
> > > > > build failures below are unrelated, but still outstanding, see be=
low.
> > > > >=20
> > > > > But then is for building tools/, not the kernel, right? The discu=
ssion
> > > > > you said this question was based on is about changing
> > > > > include/linux/xarray.h, a kernel file, so should really be includ=
ing
> > > > > just kernel headers, files in the kernel source tree, outside too=
ls/, I
> > > > > don't see where what tools/ uses to build is relevant here? Can y=
ou
> > > > > please elaborate?
> > > >=20
> > > > I believe the tool in question is tools/testing/radix-tree/xarray.c=
.
> >=20
> > I'm not sure what is being asked.  Are you suggesting that the linux
> > kernel xarray.h header is including non-kernel headers?  I don't believ=
e
> > this to be true.
>=20
> No. The problem here that tooling relies basically on the external limits=
h.
> I.o.w. from the current user space.

Indeed this is the case.  I verified with an ifndef, error in main.c and
we seem to be using the non-uapi version.

maple.c also depends on external libraries (userspace rcu) being
installed, so I'm not entirely sure how isolated we need the testing to
be as they are not shipped or built besides by developers (and bots) to
test the functionality in userspace.

As for limits.h, we can probably migrate to the uapi version but I don't
really see a reason.

Thanks,
Liam