lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87o7aha1mj.fsf@mail.lhotse>
Date: Wed, 10 Apr 2024 21:38:44 +1000
From: Michael Ellerman <michaele@....ibm.com>
To: Christophe Leroy <christophe.leroy@...roup.eu>, Andrew Donnellan
 <ajd@...ux.ibm.com>, "linuxppc-dev@...ts.ozlabs.org"
 <linuxppc-dev@...ts.ozlabs.org>, "linux-kernel@...r.kernel.org"
 <linux-kernel@...r.kernel.org>, "linux-scsi@...r.kernel.org"
 <linux-scsi@...r.kernel.org>
Cc: "fbarrat@...ux.ibm.com" <fbarrat@...ux.ibm.com>, "ukrishn@...ux.ibm.com"
 <ukrishn@...ux.ibm.com>, "manoj@...ux.ibm.com" <manoj@...ux.ibm.com>
Subject: Re: [PATCH 2/2] MAINTAINERS: Make cxl obsolete

Christophe Leroy <christophe.leroy@...roup.eu> writes:
> Le 09/04/2024 à 06:37, Michael Ellerman a écrit :
>> Andrew Donnellan <ajd@...ux.ibm.com> writes:
>>> The cxl driver is no longer actively maintained and we intend to remove it
>>> in a future kernel release. Change its status to obsolete, and update the
>>> sysfs ABI documentation accordingly.
>>>
>>> Signed-off-by: Andrew Donnellan <ajd@...ux.ibm.com>
>>> ---
>>>   Documentation/ABI/{testing => obsolete}/sysfs-class-cxl | 3 +++
>>>   MAINTAINERS                                             | 4 ++--
>>>   2 files changed, 5 insertions(+), 2 deletions(-)
>>>   rename Documentation/ABI/{testing => obsolete}/sysfs-class-cxl (99%)
>> 
>> This is a good start, but I suspect if there are any actual users they
>> are not going to be monitoring the status of cxl in the MAINTAINERS file :)
>> 
>> I think we should probably modify Kconfig so that anyone who's using cxl
>> on purpose has some chance to notice before we remove it.
>> 
>> Something like the patch below. Anyone who has an existing config and
>> runs oldconfig will get a prompt, eg:
>> 
>>    Deprecated support for IBM Coherent Accelerators (CXL) (DEPRECATED_CXL) [N/m/y/?] (NEW)
>> 
>> Folks who just use defconfig etc. won't notice any change which is a
>> pity. We could also change the default to n, but that risks breaking
>> someone's machine. Maybe we do that in a another releases time.
>
> When I boot one of my boards I see:
>
> [    0.641090] mcr3000-hwmon 10000800.hwmon: hwmon_device_register() is 
> deprecated. Please convert the driver to use 
> hwmon_device_register_with_info().
>
> Could we do something similar, write a message at boottime when the CXL 
> driver gets probed ?

Yeah, I think so.

There's still the problem that people tend not to look at dmesg until
something breaks, but at least we can try and get their attention.

cheers

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