the initscript does not implement some mandatory actions, it does not implement some actions correctly, and does not report correct status
please see https://fedoraproject.org/wiki/Packaging/SysVInitScript
1) the initscript does not implement 'condrestart'
2) the initscript does not implement 'try-restart'
3) the initscript does not implement 'force-reload'
4) running "service edac status" after "service edac stop" returns 0 instead of 3
5) running "service edac reload" on a stopped service returns 0 instead of 7 and restarts the service
6) for invalid arguments, instead of returning error message and status code 2, the initscript prints usage message and returns 0 - but usage message should be printed only if run without arguments or with "usage" argument
7) running the initscript as non-root user does not return error and status code 4 (except for usage and status, it should be allowed), but it tries to perform the actions which leads to permission denial errors
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux major release. Product Management has requested further
review of this request by Red Hat Engineering, for potential inclusion in a Red
Hat Enterprise Linux Major release. This request is not yet committed for
(In reply to comment #0)
> 4) running "service edac status" after "service edac stop" returns 0 instead of
> 5) running "service edac reload" on a stopped service returns 0 instead of 7
> and restarts the service
Such return codes wouldn't make any sense on this case. This script should not
be different than the one provided for bluetooth, where only a start method is
echo -n $"Enabling Bluetooth devices:"
udevadm trigger --subsystem-match=bluetooth
# FIXME If somebody figures out how to disable the K* script
echo -n "Stopping Bluetooth services:"
It is just the same case: we just need to run edac-ctl during start method,
in order to load some parameters at the corresponding kernel driver. This script doesn't start any deamon that needs to be stopped. I think I'll rewrite
the init script to look more like the above, adding some fake methods for all
those restart methods that will just call edac-ctl whatever argument provided,
except for stop and usage.
Created attachment 426119 [details]
New EDAC SysV script
New script that fulfills all requirements stated at:
Since this script just sends some data to EDAC kernel driver, there's no stop action, nor there is a "daemon status". So, all actions that asks for reload will just do the same action as "start": send the new memory labels table to the Kernel driver. "stop" and "status" will just return 0, as there's no way (nor make sense) to stop a non-daemon process.
New SysV script added at: edac-utils-0.9-11.2.el6.src.rpm
Red Hat Enterprise Linux 6.0 is now available and should resolve
the problem described in this bug report. This report is therefore being closed
with a resolution of CURRENTRELEASE. You may reopen this bug report if the
solution does not work for you.