All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@domain.hid>
To: Philippe Gerum <rpm@xenomai.org>
Cc: Xenomai-core@domain.hid
Subject: Re: [Xenomai-core] [PATCH] fix hw-timer setup/cleanup for i386
Date: Fri, 12 Oct 2007 11:00:56 +0200	[thread overview]
Message-ID: <470F37C8.4090301@domain.hid> (raw)
In-Reply-To: <470E8BFE.4070804@domain.hid>

Philippe,

Jan Kiszka wrote:
> This patch for SVN trunk fixes most of the current bugs around hardware
> timer takeover and release from/to Linux. Tested and found working here
> (including SMP):
> - 2.6.22, APIC, highres=off, nohz=off
> - 2.6.22, APIC, highres=on, nohz=on
> - 2.6.20, APIC
> 
> Tests to be done:
> - 2.6.22, PIT (currently building...)
> - 2.6.20, PIT

To close this topic: Also the tests with APIC switched off were
successfully completed yesterday. So the logic should be fine now, ...

> 
> Things became quite complex in i386/hal.c now. Some of this complexity
> might be avoidable if RTHAL_APIC_TIMER_VECTOR equalled
> LOCAL_TIMER_VECTOR. What's the reason for this? Is it something related
> to pre-highres times of Linux (ie. 2.6.20 and earlier)? Can we overcome
> it, at least for recent kernels?
> 
> So, while this patch may work correctly, my feeling is it needs yet
> another round of polishing, also looking for now possibly outdated comments.
> 

...just this aspect needs to be addressed.

Jan

-- 
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux


  parent reply	other threads:[~2007-10-12  9:00 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-11 20:47 [Xenomai-core] [PATCH] fix hw-timer setup/cleanup for i386 Jan Kiszka
2007-10-12  8:58 ` Stelian Pop
2007-10-12  9:14   ` Jan Kiszka
2007-10-12 11:00     ` Stelian Pop
2007-10-12 11:15       ` Jan Kiszka
2007-10-12 13:19         ` Stelian Pop
2007-10-12 13:39           ` Jan Kiszka
2007-10-12 13:41           ` Philippe Gerum
2007-10-12 14:26             ` Stelian Pop
2007-10-12 19:53       ` Jan Kiszka
2007-10-12 20:22         ` Stelian Pop
2007-10-12 21:51           ` Stelian Pop
2007-10-12 21:58             ` Philippe Gerum
2007-10-13 14:28               ` Stelian Pop
2007-10-13 14:42                 ` Jan Kiszka
2007-10-13 16:12                   ` Stelian Pop
2007-10-13 16:35                     ` Philippe Gerum
2007-10-13 16:38                     ` Jan Kiszka
2007-10-13 20:55                       ` Stelian Pop
2007-10-13 14:32           ` Stelian Pop
2007-10-12  9:00 ` Jan Kiszka [this message]
2007-10-12  9:16 ` Philippe Gerum
2007-10-12  9:47   ` Jan Kiszka
2007-10-12 12:48     ` 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=470F37C8.4090301@domain.hid \
    --to=jan.kiszka@domain.hid \
    --cc=Xenomai-core@domain.hid \
    --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.