public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: Jack Steiner <steiner@sgi.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [Linux-ia64] Re: prctl patch for fpu faults
Date: Wed, 10 Oct 2001 13:59:36 +0000	[thread overview]
Message-ID: <marc-linux-ia64-105590698805316@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-105590698805302@msgid-missing>

> 
> What if I were to get rid of the rate limited logging for both the
> unaligned and fpswa handlers?  Then there could just be a NOPRINT option
> and a signal option for each; the default behavior would be to log all
> messages.  If you'd rather not get rid of it, then I'll just send a patch
> to enable NOPRINT for fpswa (similar to what the unaligned handler does
> now) and a signal option.

IMHO, the default for FPSWA fault (& probably unaligned access) should be OFF.
Lots of apps cause FP faults. AIM7, for example, causes faults at a fairly high
rate. I doubt that any server site would like to see the log polluted with 
these errors - especially since there is no way to identify the offending app.

There should be a way a site can change the default logging behavior & have it
apply to ALL tasks.. 


Remember that FPSWA is invoked for numerous reasons that are NOT necessarily 
errors in the application. (AIM7 generates FPSWA faults when it divides a
very small number by 2.0). Intel doesnt even specify all the conditions that cause
FPSWA to be invoked.

As far as rate limited vs full logging, I like rate limited. Otherwise, you can
flood the system with messages. If you are truly trying to debug an app & identify 
places that cause faults, the SIGNAL method should be used by the app. 
That way, additional info such as traceback & arguments can be captured. 

---

After thinking about it for a while, it seems to me that the kernel logging options
should be orthoginal to the user SIGNAL option. A site should be able to select
NOPRINT/RATE-LOGGING/FULL_LOGGING. Independently, the user to be able to select 
SIGNAL/NO_SIGNAL/PRINTALL. 

The kernel logging option should NOT be a task attribute. Just have
a static flag that controls logging. The value of the flag is 
NOPRINT/RATEPRINT/FULLPRINT. Default should be NOPRINT. This option should
be able to be changed via operator command (/proc).

Independently, users might want to select SIGNAL/NOSIGNAL/(PRINT?) 
for faults from their application. 


> 
> Does anyone else have an opinion on the rate limited logging for the
> unaligned and fpswa handlers?  Some people have told me that they don't
> think it's useful for admins or users to have some of the messages but not
> all of them.
> 
> Thanks,
> Jesse
> 
> On Tue, 9 Oct 2001, David Mosberger wrote:
> 
> > >>>>> On Tue, 9 Oct 2001 10:38:57 -0700, Jesse Barnes <jbarnes@sgi.com> said:
> > 
> >   Jesse> Wouldn't that mean you could only do one at a time,
> >   Jesse> i.e. you'd be stuck with the rate limiting code even if you
> >   Jesse> just wanted a signal, since there'd be no way to say NOPRINT
> >   Jesse> | SIGFPE?  I'm really just trying to fill the needs of our
> >   Jesse> application programmers, who say they want to get all the
> >   Jesse> messages and/or get a signal.
> > 
> > No, I don't think so:
> > 
> > 	DEFAULT:	log message (with rate limit)
> > 	NOPRINT:	be quiet about fixups (not rate limited)
> > 	SIGNAL:		send signal (not rate limited)
> > 	PRINTALL:	log message (not rate limited)
> > 
> > In other words, the rate-limiting only applies for the DEFAULT setting.
> > 
> > 	--david
> > 
> 
> 
> _______________________________________________
> Linux-IA64 mailing list
> Linux-IA64@linuxia64.org
> http://lists.linuxia64.org/lists/listinfo/linux-ia64
> 


-- 
Thanks

Jack Steiner    (651-683-5302)   (vnet 233-5302)      steiner@sgi.com



  parent reply	other threads:[~2001-10-10 13:59 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-10-08 23:42 [Linux-ia64] Re: prctl patch for fpu faults David Mosberger
2001-10-08 23:46 ` Jesse Barnes
2001-10-08 23:54 ` David Mosberger
2001-10-08 23:55 ` Jesse Barnes
2001-10-08 23:57 ` Jesse Barnes
2001-10-09  0:11 ` Jesse Barnes
2001-10-09 21:40 ` Jesse Barnes
2001-10-10 13:59 ` Jack Steiner [this message]
2001-10-10 14:18 ` n0ano
2001-10-10 16:34 ` Jesse Barnes
2001-10-10 20:18 ` Mallick, Asit K
2001-10-10 20:26 ` David Mosberger
2001-10-10 20:42 ` Keith Fish
2001-10-10 22:27 ` David Mosberger
2001-10-10 22:31 ` Jesse Barnes
2001-10-10 22:51 ` David Mosberger
2001-10-10 22:53 ` Khalid Aziz
2001-10-10 22:57 ` Jesse Barnes
2001-10-10 23:05 ` David Mosberger
2001-10-11  3:11 ` Mallick, Asit K
2001-10-11  3:21 ` David Mosberger

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=marc-linux-ia64-105590698805316@msgid-missing \
    --to=steiner@sgi.com \
    --cc=linux-ia64@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox