public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@deeprootsystems.com>
To: "Premi, Sanjeev" <premi@ti.com>
Cc: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: omap3 cpuidle interrupt latency
Date: Mon, 02 Mar 2009 11:46:17 -0800	[thread overview]
Message-ID: <873advaet2.fsf@deeprootsystems.com> (raw)
In-Reply-To: <B85A65D85D7EB246BE421B3FB0FBB59301CBF74DF9@dbde02.ent.ti.com> (Sanjeev Premi's message of "Sat\, 28 Feb 2009 23\:22\:48 +0530")

"Premi, Sanjeev" <premi@ti.com> writes:

> I have noticed large interrupt latency when the cpuidle is enabled.
> e.g. response time for ping goes from avg 10-20ms to 800-1000ms.
> (I am at HEAD of the 'pm' branch)

Is it interrupt latency in general you are measuring?  or just the
interrupt latency for the smc network driver.  I think what you are
seeing is the result of the SMC IRQ not being configured as a wakeup
source, thus a network interrupt will not wake the system, but you end
up waiting for the next idle timer until the system wakes and handles
the network interrupt.

By default, I don't believe the GPIO interrupt used by the smc is
configured as a wakeup source.  Have you configured that GPIO as a
wakeup source?

Kevin


> The IRQs and FIQs are disabled at the beginning of the function
> omap3_enter_idle() but WFI is executed much later in _omap_sram_idle().
> In between, there is only one check for pending IRQs - omap_irq_pending()
>  
> If any interrupt occurs beyond this point is it considered by the WFI?
>  
> To reduce this latency, I am planning to do either/both of thse:
> - Add more checks for pending IRQs
> - Reduce the time for which the IRQs and FIQs are disabled
>  
> Benefits will depend upon the behavior of WFI.
>  
> Best regards,
> Sanjeev
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2009-03-02 19:46 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-28 17:52 omap3 cpuidle interrupt latency Premi, Sanjeev
2009-03-01  3:25 ` Dasgupta, Romit
2009-03-01  4:05   ` Woodruff, Richard
2009-03-01  6:48     ` Dasgupta, Romit
2009-03-01 20:01       ` Premi, Sanjeev
2009-03-01 20:49         ` TK, Pratheesh Gangadhar
2009-03-02 19:46 ` Kevin Hilman [this message]
2009-03-03 10:23   ` Premi, Sanjeev
2009-03-04 14:21     ` Premi, Sanjeev
2009-03-04 19:15       ` Kevin Hilman
2009-03-05 21:24       ` Premi, Sanjeev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=873advaet2.fsf@deeprootsystems.com \
    --to=khilman@deeprootsystems.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=premi@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox