All of lore.kernel.org
 help / color / mirror / Atom feed
From: Grant Grundler <grundler@parisc-linux.org>
To: Kyle McMartin <kyle@mcmartin.ca>
Cc: Grant Grundler <grundler@parisc-linux.org>,
	Carlos O'Donell <carlos@systemhalted.org>,
	Helge Deller <deller@gmx.de>,
	linux-parisc <linux-parisc@vger.kernel.org>,
	John David Anglin <dave@hiauly1.hia.nrc.ca>
Subject: Re: futex.c and EWOULDBLOCK vs. EAGAIN patch
Date: Sun, 16 May 2010 10:12:07 -0600	[thread overview]
Message-ID: <20100516161207.GA21902@lackof.org> (raw)
In-Reply-To: <20100515230359.GF6764@bombadil.infradead.org>

On Sat, May 15, 2010 at 07:03:59PM -0400, Kyle McMartin wrote:
...
> > > *: CMPI,N(=), LDI, maybe a third since I don't remember how sign
> > > extension works off the top of my head.
> > 
> > Mispredicted branches cost something like 20-25 cycles. I forgot now but I
> > do know they aren't cheap.
> 
> That's not a branch, Grant.

DOh! sorry...you are right. /o\
I was totally missing the instruction nullification.

> > I'd rather not add more hacks for this they are unlikely to ever
> > be removed again. I'd rather have status quo than add some hack
> > to the syscall return path.
> 
> The alternative is to rebuild all of userspace, again.

All?
If I generate a list of debian packages that look at EWOULDBLOCK or EAGAIN,
couldn't we just regenerate those packages once the kernel change is in
place and the kernel header files pushed to debian experimental?
There can't be that many packages.

And of those that do, I believe most time the old binaries will generally
work since EWOULDBLOCK code paths are unlikely to get exercised. Or is
there evidence to the contrary?

thanks
grant

  reply	other threads:[~2010-05-16 16:12 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-10 19:41 futex.c and EWOULDBLOCK vs. EAGAIN patch Helge Deller
2010-05-14 19:20 ` Grant Grundler
2010-05-15 11:24   ` Carlos O'Donell
2010-05-15 17:27     ` Kyle McMartin
2010-05-15 22:59       ` Grant Grundler
2010-05-15 23:03         ` Kyle McMartin
2010-05-16 16:12           ` Grant Grundler [this message]
2010-05-16 16:25             ` Kyle McMartin
2010-05-17 19:39       ` Carlos O'Donell
2010-05-15 22:54     ` Grant Grundler
2010-05-16  0:38 ` John David Anglin

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=20100516161207.GA21902@lackof.org \
    --to=grundler@parisc-linux.org \
    --cc=carlos@systemhalted.org \
    --cc=dave@hiauly1.hia.nrc.ca \
    --cc=deller@gmx.de \
    --cc=kyle@mcmartin.ca \
    --cc=linux-parisc@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 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.