From: Jan Kiszka <jan.kiszka@domain.hid>
To: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
Cc: adeos-main <adeos-main@gna.org>, Philippe Gerum <rpm@xenomai.org>
Subject: Re: [Adeos-main] [git pull] Updates for ipipe-2.6.36-noarch
Date: Tue, 18 Jan 2011 18:44:07 +0100 [thread overview]
Message-ID: <4D35D167.2040306@domain.hid> (raw)
In-Reply-To: <4D35CFE3.2020801@domain.hid>
On 2011-01-18 18:37, Gilles Chanteperdrix wrote:
> Jan Kiszka wrote:
>> On 2011-01-18 18:27, Philippe Gerum wrote:
>>> On Tue, 2011-01-18 at 18:22 +0100, Jan Kiszka wrote:
>>>> On 2011-01-18 18:13, Philippe Gerum wrote:
>>>>> On Thu, 2010-11-25 at 17:33 +0100, Jan Kiszka wrote:
>>>>>> The following changes since commit 4c83ab8e3ac5b194695e38bbc253f78e6072ad24:
>>>>>>
>>>>>> ipipe: tell __ipipe_run_irqtail about the latest IRQ number (2010-11-05 13:52:55 +0100)
>>>>>>
>>>>>> are available in the git repository at:
>>>>>> git://git.kiszka.org/ipipe-2.6 queues/2.6.35-noarch
>>>>>>
>>>>>> [edited log]
>>>>>> Jan Kiszka (4):
>>>>>> ipipe: Provide wrapper for IRQ mask/unmask at chip level
>>>>> Could you give me some hints about the intended usage of these?
>>>> The idea was (and still is but effort stalled ATM) to emulate MSI
>>>> masking on top of that.
>>> Ok, let's keep this on the shelf until we come with a complete solution.
>>> I suspect we will have to resync the Xenomai and I-pipe interfaces to
>>> this end.
>>
>> Definitely. So the patch was intended as a starting point, enbling
>> refactoring on Xenomai side. Quite a lot of work is actually to be done
>> on the rthal, also to clean up lots of duplicate irq descriptor
>> translations over there.
>
> As a maintainer of low end architectures, I am not too fond with
> introducing one more level of function pointers. A substantial part of
> the worst case interrupt latency on ARM is spent in these routines, so,
> it would be nice if we could avoid make this even longer. And still on
> ARM, acking/masking an interrupt line is just about writing some bit to
> some MMIO memory, so, the cost of the function pointer is definitely not
> negligible.
Then we need to push the definition of ipipe_irq_chip_[un]mask into arch
hands and effectively keep it for those setups as is which do not need
more. But we do need this abstraction.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
next prev parent reply other threads:[~2011-01-18 17:44 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-25 16:33 [Adeos-main] [git pull] Updates for ipipe-2.6.36-noarch Jan Kiszka
2010-11-25 16:38 ` [Adeos-main] [PATCH 1/3] ipipe: Provide wrapper for IRQ mask/unmask at chip level Jan Kiszka
2010-11-25 16:38 ` [Adeos-main] [PATCH 2/3] ipipe: Drop spurious irq_enter/exit from __ipipe_sync_stage Jan Kiszka
2010-11-25 16:38 ` [Adeos-main] [PATCH 3/3] ipipe: Provide __ipipe_spin_trylock_irq for !CONFIG_IPIPE Jan Kiszka
2011-01-18 17:13 ` [Adeos-main] [git pull] Updates for ipipe-2.6.36-noarch Philippe Gerum
2011-01-18 17:22 ` Jan Kiszka
2011-01-18 17:27 ` Philippe Gerum
2011-01-18 17:30 ` Jan Kiszka
2011-01-18 17:37 ` Gilles Chanteperdrix
2011-01-18 17:44 ` Jan Kiszka [this message]
2011-01-25 19:45 ` Philippe Gerum
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=4D35D167.2040306@domain.hid \
--to=jan.kiszka@domain.hid \
--cc=adeos-main@gna.org \
--cc=gilles.chanteperdrix@xenomai.org \
--cc=rpm@xenomai.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 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.