Received-SPF: pass (google.com: domain of linux-kernel+bounces-175702-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <7e23363b-747e-444e-9007-c806a6c5596e@nvidia.com>
Date: Fri, 10 May 2024 12:47:00 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] docs: document python version used for compilation
To: Mauro Carvalho Chehab <mchehab@kernel.org>,
 Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
 Stefan Dirsch <sndirsch@suse.de>
Cc: Jani Nikula <jani.nikula@intel.com>, Jonathan Corbet <corbet@lwn.net>,
 Rob Clark <robdclark@gmail.com>, Abhinav Kumar <quic_abhinavk@quicinc.com>,
 Sean Paul <sean@poorly.run>, Marijn Suijten <marijn.suijten@somainline.org>,
 workflows@vger.kernel.org, linux-doc@vger.kernel.org,
 linux-kernel@vger.kernel.org, Masahiro Yamada <masahiroy@kernel.org>,
 linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
 freedreno@lists.freedesktop.org
References: <20240509-python-version-v1-1-a7dda3a95b5f@linaro.org>
 <878r0ijdh5.fsf@intel.com> <20240510102036.3ea40b37@sal.lan>
 <87o79ehtbz.fsf@intel.com>
 <CAA8EJprcjDnpsriXOrRO4cVh5Sm9KDbHbsyKePru=6pn0-bfJA@mail.gmail.com>
 <20240510123911.34f050b5@sal.lan>
