From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shuah Khan Subject: Re: [PATCH] selftests/memfd: Run test on all architectures Date: Wed, 17 Sep 2014 09:41:16 -0600 Message-ID: <5419AB9C.5040409@osg.samsung.com> References: <1410821984-11160-1-git-send-email-bobby.prani@gmail.com> <5419AA71.4020506@osg.samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: David Herrmann Cc: Pranith Kumar , Andrew Morton , Hugh Dickins , Phong Tran , "open list:KERNEL SELFTEST F..." , open list List-Id: linux-api@vger.kernel.org On 09/17/2014 09:39 AM, David Herrmann wrote: > Hi >=20 > On Wed, Sep 17, 2014 at 5:36 PM, Shuah Khan = wrote: >> On 09/17/2014 06:28 AM, David Herrmann wrote: >>> Hi >>> >>> On Tue, Sep 16, 2014 at 12:59 AM, Pranith Kumar wrote: >>>> Remove the dependence on x86 to run the memfd test. Verfied on 32-= bit powerpc. >>>> >>>> Signed-off-by: Pranith Kumar >>> >>> Now that most archs have __NR_memfd_create, this is: >>> >>> Reviewed-by: David Herrmann >>> >> >> Hmm. I am seeing failures on x86_64 on top of Linus's tree >> (without this patch) >> >> make -C tools/testing/selftests/memfd/make: Entering directory >> `/lkml/linus_git_3.14/tools/testing/selftests/memfd' >> gcc -D_FILE_OFFSET_BITS=3D64 >> -I../../../../arch/x86/include/generated/uapi/ >> -I../../../../arch/x86/include/uapi/ -I../../../../include/uapi/ >> -I../../../../include/ memfd_test.c -o memfd_test >> memfd_test.c: In function =E2=80=98sys_memfd_create=E2=80=99: >> memfd_test.c:26:17: error: =E2=80=98__NR_memfd_create=E2=80=99 undec= lared (first use in >> this function) >> return syscall(__NR_memfd_create, name, flags); >> ^ >> memfd_test.c:26:17: note: each undeclared identifier is reported onl= y >> once for each function it appears in >> >> Could you please look into this?? It appears __NR_memfd_create isn't >> defined on x86_64??? >=20 > You need the headers installed in your system. By dropping the > arch-includes from the Makefile, we require 3.17 headers in > /usr/include. >=20 Sorry - wrong system without the headers installed. My bad. -- Shuah --=20 Shuah Khan Sr. Linux Kernel Developer Samsung Research America (Silicon Valley) shuahkh@osg.samsung.com | (970) 217-8978