All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
To: rpm@xenomai.org
Cc: xenomai-core <xenomai@xenomai.org>
Subject: Re: [Xenomai-core] Handle SIGILL when kernel compiled without Xenomai.
Date: Sun, 28 Sep 2008 16:39:33 +0200	[thread overview]
Message-ID: <48DF9725.2070705@domain.hid> (raw)
In-Reply-To: <48DF9543.8010608@domain.hid>

Philippe Gerum wrote:
> Gilles Chanteperdrix wrote:
>> Philippe Gerum wrote:
>>> Gilles Chanteperdrix wrote:
>>>> Hi,
>>>>
>>>> on some (all ?) platforms, we get a SIGILL when trying to emit the first 
>>>> Xenomai syscall, instead of the -ENOSYS return value. This patches 
>>>> handles the SIGILL by printing an error message and exiting.
>>>>
>>> This is an issue for archs that encode the syscall number into the trap opcode
>>> like ARM using the OABI, others will get -ENOSYS as expected, so this should
>>> move to the arch-specific code.
>> Actually, I get a SIGILL on ARM compiled with EABI as well.
> 
> It seems the ARM folks decided to send SIGILL with EABI to conform to the OABI
> behaviour, despite they did not formally need that. Only invalid calls in the
> 9f00xx..9f07ff range are expected to return -ENOSYS. Too bad that our syscall
> marker is out of that range.
> 
>  Moving this
>> code to the arch dependent code looks hard, since we need to setup the
>> jump buffer in the very function which issues the syscall (Ok, we could
>> use a macro). But what would you think of a SIGILL handler which does
>> not longjmp (like Jan suggested, simply print an error message and exit,
>> even if xeno_bind_skin_opt was called) ?
>>
> 
> Fine with me.

Ok. Commited in trunk. Do I commit in v2.4.x branch?

-- 
					    Gilles.


  reply	other threads:[~2008-09-28 14:39 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-21 20:01 [Xenomai-core] Handle SIGILL when kernel compiled without Xenomai Gilles Chanteperdrix
2008-09-22  8:25 ` Philippe Gerum
2008-09-22 15:32   ` Gilles Chanteperdrix
2008-09-22 18:18     ` Gilles Chanteperdrix
2008-09-28 14:31     ` Philippe Gerum
2008-09-28 14:39       ` Gilles Chanteperdrix [this message]
2008-09-28 14:49         ` Philippe Gerum
2008-09-22  9:14 ` Jan Kiszka
2008-09-22  9:21   ` Gilles Chanteperdrix
2008-09-22 10:43     ` Jan Kiszka
2008-09-22 12:03       ` 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=48DF9725.2070705@domain.hid \
    --to=gilles.chanteperdrix@xenomai.org \
    --cc=rpm@xenomai.org \
    --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.