kernel/include/linux/ipmi_smi.h, branch linux-4.16.y

ipmi: Make the IPMI proc interface configurable

2017-09-28T17:26:03Z

So we can remove it later. Signed-off-by: Corey Minyard

ipmi: Remove the device id from ipmi_register_smi()

2017-09-27T21:03:45Z

It's no longer used, dynamic device id handling is in place now. Signed-off-by: Corey Minyard

ipmi: Make ipmi_demangle_device_id more generic

2017-09-27T21:03:45Z

Currently, ipmi_demagle_device_id requires a full response buffer in its data argument. This means we can't use it to parse a response in a struct ipmi_recv_msg, which has the netfn and cmd as separate bytes. This change alters the definition and users of ipmi_demangle_device_id to use a split netfn, cmd and data buffer, so it can be used with non-sequential responses. Signed-off-by: Jeremy Kerr Fixed the ipmi_ssif.c and ipmi_si_intf.c changes to use data from the response, not the data from the message, when passing info to the ipmi_demangle_device_id() function. Signed-off-by: Corey Minyard

ipmi: Don't flush messages in sender() in run-to-completion mode

2015-09-03T20:02:28Z

When flushing queued messages in run-to-completion mode, smi_event_handler() is recursively called. flush_messages() smi_event_handler() handle_transaction_done() deliver_recv_msg() ipmi_smi_msg_received() smi_recv_tasklet() sender() flush_messages() smi_event_handler() ... The depth of the recursive call depends on the number of queued messages, so it can cause a stack overflow if many messages have been queued. To solve this problem, this patch removes flush_messages() from sender()@ipmi_si_intf.c. Instead, add flush_messages() to caller side of sender() if needed. Additionally, to implement this, add new handler flush_messages to struct ipmi_smi_handlers. Signed-off-by: Hidehiro Kawai Fixed up a comment and some spacing issues. Signed-off-by: Corey Minyard

ipmi: Make some data const that was only read

2015-09-03T20:02:27Z

Several data structures were only used for reading, so make them const. Signed-off-by: Corey Minyard

ipmi: Remove the now unused priority from SMI sender

2014-12-11T21:04:11Z

Since the queue was moved into the message handler, the priority field is now irrelevant. Signed-off-by: Corey Minyard

ipmi: Remove useless sysfs_name parameters

2014-12-11T21:04:05Z

It was always "bmc", so just hardcode it. It makes no sense to pass that in. Signed-off-by: Corey Minyard

ipmi: boolify some things

2014-04-17T19:30:40Z

Convert some ints to bools. Signed-off-by: Corey Minyard Signed-off-by: Linus Torvalds

ipmi: Turn off all activity on an idle ipmi interface

2014-04-17T19:23:07Z

The IPMI driver would wake up periodically looking for events and watchdog pretimeouts. If there is nothing waiting for these events, it's really kind of pointless to be checking for them. So modify the driver so the message handler can pass down if it needs the lower layer to be waiting for these. Modify the system interface lower layer to turn off all timer and thread activity if the upper layer doesn't need anything and it is not currently handling messages. And modify the message handler to not restart the timer if its timer is not needed. The timers and kthread will still be enabled if: - the SI interface is handling a message. - a user has enabled watching for events. - the IPMI watchdog timer is in use (since it uses pretimeouts). - the message handler is waiting on a remote response. - a user has registered to receive commands. This mostly affects interfaces without interrupts. Interfaces with interrupts already don't use CPU in the system interface when the interface is idle. Signed-off-by: Corey Minyard Signed-off-by: Linus Torvalds

various: Fix spelling of "asynchronous" in comments.

2012-11-19T13:32:13Z

"Asynchronous" is misspelled in some comments. No code changes. Signed-off-by: Adam Buchbinder Signed-off-by: Jiri Kosina