LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: jafo@tummy.com
Cc: linux-kernel@vger.kernel.org
Subject: Re: select() setting ERESTARTNOHAND (514).
Date: Wed, 10 Jan 2007 17:15:20 -0800 (PST)	[thread overview]
Message-ID: <20070110.171520.23015257.davem@davemloft.net> (raw)
In-Reply-To: <20070111010429.GN7121@tummy.com>

From: Sean Reifschneider <jafo@tummy.com>
Date: Wed, 10 Jan 2007 18:04:29 -0700

> On Wed, Jan 10, 2007 at 04:27:47PM -0800, David Miller wrote:
> >It gets caught by the return into userspace code.
> 
> Ok, so somehow it is leaking.  I have a system in the lab that is the same
> hardware as production, but it currently has no, you know, hard drives in
> it, so some assembly is required.  I'll see if I can reproduce it in a test
> environment and then see if I can get more information on when/where it is
> leaking.

If you're only seeing it in strace, that's expected due to some
unfortunate things in the way that x86 and x86_64 handle signal
return events via ptrace().

On sparc and sparc64 I fixed this long ago such that ptrace() will
update the user registers before ptrace parents are notified, and
therefore you'll never see those kernel internal error codes.

The upside of this is that you'll really need to see what value is
making it to the application.  What the kernel is probably
doing is looping trying to restart the system call and sending
the signal.  If it's doing that the application is being rewound
to call the system call again once the signal handler returns
(if that is even being run at all).

  reply	other threads:[~2007-01-11  1:15 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-10 23:42 Sean Reifschneider
2007-01-11  0:27 ` David Miller
2007-01-11  1:04   ` Sean Reifschneider
2007-01-11  1:15     ` David Miller [this message]
2007-01-11  8:25       ` Sean Reifschneider
2007-01-11 22:22         ` bert hubert
2007-01-24  0:50           ` Sean Reifschneider
2007-01-11  0:37 ` PATCH - x86-64 signed-compare bug, was " Neil Brown
2007-01-11  0:40   ` Andi Kleen
2007-01-11  1:02     ` Neil Brown
2007-01-11  1:37       ` Andi Kleen
2007-01-11  4:09       ` Sean Reifschneider
2007-01-11 19:38       ` Denis Vlasenko
2007-01-11  0:43   ` David Miller

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=20070110.171520.23015257.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=jafo@tummy.com \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: select() setting ERESTARTNOHAND (514).' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).