Andrew Morton wrote:

Corey Minyard <[EMAIL PROTECTED]> wrote:
Indeed, this function is badly written.  In rewriting, I couldn't find a
nice function for reading integers from userspace, and the proc_dointvec
stuff didn't seem terribly suitable.

We write numbers into profs files all the time.  Is there something
different about the IPMI requirement which makes the approach used by, say,
dirty_writeback_centisecs_handler() inappropriate?
Ok, that's probably better, and this probably belongs in /proc/sys/dev/ipmi. This is new enough that it doesn't matter, I don't think any one is using it yet.

Patch is attached.

-Corey

Attachment: ipmi-poweroff-fix-chassis-ctrl.patch
Description: unknown/unknown

Reply via email to