From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shuah Khan Subject: Re: [PATCH v4 20/20] kbuild: add a new kselftest_install make target to install selftests Date: Thu, 15 Jan 2015 16:41:32 -0700 Message-ID: <54B8502C.2040708@osg.samsung.com> References: <2c5a28faaa79d9c2415854a08817ada509fcb943.1420571615.git.shuahkh@osg.samsung.com> <54B69A1B.80006@osg.samsung.com> <1421362698.23332.6.camel@ellerman.id.au> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <1421362698.23332.6.camel-Gsx/Oe8HsFggBc27wqDAHg@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Michael Ellerman Cc: mmarek-AlSwsSmVLrQ@public.gmane.org, masami.hiramatsu.pt-FCd8Q96Dh0JBDgjK7y7TUQ@public.gmane.org, gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org, akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, rostedt-nx8X9YLhiw1AfugRpC6u6w@public.gmane.org, mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org, keescook-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org, tranmanphong-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, cov-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, dh.herrmann-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, hughd-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, bobby.prani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, serge.hallyn-GeWIH/nMZzLQT0dZR+AlfA@public.gmane.org, ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org, tim.bird-/MT0OVThwyLZJqsBc5GL+g@public.gmane.org, josh-iaAMLnmF4UmaiuxdJuQwMA@public.gmane.org, koct9i-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, linux-kbuild-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-api@vger.kernel.org On 01/15/2015 03:58 PM, Michael Ellerman wrote: > On Wed, 2015-01-14 at 09:32 -0700, Shuah Khan wrote: >> On 01/06/2015 12:43 PM, Shuah Khan wrote: >>> Add a new make target to install to install kernel selftests. >>> This new target will build and install selftests. kselftest >>> target now depends on kselftest_install and runs the generated >>> kselftest script to reduce duplicate work and for common look >>> and feel when running tests. >>> >>> make kselftest_target: >>> -- exports kselftest INSTALL_KSFT_PATH >>> default $(INSTALL_MOD_PATH)/lib/kselftest/$(KERNELRELEASE) >>> -- exports INSTALL_KSFT_PATH >>> -- runs selftests make install target >>> >>> Signed-off-by: Shuah Khan >> >> Hi Marek, >> >> Could you please Ack this patch, if this version looks good, >> so I can take this through ksefltest tree. > > Hi Shuah, > > I posted what I think is a better approach last week. It's less code and less > repetition, and installs the majority of tests this series missed - ie. the > powerpc ones. > > Please at least respond to it before you put this series into next. > > https://lkml.org/lkml/2015/1/9/45 > Michael, I didn't get a chance to review your patches yet. I plan to review and give you feedback by end of this week. This patch is needed in any case for either approach. thanks, -- Shuah -- Shuah Khan Sr. Linux Kernel Developer Open Source Innovation Group Samsung Research America (Silicon Valley) shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org | (970) 217-8978 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.s-osg.org ([54.187.51.154]:39619 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932072AbbAOXll (ORCPT ); Thu, 15 Jan 2015 18:41:41 -0500 Message-ID: <54B8502C.2040708@osg.samsung.com> Date: Thu, 15 Jan 2015 16:41:32 -0700 From: Shuah Khan MIME-Version: 1.0 Subject: Re: [PATCH v4 20/20] kbuild: add a new kselftest_install make target to install selftests References: <2c5a28faaa79d9c2415854a08817ada509fcb943.1420571615.git.shuahkh@osg.samsung.com> <54B69A1B.80006@osg.samsung.com> <1421362698.23332.6.camel@ellerman.id.au> In-Reply-To: <1421362698.23332.6.camel@ellerman.id.au> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Michael Ellerman Cc: mmarek@suse.cz, masami.hiramatsu.pt@hitachi.com, gregkh@linuxfoundation.org, akpm@linux-foundation.org, rostedt@goodmis.org, mingo@redhat.com, davem@davemloft.net, keescook@chromium.org, tranmanphong@gmail.com, cov@codeaurora.org, dh.herrmann@gmail.com, hughd@google.com, bobby.prani@gmail.com, serge.hallyn@ubuntu.com, ebiederm@xmission.com, tim.bird@sonymobile.com, josh@joshtriplett.org, koct9i@gmail.com, linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, linux-api@vger.kernel.org, netdev@vger.kernel.org On 01/15/2015 03:58 PM, Michael Ellerman wrote: > On Wed, 2015-01-14 at 09:32 -0700, Shuah Khan wrote: >> On 01/06/2015 12:43 PM, Shuah Khan wrote: >>> Add a new make target to install to install kernel selftests. >>> This new target will build and install selftests. kselftest >>> target now depends on kselftest_install and runs the generated >>> kselftest script to reduce duplicate work and for common look >>> and feel when running tests. >>> >>> make kselftest_target: >>> -- exports kselftest INSTALL_KSFT_PATH >>> default $(INSTALL_MOD_PATH)/lib/kselftest/$(KERNELRELEASE) >>> -- exports INSTALL_KSFT_PATH >>> -- runs selftests make install target >>> >>> Signed-off-by: Shuah Khan >> >> Hi Marek, >> >> Could you please Ack this patch, if this version looks good, >> so I can take this through ksefltest tree. > > Hi Shuah, > > I posted what I think is a better approach last week. It's less code and less > repetition, and installs the majority of tests this series missed - ie. the > powerpc ones. > > Please at least respond to it before you put this series into next. > > https://lkml.org/lkml/2015/1/9/45 > Michael, I didn't get a chance to review your patches yet. I plan to review and give you feedback by end of this week. This patch is needed in any case for either approach. thanks, -- Shuah -- Shuah Khan Sr. Linux Kernel Developer Open Source Innovation Group Samsung Research America (Silicon Valley) shuahkh@osg.samsung.com | (970) 217-8978