Re: [openib-general] Re: [PATCH 13/13] [RFC] ipath Kconfig and Makefile

From: Adrian Bunk
Date: Sat Dec 17 2005 - 18:57:27 EST


On Sat, Dec 17, 2005 at 02:54:44PM -0800, Robert Walsh wrote:
> > The driver shouldn't use assembler code and therefore no longer depend
> > on X86_64.
>
> Agreed about the assembler, but one way or the other, x86_64 is the only
> arch we support.
>...

There's a difference between "technically supported by the driver" and
"officially supported for our costumers":

It's fine if you tell the costumers buying your hardware "anything else
than 64bit x86_64 kernels is completely unsupported", but for getting
your driver included into the kernel it should be 32bit clean [1] and
should also work for people using 32bit kernels on an Opteron.

> > -O3 doesn't make much sense since the fight for producing the fastest
> > code is between -O2 and -Os.
>
> Makes many nanoseconds of difference to us for our latency numbers. At
> the low latency numbers we measuring (1.29us), this is a very important
> difference to our customers.
>...

There's no doubt that this is important for your customers.

What surprises me is that -O3 turned out to be the fastest flag for you.

Can you send numbers comparing -Os/-O2/-O3 (without -g3, preferable with
gcc 4.0) including a description what and how you are measuring?

> Regards,
> Robert.

cu
Adrian

[1] not long ago, it used to be the other way round that drivers weren't
64bit clean...

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
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/