mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Stephane Eranian <eranian@hpl.hp.com>
To: Andi Kleen <ak@suse.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 14/18] 2.6.17.9 perfmon2 patch for review: new i386 files
Date: Mon, 28 Aug 2006 09:10:00 -0700	[thread overview]
Message-ID: <20060828161000.GF20394@frankl.hpl.hp.com> (raw)
In-Reply-To: <p733bbn7m6o.fsf@verdi.suse.de>

Andi,

On Wed, Aug 23, 2006 at 12:58:55PM +0200, Andi Kleen wrote:
> 
> > +
> > +fastcall void smp_pmu_interrupt(struct pt_regs *regs)
> > +{
> 
> This misses enter/exit_idle on x86-64.
> 
I have been working on adding idle notifier for i386.
I am wondering about this code:

/* Called from interrupts to signify idle end */
void exit_idle(void)
{
        if (current->pid | read_pda(irqcount))
                return;
        __exit_idle();
}

And in particular the irqcount. I am guessing you are trying
to protect against nested interrupts. In fact, I think we only
want to get notified once the interrupt stack is fully unwound.
because we get way more exit_idle() than enter_idle().

Is there an irqcount mechanism on i386?

Thanks

--
-Stephane

  parent reply	other threads:[~2006-08-28 16:20 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-23  8:06 Stephane Eranian
2006-08-23 10:58 ` Andi Kleen
2006-08-25 12:49   ` Stephane Eranian
2006-08-25 13:13     ` Andi Kleen
2006-08-25 14:27       ` Stephane Eranian
2006-08-25 14:53         ` Andi Kleen
2006-08-25 15:00           ` Stephane Eranian
2006-08-25 15:18             ` Andi Kleen
2006-08-25 15:10           ` Stephane Eranian
2006-09-12 14:04       ` Stephane Eranian
2006-09-12 14:25         ` Stephane Eranian
2006-08-28 16:10   ` Stephane Eranian [this message]
2006-08-23 23:18 ` Adrian Bunk

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=20060828161000.GF20394@frankl.hpl.hp.com \
    --to=eranian@hpl.hp.com \
    --cc=ak@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    /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

Powered by JetHome