high cpu load on omap3 using musb
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Mon Jul 21 08:28:58 PDT 2014
Hi Adam,
On Wednesday 29 January 2014 08:44:57 Adam Wozniak wrote:
> With a USB 2.0 webcam attached to the OTG port on an OMAP3 (applies to
> overo gumstix, beagleboard, probably others) we see a high CPU load in a
> kworker thread.
>
> Between 2.6.33 and 2.6.34 musb_core.c changed.
>
> IRQ handlers changed with the result that a worker in musb_core.c got
> scheduled far more frequently than needed.
>
> I've included a patch below against 3.7, but i think it'll apply against
> mainline.
> [I apologize for any whitespace mangling. I've also attached the patch.]
>
> I'd like more eyeballs to tell me if this is right. I'd also like to
> know who I need to talk to to get this pushed into mainline.
Running the scripts/get_maintainer.pl script on your patch produces
Felipe Balbi <balbi at ti.com> (maintainer:MUSB MULTIPOINT H...)
Greg Kroah-Hartman <gregkh at linuxfoundation.org> (supporter:USB SUBSYSTEM)
linux-usb at vger.kernel.org (open list:MUSB MULTIPOINT H...)
linux-kernel at vger.kernel.org (open list)
Felipe Balbi (CC'ed) is the person who you should talk to.
While we're touching the subject of scripts, you should run the
scripts/checkpatch.pl script and fix errors and warnings before submitting
patches. Please see Documentation/SubmittingPatches.
Last (but not least) piece of advice, don't give up if you don't receive
replies to your patches. People are busy and mails fall to cracks from time to
time.
Felipe, apart from the coding style violation and the possibly missing
locking, what's your opinion on this ? Does the patch make sense ?
> # The MUSB IRQ schedules work on every interrupt.
> # This is unnecessary, and causes excessive CPU load.
> #
> # Here we only schedule work if there is something for
> # the worker to do.
>
> Index: git/drivers/usb/musb/musb_core.c
> ===================================================================
> --- git.orig/drivers/usb/musb/musb_core.c
> +++ git/drivers/usb/musb/musb_core.c
> @@ -925,7 +925,9 @@ b_host:
> }
> #endif
>
> - schedule_work(&musb->irq_work);
> + if (musb->xceiv->state != musb->xceiv_old_state) {
> + schedule_work(&musb->irq_work);
> + }
>
> return handled;
> }
--
Regards,
Laurent Pinchart
More information about the linux-arm
mailing list