From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id D5FC6374E for ; Wed, 13 Sep 2017 17:00:33 +0200 (CEST) Received: from [2606:a000:111b:41f4:2201:d2d7:1fb:4ffb] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1ds99C-0001TC-Df; Wed, 13 Sep 2017 11:00:28 -0400 Date: Wed, 13 Sep 2017 11:00:13 -0400 From: Neil Horman To: Olivier Matz Cc: dev@dpdk.org, bruce.richardson@intel.com Message-ID: <20170913150013.GB4320@hmswarspite.think-freely.org> References: <20170911081801.tnf7qtt4qq5jkofe@neon> <20170911084635.11707-1-olivier.matz@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170911084635.11707-1-olivier.matz@6wind.com> User-Agent: Mutt/1.8.3 (2017-05-23) X-Spam-Score: -2.9 (--) X-Spam-Status: No Subject: Re: [dpdk-dev] [PATCH v3] devtools: rework abi checker script X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Sep 2017 15:00:34 -0000 On Mon, Sep 11, 2017 at 10:46:35AM +0200, Olivier Matz wrote: > The initial version of the script had some limitations: > - cannot work on a non-clean workspace > - environment variables are not documented > - no compilation log in case of failure > - return success even it abi is incompatible > > This patch addresses these issues and rework the code. > > Signed-off-by: Olivier Matz > --- > > v2->v3: > - fix when not launched from dpdk root dir > - use "-Og -Wno-error" instead of "-O0" > - fix typo in commit log > > v1->v2: > - use /usr/bin/env to find bash (which is required) > - fix displayed path to html reports > - reword help for -f option > > devtools/validate-abi.sh | 392 ++++++++++++++++++++++++----------------------- > 1 file changed, 200 insertions(+), 192 deletions(-) > This is alot closer, I think theres just a little wierdness left. When running this checker script, the end of the log shows this: CMD: abi-compliance-checker -l librte_vhost.so -old /home/nhorman/git/dpdk/devtools/abi-check/222555480/librte_vhost.so.dump -new /home/nhorman/git/dpdk/devtools/abi-check/02657b4ad/librte_vhost.so.dump NOTICE: At least one call to abi-compliance-checker returned an error. NOTICE: ABI may be incompatible, please check logs for details. NOTICE: Incompatible list: librte_cryptodev.so librte_eal.so librte_efd.so librte_ethdev.so librte_eventdev.so librte_hash.so librte_pdump.so librte_pmd_crypto_scheduler.so librte_pmd_ring.so librte_ring.so Which I think is something of a false positive. The line: NOTICE: At least one call to abi-compliance-checker returned an error. I think is emitted simply because abi-compilance-checker returns non-zero and reports that error if it finds any incompatibilities. I'm not sure we want to flag that as an error per se. It gives the impression something has gone wrong, rather than correctly identifying that there are incompatibilities. Though on the other hand, maybe we do want to set that red flag so people look at what the incompatibilities are. As I say it out loud, perhaps that preferable. Thoughts Neil