[openib-general] Re: get_lock.h on ia64 support

[Grant Grundler]

On Mon, Jan 09, 2006 at 07:28:07PM +0200, Michael S. Tsirkin wrote:
> Hi!
> I was trying to build perftests on SLES9 SP1 (RC5).

Michael,
Why use SP1? Isn't SP2 the latest version?

> I was getting compilation errors: apparently on this distribution
> asm/timex.h includes all kind of kernel headers and
> so is not fit to be used to build userspace apps.

SLES9 doesn't support openib drivers - I'm not surprised there
are compilation problems.  Suse expects to ship OpenIB support in SLES10.
Are you building perf tests to run with mellanox drivers/libs ?

> I plan to check in the following patch to get us rid of this dependency
> on asm/timex.h for PPC64 and IA64.
> Note that get_cycles kernel function that we use just happens
> to be incline, it could move out of line at any point, so
> it might be a good idea to do this, regardless.

Yeah, I've been real comfortable including kernel asm files.
But it was the most portable way of getting the cycle counts.
If you feel strongly about it, I don't mind getting rid of
this dependency.

thanks,
grant