From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751867AbdBBXgU (ORCPT ); Thu, 2 Feb 2017 18:36:20 -0500 Received: from ec2-52-27-115-49.us-west-2.compute.amazonaws.com ([52.27.115.49]:50858 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751500AbdBBXgT (ORCPT ); Thu, 2 Feb 2017 18:36:19 -0500 Subject: Re: [PATCH 1/4] selftests, x86, protection_keys: fix unused variable compile warnings To: Dave Hansen , linux-kernel@vger.kernel.org References: <20170202232842.C3C30DB2@viggo.jf.intel.com> <20170202232844.BFD31FAD@viggo.jf.intel.com> Cc: x86@kernel.org, Shuah Khan From: Shuah Khan Message-ID: <0262a979-6051-7007-f119-f45ede9449b7@osg.samsung.com> Date: Thu, 2 Feb 2017 16:36:16 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20170202232844.BFD31FAD@viggo.jf.intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/02/2017 04:28 PM, Dave Hansen wrote: > From: Shuah Khan > > Fix unused variable compile warnings in protection_keys.c > > Signed-off-by: Shuah Khan > Signed-off-by: Dave Hansen Hi Dave, This patches is already in linux-kselftest next for 4.11 Is there a reason why you chose to resend these. -- Shuah > --- > > b/tools/testing/selftests/x86/protection_keys.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff -puN tools/testing/selftests/x86/protection_keys.c~selftests-x86-protection-keys-fix-unused-variable-compile-warnings tools/testing/selftests/x86/protection_keys.c > --- a/tools/testing/selftests/x86/protection_keys.c~selftests-x86-protection-keys-fix-unused-variable-compile-warnings 2017-02-02 15:27:12.898110827 -0800 > +++ b/tools/testing/selftests/x86/protection_keys.c 2017-02-02 15:27:12.901110962 -0800 > @@ -812,8 +812,6 @@ void setup_hugetlbfs(void) > { > int err; > int fd; > - int validated_nr_pages; > - int i; > char buf[] = "123"; > > if (geteuid() != 0) { > @@ -1133,7 +1131,6 @@ void test_pkey_syscalls_on_non_allocated > void test_pkey_syscalls_bad_args(int *ptr, u16 pkey) > { > int err; > - int bad_flag = (PKEY_DISABLE_ACCESS | PKEY_DISABLE_WRITE) + 1; > int bad_pkey = NR_PKEYS+99; > > /* not enforced when pkey_get() is not a syscall > @@ -1149,8 +1146,6 @@ void test_pkey_syscalls_bad_args(int *pt > /* Assumes that all pkeys other than 'pkey' are unallocated */ > void test_pkey_alloc_exhaust(int *ptr, u16 pkey) > { > - unsigned long flags; > - unsigned long init_val; > int err; > int allocated_pkeys[NR_PKEYS] = {0}; > int nr_allocated_pkeys = 0; > _ >