All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@domain.hid>
To: Philippe Gerum <rpm@xenomai.org>
Cc: adeos-main <adeos-main@gna.org>
Subject: Re: [Adeos-main] [pull request] x86: critical fix for native_safe_halt
Date: Sun, 13 Dec 2009 18:48:57 +0100	[thread overview]
Message-ID: <4B252909.9090304@domain.hid> (raw)
In-Reply-To: <1260723757.2216.2.camel@domain.hid>

[-- Attachment #1: Type: text/plain, Size: 1025 bytes --]

Philippe Gerum wrote:
> On Sat, 2009-12-12 at 22:37 +0100, Jan Kiszka wrote:
>> Recent moving of ipipe_suspend_domain finally exposed a deeper flaw in
>> cpu_idle on x86: We failed to check the pipeline log before issuing the
>> real hlt. This caused IRQ latencies or even drops for Linux,
>> specifically on SMP. Credits go to plain QEMU whose slow SMP mode caused
>> ipipe_critical_enter to deadlock frequently enough.
>>
>> The first patch of this series fixes this (see below), the second one
>> simply removes the two useless ipipe_suspend_domain calls.
>>
> 
> What your patch does as well, is killing the ability to run low priority
> domains below the root level.

Yes, I'm killing the dream.

I heavily doubt that the functions I removed in the second patch ever
contributed something good to this. It's always the job of the lowest
domain to issue hardware halt, not of some arbitrary mid-prio domain.
Moreover, what would be the practical use for such model in the context
of Linux?

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 257 bytes --]

  reply	other threads:[~2009-12-13 17:48 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-12 21:37 [Adeos-main] [pull request] x86: critical fix for native_safe_halt Jan Kiszka
2009-12-13 17:02 ` Philippe Gerum
2009-12-13 17:48   ` Jan Kiszka [this message]
2009-12-13 18:05     ` Philippe Gerum
2009-12-13 18:19       ` Jan Kiszka
2009-12-16 11:26         ` Philippe Gerum
2009-12-16 11:53           ` Jan Kiszka
2009-12-16 14:19             ` Philippe Gerum
2009-12-16 14:30               ` Philippe Gerum
2009-12-16 14:39                 ` Jan Kiszka
2009-12-16 14:51                   ` Philippe Gerum
2009-12-16 14:32               ` Jan Kiszka
2009-12-16 14:41                 ` 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=4B252909.9090304@domain.hid \
    --to=jan.kiszka@domain.hid \
    --cc=adeos-main@gna.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.