Linux NFS development
 help / color / mirror / Atom feed
From: Steve Dickson <SteveD@redhat.com>
To: Trond Myklebust <trond.myklebust@fys.uio.no>
Cc: nfs@lists.sourceforge.net
Subject: Re: [PATCH] -o intr  mount option prevents core dumps on 2.4 kernel
Date: Fri, 14 Jan 2005 14:52:35 -0500	[thread overview]
Message-ID: <41E82303.2070804@RedHat.com> (raw)
In-Reply-To: <1105671434.20314.3.camel@lade.trondhjem.org>



Trond Myklebust wrote:

>to den 13.01.2005 Klokka 16:37 (-0500) skreiv Steve Dickson:
>
>  
>
>>after further review.... it seems the reason your patch works on a 
>>2.4.28 kernel, is
>>because signalled() never returns true, even when the process is sent 
>>the SIGSEGV signal. Why signalled() never returns true, I don't know.... 
>>but the
>>reason your patch did not work on a RHEL3 kernel is because signalled() 
>>does
>>return true and calls to recalc_sigpending() do not reset that pending 
>>signal....
>>which seems to be a bug in the NTPL-related code.... code that is not in the
>>2.4.28 kernel....
>>    
>>
>
>recalc_sigpending() failing to honour the sigmask sounds like a pretty major bug! 
>  
>
Well it turns it was a bug in the kernel signaling code which was causing
this issue.... which means there are no changes needed to the NFS code,
which is always a good thing... :)

See bz 132162 if interested....

steved.





-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs

      reply	other threads:[~2005-01-14 19:52 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-06 18:31 [PATCH] -o intr mount option prevents core dumps on 2.4 kernel Steve Dickson
2005-01-07 17:14 ` Trond Myklebust
2005-01-11 19:36   ` Steve Dickson
2005-01-11 20:10     ` Trond Myklebust
2005-01-11 20:58       ` Steve Dickson
2005-01-11 21:59         ` Trond Myklebust
2005-01-12 18:18           ` Steve Dickson
2005-01-12 18:25             ` Trond Myklebust
2005-01-12 19:11               ` Steve Dickson
2005-01-13  0:38                 ` Trond Myklebust
2005-01-13 14:01                   ` Steve Dickson
2005-01-13 21:37                     ` Steve Dickson
2005-01-14  2:57                       ` Trond Myklebust
2005-01-14 19:52                         ` Steve Dickson [this message]

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=41E82303.2070804@RedHat.com \
    --to=steved@redhat.com \
    --cc=nfs@lists.sourceforge.net \
    --cc=trond.myklebust@fys.uio.no \
    /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