On Sat, Apr 25, 2026 at 10:36:05AM +0100, Matt Fleming wrote:
> On Tue, Apr 21, 2026 at 07:42:44AM -0500, Corey Minyard wrote:
> > The driver would just fetch events and receive messages until the
> > BMC said it was done. To avoid issues with BMCs that never say they are
> > done, add a limit of 10 fetches at a time.
> >
> > This is a more general fix than the previous fix for the specific bad
> > BMC, but should fix the more general issue of a BMC that won't stop
> > saying it has data.
> >
> > This has been there from the beginning of the driver.
> >
> > Reported-by: Matt Fleming <[email protected]>
> > Closes:
> > https://lore.kernel.org/lkml/[email protected]/
> > Fixes: <1da177e4c3f4> ("Linux-2.6.12-rc2")
> > Cc: [email protected]
> > Signed-off-by: Corey Minyard <[email protected]>
> > ---
> > drivers/char/ipmi/ipmi_si_intf.c | 15 +++++++++++++++
> > drivers/char/ipmi/ipmi_ssif.c | 15 +++++++++++++++
> > 2 files changed, 30 insertions(+)
>
> [...]
>
> > @@ -410,6 +413,7 @@ static void start_getting_msg_queue(struct smi_info
> > *smi_info)
> >
> > start_new_msg(smi_info, smi_info->curr_msg->data,
> > smi_info->curr_msg->data_size);
> > + smi_info->num_requests_in_a_row = 0;
> > smi_info->si_state = SI_GETTING_MESSAGES;
> > }
> >
> > @@ -421,6 +425,7 @@ static void start_getting_events(struct smi_info
> > *smi_info)
> >
> > start_new_msg(smi_info, smi_info->curr_msg->data,
> > smi_info->curr_msg->data_size);
> > + smi_info->num_requests_in_a_row = 0;
> > smi_info->si_state = SI_GETTING_EVENTS;
> > }
> >
>
> Would it be better to move this zeroing to handle_transaction_done()?
> Otherwise we reset the counter in handle_flags() ->
> start_getting_events() and the threshold is never reached.
Oh, yeah.
Moving it to handle_transaction_done() is not ideal, though. If
something was spewing receive messages, it would never get to events,
which is why I did it like I did.
The following should fix this. You could also have different limits for
receive messages and events, but I think the following is more clear.
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c
index 2a739123270c..e46f4150ceb5 100644
--- a/drivers/char/ipmi/ipmi_si_intf.c
+++ b/drivers/char/ipmi/ipmi_si_intf.c
@@ -413,8 +413,10 @@ static void start_getting_msg_queue(struct smi_info
*smi_info)
start_new_msg(smi_info, smi_info->curr_msg->data,
smi_info->curr_msg->data_size);
- smi_info->num_requests_in_a_row = 0;
- smi_info->si_state = SI_GETTING_MESSAGES;
+ if (smi_info->si_state != SI_GETTING_MESSAGES) {
+ smi_info->num_requests_in_a_row = 0;
+ smi_info->si_state = SI_GETTING_MESSAGES;
+ }
}
static void start_getting_events(struct smi_info *smi_info)
@@ -425,8 +427,10 @@ static void start_getting_events(struct smi_info *smi_info)
start_new_msg(smi_info, smi_info->curr_msg->data,
smi_info->curr_msg->data_size);
- smi_info->num_requests_in_a_row = 0;
- smi_info->si_state = SI_GETTING_EVENTS;
+ if (smi_info->si_state != SI_GETTING_EVENTS) {
+ smi_info->num_requests_in_a_row = 0;
+ smi_info->si_state = SI_GETTING_EVENTS;
+ }
}
/*
>
> Thanks,
> Matt
_______________________________________________
Openipmi-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openipmi-developer
