Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966016AbbBCPcr (ORCPT ); Tue, 3 Feb 2015 10:32:47 -0500 Received: from lists.s-osg.org ([54.187.51.154]:46651 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753568AbbBCPcp (ORCPT ); Tue, 3 Feb 2015 10:32:45 -0500 Message-ID: <54D0EA1B.1000508@osg.samsung.com> Date: Tue, 03 Feb 2015 08:32:43 -0700 From: Shuah Khan Organization: Samsung Open Source Group User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: David Drysdale , Michael Ellerman CC: "linux-kernel@vger.kernel.org" , Linux API , Geert Uytterhoeven , davej@codemonkey.org.uk Subject: Re: [PATCH v2] selftests/exec: Check if the syscall exists and bail if not References: <1422935588-9973-1-git-send-email-mpe@ellerman.id.au> In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1125 Lines: 32 On 02/03/2015 12:58 AM, David Drysdale wrote: > On Tue, Feb 3, 2015 at 3:53 AM, Michael Ellerman wrote: >> On systems which don't implement sys_execveat(), this test produces a >> lot of output. >> >> Add a check at the beginning to see if the syscall is present, and if >> not just note one error and return. >> >> When we run on a system that doesn't implement the syscall we will get >> ENOSYS back from the kernel, so change the logic that handles >> __NR_execveat not being defined to also use ENOSYS rather than -ENOSYS. >> >> Signed-off-by: Michael Ellerman > > Acked-by: David Drysdale Thanks Michael, and David. I will queue this for 3.20 -- Shuah -- Shuah Khan Sr. Linux Kernel Developer Open Source Innovation Group Samsung Research America (Silicon Valley) shuahkh@osg.samsung.com | (970) 217-8978 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/