From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756158AbbCSAVw (ORCPT ); Wed, 18 Mar 2015 20:21:52 -0400 Received: from ozlabs.org ([103.22.144.67]:37753 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754608AbbCSAVu (ORCPT ); Wed, 18 Mar 2015 20:21:50 -0400 Message-ID: <1426724509.14101.3.camel@ellerman.id.au> Subject: Re: [PATCH 2/2] selftests/timers: change to use shared logic to run and install tests From: Michael Ellerman To: Shuah Khan Cc: John Stultz , Thomas Gleixner , Linux API , lkml Date: Thu, 19 Mar 2015 11:21:49 +1100 In-Reply-To: <55099FD8.1080502@osg.samsung.com> References: <1426474128.20210.2.camel@ellerman.id.au> <55099FD8.1080502@osg.samsung.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.10-0ubuntu1~14.10.1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2015-03-18 at 09:55 -0600, Shuah Khan wrote: > On 03/15/2015 08:48 PM, Michael Ellerman wrote: > > On Fri, 2015-03-13 at 20:14 -0700, John Stultz wrote: > >> My only thoughts: > >> 1) Would it be better if threadtest can be made to have better > >> defaults for kselftest so you don't need that extra logic? > > > > That would help. But with the patch I just sent I think it's no bother, it's > > only a little extra logic and it's only in the timers Makefile. > > Let's go with a threadtest patch with better defaults. It will emit > scripts logic as well. Awesome. I just saw John's new patch in my > Inbox. Thanks. Fine by me. > >> 2) While I get that TEST_FILES is likely going to be used to copy the > >> destructive tests over, It feels a little like its being bundled in > >> with something like data files that tests might need, which seems sort > >> of hackish. Would TEST_PROGS_EXTENDED or something be more clear and > >> make more sense? > > > > That doesn't really bother me. You're right that TEST_FILES is originally > > intended for data files etc. but I don't think it's a big hack to use it for > > other tests that shouldn't be run by default. Still if it bothers you I'm happy > > to add a separate variable for it, they are cheap :) > > Could you please make the change from TEST_FILES to TEST_PROGS_EXTENDED > which is definitely better than overloading TEST_FILES? I don't see the point, but I don't care that much. > I think this would probably only change Add Install target patch. You > can send me patch v5 with that change and I will override the next with > your new one. Thanks for doing this. You shouldn't rebase your next branch, people may have already merged it (like me). You should apply it as an additional patch on top. Patch sent. cheers