All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philippe Gerum <rpm@xenomai.org>
To: Jan Kiszka <jan.kiszka@domain.hid>
Cc: adeos-main <adeos-main@gna.org>
Subject: Re: [Adeos-main] [PATCH] x86: revert IRQs replay order
Date: Fri, 20 Oct 2006 16:05:48 +0200	[thread overview]
Message-ID: <1161353148.4988.51.camel@domain.hid> (raw)
In-Reply-To: <4538D4C8.1090603@domain.hid>

On Fri, 2006-10-20 at 15:53 +0200, Jan Kiszka wrote:
> Philippe Gerum wrote:
> > On Fri, 2006-10-20 at 12:33 +0200, Jan Kiszka wrote:
> >> While digging into a latency issue with multiple IRQs pending (patch
> >> will likely follow soon), I noticed that the replay order on x86 is the
> >> inverse of the hardware order. Instead of iterating from lowest IRQ
> >> number to highest, ipipe currently starts with the highest one. The
> >> attached patch fixes this.
> >>
> > 
> > We want to give a priority boost to virtual IRQs over real ones, at
> > least for the root domain. Since virqs are high-numbered, bsrl has been
> 
> Hmm, are the virtual IRQs differently numbered on blackfin? Because
> there we have ffs behind __ipipe_ffnz.
> 

They are not, but the priority boost for virqs is most significant on
x86, this is why I did not bothered that much for other archs, including
on the Blackfin. The point is that we want the root domain to process
virqs sent by the RTOSes running on higher domains asap, at least before
long and costly Linux interrupt handlers may run; e.g. the IDE interrupt
handler on x86 (which gets even worse if you run that in PIO mode). On
the contrary, the Linux domain handlers over Adeos/Blackfin are
threaded; only a very simple primary handler wakes up the IRQ thread, so
the worst incurred delay before processing the virqs is known, short and
limited.

> > used on purpose to scan the pending IRQ mask. Additionally, low-numbered
> > IRQs have higher priority only if you consider the ISA ones as managed
> > by the 8259. Bringing the APIC into the picture, the APIC-based timer
> > IRQ used by RTOSes over Adeos is a high-numbered one.
> 
> Ok, so there is no simple way to emulate reality, thus we can simply
> leave it as it is. No problem.
> 
> Jan
> 
> _______________________________________________
> Adeos-main mailing list
> Adeos-main@domain.hid
> https://mail.gna.org/listinfo/adeos-main
-- 
Philippe.




      reply	other threads:[~2006-10-20 14:05 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-20 10:33 [Adeos-main] [PATCH] x86: revert IRQs replay order Jan Kiszka
2006-10-20 13:21 ` Philippe Gerum
2006-10-20 13:53   ` Jan Kiszka
2006-10-20 14:05     ` Philippe Gerum [this message]

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=1161353148.4988.51.camel@domain.hid \
    --to=rpm@xenomai.org \
    --cc=adeos-main@gna.org \
    --cc=jan.kiszka@domain.hid \
    /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.