On 2014-07-21, Andi Kleen wrote: > Qiaowei Ren <qiaowei.ren@xxxxxxxxx> writes: >> + */ >> +#ifdef CONFIG_X86_64 >> + insn->x86_64 = 1; >> + insn->addr_bytes = 8; >> +#else >> + insn->x86_64 = 0; >> + insn->addr_bytes = 4; >> +#endif > > How would that handle compat mode on a 64bit kernel? > Should likely look at the code segment instead of ifdef. >> + /* Note: the upper 32 bits are ignored in 32-bit mode. */ > > Again correct for compat mode? I believe the upper bits are undefined. > Compat mode will be supported on next patch in future, as 0/ patch mentioned. ^-^ Thanks, Qiaowei -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href