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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux