From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D8BFBA09E4; Thu, 28 Jan 2021 19:34:23 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7467840682; Thu, 28 Jan 2021 19:34:23 +0100 (CET) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mails.dpdk.org (Postfix) with ESMTP id 8313C4067A for ; Thu, 28 Jan 2021 19:34:21 +0100 (CET) IronPort-SDR: e1MUL6MSZaxjnER+eSpPArLNWRessKQjsTcK2RoLz4ch2XMTrwKxFIS5Z5PpZsc8Y2K+2ByL/d ExiM3TW/4AiQ== X-IronPort-AV: E=McAfee;i="6000,8403,9878"; a="241814763" X-IronPort-AV: E=Sophos;i="5.79,383,1602572400"; d="scan'208";a="241814763" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Jan 2021 10:34:20 -0800 IronPort-SDR: 6NFkVIf56adKVkuTkX71j8PcjQMTwfulCCmtooE/am7eMzXIxHmWndFWgTd4IlBGyHEP0F9OXC Gfi9LNn7Dh0w== X-IronPort-AV: E=Sophos;i="5.79,383,1602572400"; d="scan'208";a="354273084" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.11.53]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 28 Jan 2021 10:34:18 -0800 Date: Thu, 28 Jan 2021 18:34:15 +0000 From: Bruce Richardson To: Thomas Monjalon Cc: dev@dpdk.org, david.marchand@redhat.com, Aaron Conole , Michael Santana Message-ID: <20210128183415.GM899@bricha3-MOBL.ger.corp.intel.com> References: <20210114110606.21142-1-bruce.richardson@intel.com> <20210127173330.1671341-1-bruce.richardson@intel.com> <20210127173330.1671341-9-bruce.richardson@intel.com> <3749594.2Ve71FdZX2@thomas> <20210128114144.GF899@bricha3-MOBL.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210128114144.GF899@bricha3-MOBL.ger.corp.intel.com> Subject: Re: [dpdk-dev] [PATCH v6 8/8] ci: add checking of includes to CI builds X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, Jan 28, 2021 at 11:41:44AM +0000, Bruce Richardson wrote: > On Thu, Jan 28, 2021 at 12:12:44PM +0100, Thomas Monjalon wrote: > > 27/01/2021 18:33, Bruce Richardson: > > > For CI builds, turn on the checking of includes. > > > > > > Signed-off-by: Bruce Richardson Acked-by: > > > Aaron Conole --- .ci/linux-build.sh | 1 + 1 file > > > changed, 1 insertion(+) > > > > > > diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh index > > > afa3689a09..fdbeb5a616 100755 --- a/.ci/linux-build.sh +++ > > > b/.ci/linux-build.sh @@ -57,6 +57,7 @@ fi OPTS="$OPTS > > > -Dmachine=default" OPTS="$OPTS --default-library=$DEF_LIB" OPTS="$OPTS > > > --buildtype=debugoptimized" +OPTS="$OPTS -Dcheck_includes=true" > > > > Which percentage of time does it add on GHA? > > > I'd have to gather some before/after tests to know for sure, but to check > it wasn't too heavy-weight I was just monitoring the log at the end of a > build on GHA. The specific build I watched took nearly 15 minutes, and I > would guess-timate that the extra task took about 20 seconds of that. Related to this, and to ensure we don't increase the GHA build time massively, is there a reliably way to test this out? Looking at the daily builds on GHA I see some successful builds completing in 7.5 mins, while others take almost 18 minutes. The extra build time from this chkincs takes very little time on my own system but perhaps we want to limit it to a single build on GHA. I'm not very familiar with configuring those builds, so perhaps someone could suggest how to do so. Thanks, /Bruce