On 6/20/26 4:06 PM, Zenghui Yu wrote: > On 6/21/26 2:40 AM, Randy Dunlap wrote: >> >> >> On 6/20/26 5:27 AM, Zenghui Yu wrote: >>> The correct path of the "hotmod" module parameter should be >>> /sys/module/ipmi_si/parameters/hotmod. Fix it. >>> >>> Signed-off-by: Zenghui Yu <[email protected]> >>> --- >>> Documentation/driver-api/ipmi.rst | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> There are several other places that use /sys/modules/ instead of >> /sys/module/. > > Yup. There are: > > Documentation/driver-api/ipmi.rst:This is done using > /sys/modules/ipmi_si/parameters/hotmod, which is a > Documentation/process/debugging/kgdb.rst:config string to > ``/sys/module/<driver>/parameter/<option>``. The driver > Documentation/translations/zh_CN/admin-guide/mm/damon/lru_sort.rst:参数，或者在 > ``/sys/modules/damon_lru_sort/parameters/<parameter>`` 写入正确的 > Documentation/translations/zh_CN/admin-guide/mm/damon/lru_sort.rst: # cd > /sys/modules/damon_lru_sort/parameters > Documentation/translations/zh_TW/admin-guide/mm/damon/lru_sort.rst:參數，或者在 > ``/sys/modules/damon_lru_sort/parameters/<parameter>`` 寫入正確的 > Documentation/translations/zh_TW/admin-guide/mm/damon/lru_sort.rst: # cd > /sys/modules/damon_lru_sort/parameters > drivers/acpi/sysfs.c: * /sys/modules/acpi/parameters/debug_layer > drivers/acpi/sysfs.c: * /sys/modules/acpi/parameters/debug_level > drivers/acpi/sysfs.c: * /sys/modules/acpi/parameters/trace_method_name > drivers/acpi/sysfs.c: * /sys/modules/acpi/parameters/trace_state > drivers/acpi/sysfs.c: * /sys/modules/acpi/parameters/trace_debug_layer > drivers/acpi/sysfs.c: * /sys/modules/acpi/parameters/trace_debug_level > drivers/acpi/sysfs.c:/* /sys/modules/acpi/parameters/aml_debug_output */ > drivers/base/module.c: /* Lookup or create built-in module entry in > /sys/modules */ > drivers/gpib/lpvo_usb_gpib/lpvo_usb_gpib.c: * On the fly: echo > {0,1,2} > /sys/modules/lpvo_usb_gpib/parameters/debug > fs/btrfs/sysfs.c:/* Set perms to 0, disable > /sys/module/btrfs/parameter/read_policy interface. */ > fs/cachefiles/Kconfig: enabled by setting bits in > /sys/modules/cachefiles/parameter/debug or > kernel/params.c:/* sysfs output in /sys/modules/XYZ/parameters/ */ > >> >> Would you care to fix those also? > > I plan to fix them by subsystem like: > > https://lore.kernel.org/[email protected]
Great. Thanks. -- ~Randy _______________________________________________ Openipmi-developer mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openipmi-developer
