On Tuesday 03 November 2015 02:30:43 Yury Norov wrote: > Generic 32-bit and AARCH32 stat64 structures has same names. > ILP32 needs generic stat64. So we can either make ILP32 mutual > exclusive with AARCH32, or introduce __stat64, and where needed, > cast stat64 to it with #define. > > Second option requires to duplicate cp_new_stat64 code. But we > choose it, because it's more flexible to have independend support > for ABIs. > > Signed-off-by: Yury Norov <yno...@caviumnetworks.com> > Reviewed-by: David Daney <david.da...@cavium.com>
I'm sorry I screwed up with the definition for stat64 in asm-generic, by defining it to have too short st_{a,m,c}time members. This means we have to introduce another 'struct stat64' variant already to solve the y2038 problem, and I have patches for it already, to make that implementation use the same structure as the 64-bit 'struct stat' with 64-bit timestamps. There is also an ongoing effort from David Howells to create a new 'struct statx' (or maybe xstat) system call with a completely new layout. I think here it would be better to avoid adding yet another version for ilp32 mode, and we are better off defining 'struct stat64' in the arm64 headers, and share either the normal arm32 syscall entry (with the default asm-generic syscall) or the normal arm64 syscall entry (as we will do for the future replacement). Arnd -- 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 http://www.tux.org/lkml/