From: Jon Hunter <jonathanh@nvidia.com>
Content-Language: en-US
In-Reply-To: <20240510123911.34f050b5@sal.lan>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?aGFqRHNPRHBybjBKbnB0QWpLTmZGRHZYTzRxQlFmNGxwZGxsL1pQUDN0L3pj?=
 =?utf-8?B?dzNGWHFDdytxTW5jdjFLWFRPUEhFY055eC9wM3JmV1lwZFlvQzlRTnErSER3?=
 =?utf-8?B?cFR5WFErd25KNmhaeVpCR2cyU0lDWGJ1M1htSGJHeW9ZOXdybkU4WWpOU2Rk?=
 =?utf-8?B?ZkRjMFNabHVJaHJRM1JZcWRTdFNBMWtjcGRjazVlY0ljaEhaTHNndm14TnBY?=
 =?utf-8?B?WmdBK0szb2prSTNQTWVRbXRPV2Fua3I5cDc4TXplRjFsWWFEY2RVUnlGdGI4?=
 =?utf-8?B?RWJxbjFVenV5bGkxY2g2ZHQ4NVdJTDg5ZmxHZWRYTFNnOWt1aHBnMUQ3dkU1?=
 =?utf-8?B?VnJRejEwbk5XRzl1OVdsOTN6ZU9nNlhaMXFESHVibEIyZWV5UHdOYW1hZ280?=
 =?utf-8?B?dFVGQjRyY2NkbUJMb012SHF3VTlPTzFUZExPZEQ4b01ENHp0UVVucWx3WWt4?=
 =?utf-8?B?akM1bzVpWnpmNlRidDNLWFlUZG9qNjlxdXFSRlNzRjVXZHI4YmYvajVZTnBL?=
 =?utf-8?B?MmlnbWNzOWRrdTdZRWtKMjRvdjl0NEk2VlV5MjkzUkxUcTNpQXBoODAwZm1Q?=
 =?utf-8?B?amk3OStpRGxBUk9mZjJocE41Q2tvejc5RnE0WW5TUkFlRW5UcTR5NjNrSzYx?=
 =?utf-8?B?U25qaytGQlRFMWJlK3hQSTQ3SWxPT2VwVDI4U0xoRUp3THNRc1JBcHUrdTNv?=
 =?utf-8?B?aVp6Nmc3a3dlbzhraGxtMTA5NEhqMFdWa2FkdmZpSE9Hako0SjEvSGFmbFdv?=
 =?utf-8?B?UFdHVWVGOFlEbkdhdWsrQjNTdTJxa0JvVWovWllRWitianVHSnJwblJ6eHBL?=
 =?utf-8?B?Y2pFZWtWQkZCZXZxMXl2cEUxNGFNaUR6bWN2WUsyVEdkSjZLNFREWUl6MkU0?=
 =?utf-8?B?WTJYMDZFblozblVCNTB4dVVNWS9nY3B4cCtXeklPVElOdFc0NmFFd0VMRXZi?=
 =?utf-8?B?YTZJeHdVM1lJdnViczhhQVl2b3AvUjNoUEl6azc0OVlRYzJhQm5FS1VEbWNM?=
 =?utf-8?B?dEo1VDhHeDZpMGdUWGYvUjdkM3FveWNGbThkVWxsa0pQa2tGWnZvWGFWM3pH?=
 =?utf-8?B?VXMxaFR2eDljcHZGeG43RWJVd0lsYy9nZHBhT1NqOExWN0dXVUttY1d6b0VU?=
 =?utf-8?B?L3N0dDFvY3JIY0ltcDNhQVJ1c0FOQ1hSZ3dSQ2ViV0M5b1dCR2hHRHRKRDMv?=
 =?utf-8?B?VDdYVWxiN3dTTWNZVDNVTUhRQnNNVWpnWThoUW5icnh5NG53V2VLN1NiTy9n?=
 =?utf-8?B?TkRKT1dqYTdWVGl3ek5jY25lY3dCemN6YVgyTFVuZksrWDFiZHVFMnh1ZnRO?=
 =?utf-8?B?WFdnWlczWi8wRUpsWVplTHRWTzJMRWVjLzh2c0Q3MkNLSUE5WS9ERkhUTnlM?=
 =?utf-8?B?dSt4eUlCMDV4YTN2aDJOZWNuWlFjT2tLZDhBbnlyNG0zbTZucUN5RW1nVTBQ?=
 =?utf-8?B?dDNCNWM1M3gxS0d2VzR2eTB5WWVBRjZMRWQzR25jQXpKb0VaSnVtN0ZJM2JN?=
 =?utf-8?B?VDlVQ24vZG4yMGZONnNDTk12R1VDNVdKaExBSGM2aWpvczN0eXFodUpobHdW?=
 =?utf-8?B?bnFkVDIzWkdIK2RrN0I2RXdnOVhTclJDbUlvRTNzUzFFMm5qTXJ0cnBMQmdZ?=
 =?utf-8?B?bHRWMkdMNy9QNWd0Ukgzdm1NZU1NaXk3eDZ2Tk11aE10K1h2UlIvU1YyRDhS?=
 =?utf-8?B?OGpweFZqVmFPZGNOQkwvd0VTRWV4a3Y0VDhYWkk2S2VSN3FYMnNzUXZKWGZB?=
 =?utf-8?B?Nm9NaExHM3RxSmlXbmJqdEhxQkgxUm03cHRKdGIvYWs1SjFxRjhvYU0vditX?=
 =?utf-8?B?YmZRMTlWOHZKb2kvajZIbUROdkEraS9aVVZIV2Fpdlp0dUVJZVFsUkNIRStk?=
 =?utf-8?B?dC92a2JsdjBCYzIyajdIcXJPWmJMOSt0R2FMYlIxV0hKNTNLL2t3M3JYRjhi?=
 =?utf-8?B?UVNOVm9wKzB2ekg2VFBlT0R2aVZ4SVdHaHBoYzBVSHlnNVlQak5saW1TYmd5?=
 =?utf-8?B?R1RGSTY4clhyS2dpcGFnSzBQZDZLQ3BzeFJBbWlJNjQ2SS9uY1Jka3RJL3V2?=
 =?utf-8?B?KzZFVmRhcEZYWERvTW1UejhnNzhMOUxvSmJweFJEWHVSdk5oRDNtV1g0V1c1?=
 =?utf-8?B?c29IdGFSemQ0V0hIdWJ1VmxRMnFsZ2lrT2RGczkrUk05akVIenV6bGFNTFlp?=
 =?utf-8?B?enp6MlVPN2JqTDhuUm50UXZqWkc0SDJOd3FDTm1jNHU1WVZHUWhRcUdRNlMr?=
 =?utf-8?B?WGwrWU9YVjkwdytiUmdTcG5nN2JRPT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 52fc51f5-c8d1-49c1-6d3c-08dc70e6eba8
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5444.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 May 2024 11:47:08.0289
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: MwWVbFoeON33a/iF7tZovnHT9V09GJtqipK1KG3EP7rJQvzjXRv5TQbLrnmwXWZU7OeJNliW0i+N62gHAvzwYw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB7687


