All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.