From: Philippe Gerum <rpm@xenomai.org>
To: Dmitry Adamushko <dmitry.adamushko@domain.hid>
Cc: Jan Kiszka <jan.kiszka@domain.hid>, xenomai@xenomai.org
Subject: Re: [Xenomai-core] [PATCH 0/3] Reworked nucleus statistics
Date: Thu, 19 Oct 2006 22:53:46 +0200 [thread overview]
Message-ID: <1161291226.4959.97.camel@domain.hid> (raw)
In-Reply-To: <b647ffbd0610181306k65d7308ei10783073e550f0d1@domain.hid>
On Wed, 2006-10-18 at 22:06 +0200, Dmitry Adamushko wrote:
>
> ...
>
> Dmitry, are there other issues I'm missing, that you think
> would be
> better solved using a simpler accounting model?
>
> So according to you, the more complex model gives users a better
> understanding of behavior of their drivers/applications?
>
I said the latter goal should be considered as most important one when
discussing this particular issue. Each approach having its pros/cons
regarding this goal depending on the aspect considered
(shared/non-shared, accounting accuracy etc.), the choice is about
picking the one which fulfills this goal as much as possible.
Basically, your respective approaches (i.e. Jan and yours) differ on the
criterion measuring such "fulfillment", and since there is likely no
perfect solution, we should make a trade-off on that. To sum up, you
consider that unambiguous shared IRQ accounting carries more weight,
whilst Jan seems to put more weight on charging any rescheduling cost
that could result from the ISR to the proper statistic counter, i.e. the
ISR, and not any random thread the corresponding IRQ happened to
preempt.
My guts feeling, as a possible user of this code, would be to favour
Jan's option, not because yours is essentially wrong, but rather because
it's missing what Jan's solution brings regarding the rescheduling
issue. In practice, a rescheduling costs a _lot_ more to perform
time-wise by the nucleus, than the less ambiguous measurement you could
do from an ISR prologue. My point is that, in that particular case, I'd
always prefer getting the right information from the most significant
order of magnitude.
Regarding the accounting upon XN_IRQ_HANDLED status, well, it's again a
matter of trade-off, even if the two options seem much more balanced in
their respective merits and caveats. I could vote for any of them; this
said, I think Jan's argument about shared IRQs being unusual (and I
would even add sub-optimal in a RT context) should be taken for what it
says: we should better try optimizing for the common case when there is
no real win in optimizing for the rare one. My initial question boils
down to make sure that I did not underestimate the importance of
improving the rare case.
[...]
--
Philippe.
next prev parent reply other threads:[~2006-10-19 20:53 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-10-12 19:43 [Xenomai-core] [PATCH 0/3] Reworked nucleus statistics Jan Kiszka
2006-10-18 13:42 ` Philippe Gerum
2006-10-18 20:06 ` Dmitry Adamushko
2006-10-18 20:29 ` Jan Kiszka
2006-10-19 9:59 ` Dmitry Adamushko
2006-10-19 20:53 ` Philippe Gerum [this message]
2006-10-27 16:00 ` 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=1161291226.4959.97.camel@domain.hid \
--to=rpm@xenomai.org \
--cc=dmitry.adamushko@domain.hid \
--cc=jan.kiszka@domain.hid \
--cc=xenomai@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.