On 10/05/2024 12:39, Mauro Carvalho Chehab wrote:
> Em Fri, 10 May 2024 13:39:17 +0300
> Dmitry Baryshkov <dmitry.baryshkov@linaro.org> escreveu:
> 
>> On Fri, 10 May 2024 at 13:09, Jani Nikula <jani.nikula@intel.com> wrote:
>>>
>>> On Fri, 10 May 2024, Mauro Carvalho Chehab <mchehab@kernel.org> wrote:
>>>> Em Fri, 10 May 2024 11:08:38 +0300
>>>> Jani Nikula <jani.nikula@intel.com> escreveu:
>>>>   
>>>>> On Thu, 09 May 2024, Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote:
>>>>>> The drm/msm driver had adopted using Python3 script to generate register
>>>>>> header files instead of shipping pre-generated header files. Document
>>>>>> the minimal Python version supported by the script.
>>>>>>
>>>>>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>>>>>> ---
>>>>>>   Documentation/process/changes.rst | 1 +
>>>>>>   1 file changed, 1 insertion(+)
>>>>>>
>>>>>> diff --git a/Documentation/process/changes.rst b/Documentation/process/changes.rst
>>>>>> index 5685d7bfe4d0..8d225a9f65a2 100644
>>>>>> --- a/Documentation/process/changes.rst
>>>>>> +++ b/Documentation/process/changes.rst
>>>>>> @@ -63,6 +63,7 @@ cpio                   any              cpio --version
>>>>>>   GNU tar                1.28             tar --version
>>>>>>   gtags (optional)       6.6.5            gtags --version
>>>>>>   mkimage (optional)     2017.01          mkimage --version
>>>>>> +Python (optional)      3.5.x            python3 --version
>>>>>
>>>>> Python 3.5 reached end-of-life 3½ years ago [1]. What's the point in
>>>>> using anything older than the oldest supported version of Python,
>>>>> i.e. 3.8 at this time?
>>>>
>>>> What's the point of breaking compilation with on older distros?
>>>> The idea of minimal versions here is to specify the absolute minimum
>>>> version that it is required for the build to happen. If 3.5 is
>>>> the minimal one, then be it.
>>>
>>> AFAICT 3.5 was an arbitrary rather than a deliberate choice. We should
>>> at least be aware *why* we'd be sticking to old versions.
>>
>>  From my side, the 3.5 was chosen basing on the previous feedback from
>> Jon Hunter: https://lore.kernel.org/dri-devel/20240412165407.42163-1-jonathanh@nvidia.com/
> 
> Patch there seems small/simple enough if it is all it takes for 3.5.
> 
> Yet, it would be nice to hear from Jon Hunter about the rationale
> for 3.5 support (if any).


We just have some legacy builders for legacy Tegra devices that are 
still using python 3.5. I will request that these are updated but these 
are not machines that I own and so may take some time.

>>> Minimum versions here also means sticking to features available in said
>>> versions, for Python just as well as for GCC or any other tool. That's
>>> not zero cost.
>>>
>>> I guess there are two angles here too. The absolute minimum version
>>> currently required, and the, uh, maximum the minimum version can be
>>> safely bumped to. Say, you want to use a feature not available in the
>>> current minimum, how far up can you bump the version to?
>>>
>>> Could we define and document the criteria (e.g. based on distros as you
>>> suggest below) so we don't have to repeat the discussion?
> 
> Agreed. While we should not bump version randomly, defining a
> criteria about when we should update the requirement sounds a great idea.
> 
> For me, the criteria is:
> 
> - the minimal version shall be at least the minimal one required for the
>    Kernel to build at the most used LTS distros that are not EOL, e. g.:
>    Debian, openSUSE/SUSE, CentOS/RHEL and Ubuntu LTS[1].
> 
> [1] In practice, Ubuntu LTS usually has a python version newer than
>      Debian LTS, and CentOS versions are identical to RHEL ones, so
>      I guess checking for Debian, openSUSE, SUSE and RHEL should be
>      enough.


Adding Stefan from SUSE because Stefan also reported a similar issue 
[0]. Note that subject of this email is incorrect and should be python 
3.6 and not 2.6 :-)

Jon

[0] https://lore.kernel.org/all/20240118123752.bl3qss3qbbxgvpdk@suse.de/
-- 
nvpublic