From: Neil Armstrong <narmstrong@domain.hid>
To: xenomai@xenomai.org
Subject: Re: [Xenomai-core] Porting Ipipe/Adeos patch to new arm9 board
Date: Fri, 28 Mar 2008 16:51:39 +0100 [thread overview]
Message-ID: <47ED140B.2040406@domain.hid> (raw)
In-Reply-To: <2ff1a98a0803280811n702cf80ble01c121959bbcf21@domain.hid>
[-- Attachment #1.1: Type: text/plain, Size: 1469 bytes --]
First of all, thanks for your aswer,
Gilles Chanteperdrix a écrit :
>> I patched the kernel with the arm1 patch (from the ARM website), the
>> board non-rt patch enabling serial and the macb ethernet controller and
>> finnally tried to apply the ipipe arm patch and resolving failures.
>
> At first sight, you call spin_lock_irqsave in __ipipe_mach_set_dec.
> This is illegal, __ipipe_mach_set_dec may (and will) be called from
> real-time context, you should be using the _hw variants. I do not know
> if it is what causes the oops, but it would cause an oops later.
I based my board patch on integrator's and s3c24xx code and they use
spin_lock_irqsave in __ipipe_mach_set_dec, but I found local_irq_save_hw
used on an AT91 patch.
Finally the change did nothing, are the integrator and s3c24xx patchs
up-to-date ?
Should I drop the arm1 patch from ARM ? and start from a fresh vanilla
kernel ?
> I am not sure it makes sense to set the timer auto-reload mode when
> the timer is used in one-shot mode: imagine that the timer is
> programmed in one-shot mode, it will tick several time before the
> interrupt handler is called.
>
Integrator seems using the timer in one-shot mode and changes the
prescaler with the delay value. But won't I loose to much timer ticks
between two reloads ?
Neil
--
Neil Armstrong
Neotion Sophia Antipolis
Stagiaire Mars-Septembre 2008 Polytech'Nice-Sophia
narmstrong@domain.hid (PGP:0x1166F485)
[-- Attachment #1.2: narmstrong.vcf --]
[-- Type: text/x-vcard, Size: 256 bytes --]
begin:vcard
fn:Neil Armstrong
n:Armstrong;Neil
org:Neotion;Neotion Sophia Antipolis
email;internet:narmstrong@domain.hid
title;quoted-printable:Ing=C3=A9nieur Software Embarqu=C3=A9
tel;cell:0667474169
note:PGP 0x1166F485
version:2.1
end:vcard
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
next prev parent reply other threads:[~2008-03-28 15:51 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-28 9:38 [Xenomai-core] Porting Ipipe/Adeos patch to new arm9 board Neil Armstrong
2008-03-28 15:11 ` Gilles Chanteperdrix
2008-03-28 15:18 ` Gilles Chanteperdrix
2008-03-28 15:51 ` Neil Armstrong [this message]
2008-03-28 16:13 ` Gilles Chanteperdrix
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=47ED140B.2040406@domain.hid \
--to=narmstrong@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.