Re: [PATCH] readahead: initial method - expected read size - fixfastcall

From: Arjan van de Ven
Date: Sun Jun 04 2006 - 05:24:21 EST


On Sun, 2006-06-04 at 02:07 -0700, Andrew Morton wrote:
> On Sun, 4 Jun 2006 15:34:15 +0800
> Fengguang Wu <wfg@xxxxxxxxxxxxxxxx> wrote:
>
> > Remove 'fastcall' directive for function readahead_close().
> >
> > It has drawn concerns from Andrew Morton.
>
> Well. I think fastcall is ugly and vaguely silly. Now if we has a
> really_really_fastcall then I'd like to use that!
>
>
> > Now I have some benchmarks
> > on it, and proved it as a _false_ optimization.
>
> Sorry, I don't believe this will be measurable (and with CONFIG_REGPARM
> it'll be a no-op).

we should just make CONFIG_REGPARM be "it" always (and thus make it go
away as config option) and then just remove all "fastcall" from the
kernel...

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/