[PATCH v2 0/3] Cleanups in amd64_edac and removal of legacy EDAC sysfs interface

Avadhut Naik posted 3 patches 1 month, 1 week ago
Documentation/admin-guide/RAS/main.rst     | 142 +-------
arch/loongarch/configs/loongson3_defconfig |   1 -
drivers/edac/Kconfig                       |   8 -
drivers/edac/amd64_edac.c                  |  61 +---
drivers/edac/amd64_edac.h                  |   7 +-
drivers/edac/edac_mc_sysfs.c               | 404 ---------------------
6 files changed, 22 insertions(+), 601 deletions(-)
[PATCH v2 0/3] Cleanups in amd64_edac and removal of legacy EDAC sysfs interface
Posted by Avadhut Naik 1 month, 1 week ago
This patchset undertakes some cleanups in the amd64_edac module and
removes the legacy EDAC sysfs interface since it has been obsolete for
more than a decade.

The first patch removes explicit assignment of the ctl_name string and
instead generates it at runtime using scnprintf.

The second patch removes the NUM_CONTROLLERS macro and instead uses the
max_mcs variable to determine the size of chipselect array.

The third patch removes the old EDAC sysfs interface.

NOTE:
These cleanups were earlier submitted as part of the set adding EDAC
support for AMD's newer Family 1Ah-based SOCs.[1] However, since the
support patches were critical to enabling EDAC on newer AMD SOCs and
since these cleanups were still under review, the set was split up.

[1]: https://lore.kernel.org/all/20250909185748.1621098-1-avadhut.naik@amd.com/

Changes in v2:
 - Drop the patch that set zn_regs_v2 flag for all SOCs based on Family
1Ah and later.
 - Change MAX_CTL_NAMELEN macro from 20 to 19.
 - Modify commit message of the second patch per feedback received.
 - Skip deprecation and just remove the old EDAC sysfs interface.

Links:
v1: https://lore.kernel.org/all/20251013173632.1449366-1-avadhut.naik@amd.com/

Avadhut Naik (3):
  EDAC/amd64: Generate ctl_name string at runtime
  EDAC/amd64: Remove NUM_CONTROLLERS macro
  edac: Remove the legacy EDAC sysfs interface

 Documentation/admin-guide/RAS/main.rst     | 142 +-------
 arch/loongarch/configs/loongson3_defconfig |   1 -
 drivers/edac/Kconfig                       |   8 -
 drivers/edac/amd64_edac.c                  |  61 +---
 drivers/edac/amd64_edac.h                  |   7 +-
 drivers/edac/edac_mc_sysfs.c               | 404 ---------------------
 6 files changed, 22 insertions(+), 601 deletions(-)


base-commit: d05ab93a9183520b2a6965765e14a3e6bdf01f90
-- 
2.43.0
Re: [PATCH v2 0/3] Cleanups in amd64_edac and removal of legacy EDAC sysfs interface
Posted by Borislav Petkov 1 month, 1 week ago
On Thu, Nov 06, 2025 at 01:54:43AM +0000, Avadhut Naik wrote:
> This patchset undertakes some cleanups in the amd64_edac module and
> removes the legacy EDAC sysfs interface since it has been obsolete for
> more than a decade.
> 
> The first patch removes explicit assignment of the ctl_name string and
> instead generates it at runtime using scnprintf.
> 
> The second patch removes the NUM_CONTROLLERS macro and instead uses the
> max_mcs variable to determine the size of chipselect array.
> 
> The third patch removes the old EDAC sysfs interface.
> 
> NOTE:
> These cleanups were earlier submitted as part of the set adding EDAC
> support for AMD's newer Family 1Ah-based SOCs.[1] However, since the
> support patches were critical to enabling EDAC on newer AMD SOCs and
> since these cleanups were still under review, the set was split up.
> 
> [1]: https://lore.kernel.org/all/20250909185748.1621098-1-avadhut.naik@amd.com/
> 
> Changes in v2:
>  - Drop the patch that set zn_regs_v2 flag for all SOCs based on Family
> 1Ah and later.
>  - Change MAX_CTL_NAMELEN macro from 20 to 19.
>  - Modify commit message of the second patch per feedback received.
>  - Skip deprecation and just remove the old EDAC sysfs interface.
> 
> Links:
> v1: https://lore.kernel.org/all/20251013173632.1449366-1-avadhut.naik@amd.com/
> 
> Avadhut Naik (3):
>   EDAC/amd64: Generate ctl_name string at runtime
>   EDAC/amd64: Remove NUM_CONTROLLERS macro
>   edac: Remove the legacy EDAC sysfs interface
> 
>  Documentation/admin-guide/RAS/main.rst     | 142 +-------
>  arch/loongarch/configs/loongson3_defconfig |   1 -
>  drivers/edac/Kconfig                       |   8 -
>  drivers/edac/amd64_edac.c                  |  61 +---
>  drivers/edac/amd64_edac.h                  |   7 +-
>  drivers/edac/edac_mc_sysfs.c               | 404 ---------------------
>  6 files changed, 22 insertions(+), 601 deletions(-)

Queued, thanks.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette