Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-15 Thread H. Peter Anvin
The generated/ prefix is wrong because when using O= to build we add $(srcdir)/arch/x86/include and $(objdir)/arch/x86/include/generated to the include path, not $(objdir)/arch/x86/include. On February 15, 2014 6:33:21 AM PST, Stefani Seibold wrote: > >Am Freitag, den 14.02.2014, 14:32 -0800

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-15 Thread Stefani Seibold
Am Freitag, den 14.02.2014, 14:40 -0800 schrieb Andy Lutomirski: > On Fri, Feb 14, 2014 at 12:52 AM, Stefani Seibold wrote: > > This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() > > and vdso_time() to the 32 bit VDSO. > > Is there any chance you can shove this in a git tree

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-15 Thread Stefani Seibold
Am Freitag, den 14.02.2014, 14:32 -0800 schrieb H. Peter Anvin: > I still get build errors. > Oops, i did it again... > i386 allyesconfig, i386 allmodconfig as well as a more basic i386 > configuration: > > arch/x86/vdso/vdso32-int80.so.dbg: undefined symbols found > make[4]: ***

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-15 Thread Stefani Seibold
Am Freitag, den 14.02.2014, 14:32 -0800 schrieb H. Peter Anvin: I still get build errors. Oops, i did it again... i386 allyesconfig, i386 allmodconfig as well as a more basic i386 configuration: arch/x86/vdso/vdso32-int80.so.dbg: undefined symbols found make[4]: ***

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-15 Thread Stefani Seibold
Am Freitag, den 14.02.2014, 14:40 -0800 schrieb Andy Lutomirski: On Fri, Feb 14, 2014 at 12:52 AM, Stefani Seibold stef...@seibold.net wrote: This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() and vdso_time() to the 32 bit VDSO. Is there any chance you can shove this in

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-15 Thread H. Peter Anvin
The generated/ prefix is wrong because when using O= to build we add $(srcdir)/arch/x86/include and $(objdir)/arch/x86/include/generated to the include path, not $(objdir)/arch/x86/include. On February 15, 2014 6:33:21 AM PST, Stefani Seibold stef...@seibold.net wrote: Am Freitag, den

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Greg KH
On Fri, Feb 14, 2014 at 02:40:12PM -0800, Andy Lutomirski wrote: > On Fri, Feb 14, 2014 at 12:52 AM, Stefani Seibold wrote: > > This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() > > and vdso_time() to the 32 bit VDSO. > > Is there any chance you can shove this in a git tree

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Andy Lutomirski
On Fri, Feb 14, 2014 at 12:52 AM, Stefani Seibold wrote: > This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() > and vdso_time() to the 32 bit VDSO. Is there any chance you can shove this in a git tree somewhere for the next version (in addition to sending patches)? I don't

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread H. Peter Anvin
I still get build errors. i386 allyesconfig, i386 allmodconfig as well as a more basic i386 configuration: arch/x86/vdso/vdso32-int80.so.dbg: undefined symbols found make[4]: *** [arch/x86/vdso/vdso32-int80.so.dbg] Error 1 x86-64 allyesconfig and x86-64 allmodconfig:

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Andi Kleen
This version looks good to me from a quick read. -Andi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

[PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Stefani Seibold
This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() and vdso_time() to the 32 bit VDSO. The reason to do this was to get a fast reliable time stamp. Many developers uses TSC to get a fast time stamp, without knowing the pitfalls. VDSO time functions a fast and a reliable way,

[PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Stefani Seibold
This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() and vdso_time() to the 32 bit VDSO. The reason to do this was to get a fast reliable time stamp. Many developers uses TSC to get a fast time stamp, without knowing the pitfalls. VDSO time functions a fast and a reliable way,

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Andi Kleen
This version looks good to me from a quick read. -Andi -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread H. Peter Anvin
I still get build errors. i386 allyesconfig, i386 allmodconfig as well as a more basic i386 configuration: arch/x86/vdso/vdso32-int80.so.dbg: undefined symbols found make[4]: *** [arch/x86/vdso/vdso32-int80.so.dbg] Error 1 x86-64 allyesconfig and x86-64 allmodconfig:

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Andy Lutomirski
On Fri, Feb 14, 2014 at 12:52 AM, Stefani Seibold stef...@seibold.net wrote: This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() and vdso_time() to the 32 bit VDSO. Is there any chance you can shove this in a git tree somewhere for the next version (in addition to sending

Re: [PATCH v16 0/10] Add 32 bit VDSO time function support

2014-02-14 Thread Greg KH
On Fri, Feb 14, 2014 at 02:40:12PM -0800, Andy Lutomirski wrote: On Fri, Feb 14, 2014 at 12:52 AM, Stefani Seibold stef...@seibold.net wrote: This patch add the functions vdso_gettimeofday(), vdso_clock_gettime() and vdso_time() to the 32 bit VDSO. Is there any chance you can shove this in