During BMC firmware upgrades on live systems, the ipmi_msghandler
generates excessive "BMC returned incorrect response" warnings
while the BMC is temporarily offline. This can flood system logs
in large deployments.

Replace dev_warn() with dev_warn_ratelimited() to throttle these
warnings and prevent log spam during BMC maintenance operations.

Signed-off-by: Breno Leitao <leitao@debian.org>
---
 drivers/char/ipmi/ipmi_msghandler.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c
index 064944ae9fdc3..8e9050f99e9ef 100644
--- a/drivers/char/ipmi/ipmi_msghandler.c
+++ b/drivers/char/ipmi/ipmi_msghandler.c
@@ -4607,10 +4607,10 @@ static int handle_one_recv_msg(struct ipmi_smi *intf,
 		 * The NetFN and Command in the response is not even
 		 * marginally correct.
 		 */
-		dev_warn(intf->si_dev,
-			 "BMC returned incorrect response, expected netfn %x cmd %x, got netfn %x cmd %x\n",
-			 (msg->data[0] >> 2) | 1, msg->data[1],
-			 msg->rsp[0] >> 2, msg->rsp[1]);
+		dev_warn_ratelimited(intf->si_dev,
+				     "BMC returned incorrect response, expected netfn %x cmd %x, got netfn %x cmd %x\n",
+				     (msg->data[0] >> 2) | 1, msg->data[1],
+				     msg->rsp[0] >> 2, msg->rsp[1]);
 
 		goto return_unspecified;
 	}

---
base-commit: d7b8f8e20813f0179d8ef519541a3527e7661d3a
change-id: 20250710-ipmi_ratelimit-7e1af5170c57

Best regards,
--  
Breno Leitao <leitao@debian.org>

On Thu, Jul 10, 2025 at 05:57:26AM -0700, Breno Leitao wrote:
> During BMC firmware upgrades on live systems, the ipmi_msghandler
> generates excessive "BMC returned incorrect response" warnings
> while the BMC is temporarily offline. This can flood system logs
> in large deployments.

So the BMC is returning bogus messages during an upgrade?  That
sounds like a bug in the BMC, it should either work or be
quiescent.  Returning incorrect data can result in bad things
happening.

But, as many thing IPMI, we have to accommodate bad designs.
This change is probably a good idea, anyway.

Queued for next merge window.

Thanks,

-corey

> 
> Replace dev_warn() with dev_warn_ratelimited() to throttle these
> warnings and prevent log spam during BMC maintenance operations.
> 
> Signed-off-by: Breno Leitao <leitao@debian.org>
> ---
>  drivers/char/ipmi/ipmi_msghandler.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c
> index 064944ae9fdc3..8e9050f99e9ef 100644
> --- a/drivers/char/ipmi/ipmi_msghandler.c
> +++ b/drivers/char/ipmi/ipmi_msghandler.c
> @@ -4607,10 +4607,10 @@ static int handle_one_recv_msg(struct ipmi_smi *intf,
>  		 * The NetFN and Command in the response is not even
>  		 * marginally correct.
>  		 */
> -		dev_warn(intf->si_dev,
> -			 "BMC returned incorrect response, expected netfn %x cmd %x, got netfn %x cmd %x\n",
> -			 (msg->data[0] >> 2) | 1, msg->data[1],
> -			 msg->rsp[0] >> 2, msg->rsp[1]);
> +		dev_warn_ratelimited(intf->si_dev,
> +				     "BMC returned incorrect response, expected netfn %x cmd %x, got netfn %x cmd %x\n",
> +				     (msg->data[0] >> 2) | 1, msg->data[1],
> +				     msg->rsp[0] >> 2, msg->rsp[1]);
>  
>  		goto return_unspecified;
>  	}
> 
> ---
> base-commit: d7b8f8e20813f0179d8ef519541a3527e7661d3a
> change-id: 20250710-ipmi_ratelimit-7e1af5170c57
> 
> Best regards,
> --  
> Breno Leitao <leitao@debian.org>
>