From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751303AbdJDUYZ (ORCPT ); Wed, 4 Oct 2017 16:24:25 -0400 Received: from mx1.redhat.com ([209.132.183.28]:45284 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750779AbdJDUYY (ORCPT ); Wed, 4 Oct 2017 16:24:24 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com BFBAD7E381 Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=swood@redhat.com Message-ID: <1507148663.2256.26.camel@redhat.com> Subject: Re: [PATCH 8/8] ktest: Use config-bisect.pl in ktest.pl From: Scott Wood To: Steven Rostedt Cc: linux-kernel@vger.kernel.org Date: Wed, 04 Oct 2017 15:24:23 -0500 In-Reply-To: <20171004151832.512f774e@gandalf.local.home> References: <20170717001630.10518-1-swood@redhat.com> <20170717001630.10518-8-swood@redhat.com> <20171004151832.512f774e@gandalf.local.home> Organization: Red Hat Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Wed, 04 Oct 2017 20:24:24 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2017-10-04 at 15:18 -0400, Steven Rostedt wrote: > On Sun, 16 Jul 2017 19:16:30 -0500 > Scott Wood wrote: > > > Reduce code duplication and take advantage of bisection logic > > improvements by calling config-bisect.pl. > > > > The output of make oldconfig is now copied directly to the desired > > file, > > rather than doing assign_configs+save_config, in order to preserve > > the > > ordering so that diffing the configs at the end will provide useful > > output. > > The reason I never did this, was that I copy ktest.pl all over the > place :-/ I need it to be a stand alone. I don't copy config-bisect > around. Not sure how to deal with this. By "this" I assume you mean using the external config-bisect code, rather than the part about copying oldconfig output? The options I can see are to copy both files as a group (possibly making that easier by renaming config-bisect.pl to ktest-conf-bisect.pl, so you can "cp tools/testing/ktest/ktest*.pl "), or just living with code duplication. In a previous discussion you suggested you preferred that latter option, though in that case the config bisect logic changes should ideally be done before the fork. It should also be noted that ktest.pl only depends on config-bisect.pl if a config bisect is being performed, so other ktest.pl functions still work standalone. -Scott