On Sat, Feb 15, 2020 at 08:07:52AM +0100, Andrew Jones wrote: > On Fri, Feb 14, 2020 at 05:26:39PM -0500, Peter Xu wrote: > > On Fri, Feb 14, 2020 at 03:59:07PM +0100, Andrew Jones wrote: > > > This series has several parts: > > > > > > * First, a hack to get x86 to compile. The missing __NR_userfaultfd > > > define should be fixed a better way. > > > > Yeh otherwise I think it will only compile on x86_64. > > The opposite for me. I could compile on AArch64 without this hack, but on > x86 (my Fedora 30 laptop) I could not. Ah, then probably because ARM does not have that artificial unisted*.h defined (tools/arch/x86/include/asm/unistd_{32|64}.h for x86), then <sys/syscall.h> can find the correct headers. And I have said it wrong above... with patch 1 compilation should always work, but IIUC the syscall number will be wrong on 32bit systems. Maybe we still need the other solution to make the test runnable on all platforms by removing those two artificial headers. Thanks, -- Peter Xu