Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933647AbbBJBC1 (ORCPT ); Mon, 9 Feb 2015 20:02:27 -0500 Received: from lists.s-osg.org ([54.187.51.154]:55843 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932287AbbBJBCZ (ORCPT ); Mon, 9 Feb 2015 20:02:25 -0500 Message-ID: <54D958A0.1020404@osg.samsung.com> Date: Mon, 09 Feb 2015 18:02:24 -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: Michael Ellerman CC: Linus Torvalds , linux-kernel@vger.kernel.org, linux-api@vger.kernel.org Subject: Re: [GIT PULL] Kselftest updates for 3.20-rc1 References: <54D8FE1C.5040303@osg.samsung.com> <1423528239.19657.3.camel@ellerman.id.au> <54D95271.7070708@osg.samsung.com> <1423529039.19657.6.camel@ellerman.id.au> In-Reply-To: <1423529039.19657.6.camel@ellerman.id.au> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4475 Lines: 111 On 02/09/2015 05:43 PM, Michael Ellerman wrote: > On Mon, 2015-02-09 at 17:36 -0700, Shuah Khan wrote: >> On 02/09/2015 05:30 PM, Michael Ellerman wrote: >>> On Mon, 2015-02-09 at 11:36 -0700, Shuah Khan wrote: >>>> Hi Linus, >>>> >>>> Please pull the following Kselftest updates for 3.20-rc1 >>>> >>>> thanks, >>>> -- Shuah >>>> >>>> The following changes since commit 97bf6af1f928216fd6c5a66e8a57bfa95a659672: >>>> >>>> Linux 3.19-rc1 (2014-12-20 17:08:50 -0800) >>>> >>>> are available in the git repository at: >>>> >>>> git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest >>>> tags/linux-kselftest-3.20-rc1 >>>> >>>> for you to fetch changes up to 6ddf898c23d62c974e148efd9e509731324a167a: >>>> >>>> selftests/exec: Check if the syscall exists and bail if not >>>> (2015-02-04 10:17:35 -0700) >>>> >>>> ---------------------------------------------------------------- >>>> Kselftest updates for 3.20-rc1 >>>> >>>> This update adds: >>>> - Kselftest install target feature >>>> - Fix for selftests/exec test >>>> >>>> ---------------------------------------------------------------- >>>> Michael Ellerman (1): >>>> selftests/exec: Check if the syscall exists and bail if not >>>> >>>> Shuah Khan (20): >>>> selftests/breakpoints: add install target to enable test install >>>> selftests/cpu-hotplug: add install target to enable test install >>>> selftests/efivarfs: add install target to enable test install >>>> selftests/firmware: add install target to enable test install >>>> selftests/ftrace: add install target to enable test install >>>> selftests/ipc: add install target to enable test install >>>> selftests/kcmp: add install target to enable test install >>>> selftests/memfd: add install target to enable test install >>>> selftests/memory-hotplug: add install target to enable test install >>>> selftests/mount: add install target to enable test install >>>> selftests/mqueue: add install target to enable test install >>>> selftests/net: add install target to enable test install >>>> selftests/ptrace: add install target to enable test install >>>> selftests/size: add install target to enable test install >>>> selftests/sysctl: add install target to enable test install >>>> selftests/timers: add install target to enable test install >>>> selftests/user: add install target to enable test install >>>> selftests/vm: add install target to enable test install >>>> selftests: add install target to enable test install >>>> kbuild: add a new kselftest_install make target to install selftests >>> >>> >>> I don't understand why you insist on merging this series with the logic copied >>> 18 times. >>> >>> I'm happy to tweak my series that uses an include file, but I don't see the >>> point of merging this series first when almost every line will be removed when >>> my series goes in. >> >> Please work on the suggestions I made and rework the patches >> and resend. As I mentioned earlier, I want to enable this work >> and them make improvements. > > Yes I would like install to work to. I'd also like it to work for the powerpc > tests you ignored. But I don't want it to involve copying the same logic into > every Makefile in the tree. Michael, powerpc tests aren't ignored. They are in the list to do as the next step. > > My series was sent over a month ago, with plenty of time for you to merge it > instead of this cut-and-paste solution. I asked you to re-work the patches based on my suggestions and resend. I didn't see any patches from you that addressed the comments. I can't merge the patches you sent without addressing the comments. I want to get this feature implemented in this series as the first step and then make improvements that are isolated to the selftests makefile hierarchy. This series I am requesting to be pulled in has been in progress for a while now and the series is v4. Please plan upon re-working resending the patches for the next release. thanks, -- 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/