Subject: + drivers-misc-mei-ratelimit-several-error-messages.patch added to -mm tree To: imunsie@xxxxxxxxxxx,greg@xxxxxxxxx,tomas.winkler@xxxxxxxxx From: akpm@xxxxxxxxxxxxxxxxxxxx Date: Tue, 07 Jan 2014 15:45:15 -0800 The patch titled Subject: drivers/misc/mei: ratelimit several error messages has been added to the -mm tree. Its filename is drivers-misc-mei-ratelimit-several-error-messages.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/drivers-misc-mei-ratelimit-several-error-messages.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/drivers-misc-mei-ratelimit-several-error-messages.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Ian Munsie <imunsie@xxxxxxxxxxx> Subject: drivers/misc/mei: ratelimit several error messages I'm unsure of the cause, but I found irq/40-mei_me consuming 100% CPU and my disk full due to kern.log, syslog and messages rapidly growing in size filled with these messages: Dec 23 12:29:57 dukhat kernel: [336224.363138] mei_me 0000:00:16.0: reset: wrong host start response Dec 23 12:29:57 dukhat kernel: [336224.363140] mei_me 0000:00:16.0: unexpected reset: dev_state = RESETTING Dec 23 12:29:57 dukhat kernel: [336224.363152] mei_me 0000:00:16.0: reset: unexpected enumeration response hbm. Dec 23 12:29:57 dukhat kernel: [336224.363155] mei_me 0000:00:16.0: unexpected reset: dev_state = RESETTING Dec 23 12:29:57 dukhat kernel: [336224.363185] mei_me 0000:00:16.0: reset: wrong host start response Dec 23 12:29:57 dukhat kernel: [336224.363187] mei_me 0000:00:16.0: unexpected reset: dev_state = RESETTING Dec 23 12:29:57 dukhat kernel: [336224.363199] mei_me 0000:00:16.0: reset: unexpected enumeration response hbm. Dec 23 12:29:57 dukhat kernel: [336224.363201] mei_me 0000:00:16.0: unexpected reset: dev_state = RESETTING This patch ratelimits those specific messages to reduce the impact if this problem ever recurs. Signed-off-by: Ian Munsie <imunsie@xxxxxxxxxxx> Cc: Tomas Winkler <tomas.winkler@xxxxxxxxx> Cc: Greg KH <greg@xxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/misc/mei/hbm.c | 4 ++-- drivers/misc/mei/init.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff -puN drivers/misc/mei/hbm.c~drivers-misc-mei-ratelimit-several-error-messages drivers/misc/mei/hbm.c --- a/drivers/misc/mei/hbm.c~drivers-misc-mei-ratelimit-several-error-messages +++ a/drivers/misc/mei/hbm.c @@ -613,7 +613,7 @@ void mei_hbm_dispatch(struct mei_device dev->init_clients_timer = 0; mei_hbm_enum_clients_req(dev); } else { - dev_err(&dev->pdev->dev, "reset: wrong host start response\n"); + dev_err_ratelimited(&dev->pdev->dev, "reset: wrong host start response\n"); mei_reset(dev, 1); return; } @@ -690,7 +690,7 @@ void mei_hbm_dispatch(struct mei_device /* first property reqeust */ mei_hbm_prop_req(dev); } else { - dev_err(&dev->pdev->dev, "reset: unexpected enumeration response hbm.\n"); + dev_err_ratelimited(&dev->pdev->dev, "reset: unexpected enumeration response hbm.\n"); mei_reset(dev, 1); return; } diff -puN drivers/misc/mei/init.c~drivers-misc-mei-ratelimit-several-error-messages drivers/misc/mei/init.c --- a/drivers/misc/mei/init.c~drivers-misc-mei-ratelimit-several-error-messages +++ a/drivers/misc/mei/init.c @@ -148,7 +148,7 @@ void mei_reset(struct mei_device *dev, i dev->dev_state != MEI_DEV_POWER_UP); if (unexpected) - dev_warn(&dev->pdev->dev, "unexpected reset: dev_state = %s\n", + dev_warn_ratelimited(&dev->pdev->dev, "unexpected reset: dev_state = %s\n", mei_dev_state_str(dev->dev_state)); ret = mei_hw_reset(dev, interrupts_enabled); _ Patches currently in -mm which might be from imunsie@xxxxxxxxxxx are drivers-misc-mei-ratelimit-several-error-messages.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html