Re: [PATCH 3/3] Use conditional

From: Kurt Garloff (garloff@private)
Date: Sun Jul 03 2005 - 14:17:32 PDT


Hi James,

[added linux-security-module@private to Cc]

On Sun, Jul 03, 2005 at 12:51:20PM -0400, James Morris wrote:
> On Sun, 3 Jul 2005, Kurt Garloff wrote:
> 
> > capablities is used. These are not called via indirect calls but 
> > called as hardcoded calls and might thus be inlined; the price for
> > this is a conditional -- benchmarks done by hp showed this to be
> > beneficial (on ia64).
> 
> What about on i386, x86_64 or ppc64?

We tested on i386 as well at the time, and it looked like a tiny
improvement. But doing the statistics, it was in the noise. 
I have no numbers for x86_64 or ppc64.

If you have reason to believe that there could be regressions, we 
should indeed do the benchmarks.

Sidenote: The patches 1 -- 2b alone still make sense, so I would
vote not for delaying their inclusion until we can collect numbers
for all arches we care about to take a decision on patch 3.

Best,
-- 
Kurt Garloff, Director SUSE Labs, Novell Inc.





This archive was generated by hypermail 2.1.3 : Sun Jul 03 2005 - 14:20:06 PDT