From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: Coverity policy for upstream (base) drivers. Date: Thu, 12 Nov 2015 23:18:49 +0100 Message-ID: <3288160.AgV0ATSvM7@xps13> References: <20151112140508.79489210@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Stephen Hemminger Return-path: Received: from mail-wm0-f42.google.com (mail-wm0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id D4CA39252 for ; Thu, 12 Nov 2015 23:20:02 +0100 (CET) Received: by wmec201 with SMTP id c201so6402078wme.1 for ; Thu, 12 Nov 2015 14:20:02 -0800 (PST) In-Reply-To: <20151112140508.79489210@xeon-e3> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2015-11-12 14:05, Stephen Hemminger: > Looking at the Coverity scan for DPDK, it looks like all the base > drivers are marked to be ignored. > > Although the changes to base drivers should not be done directly through > DPDK list. I think it is still valuable to have these driver scanned and > notify (badger) the vendors to fix there code. > > Since lots of the bugs could be there, just blindly ignoring warnings > and issues is being naive. I think the Coverity setup is outdated: ignore_driver_1 /lib/librte_pmd_e1000/e1000/.* Yes Remove ignore_driver_2 /lib/librte_pmd_fm10k/base/.* Yes Remove ignore_driver_3 /lib/librte_pmd_i40e/i40e/.* Yes Remove ignore_driver_4 /lib/librte_pmd_ixgbe/ixgbe/.* Yes Remove These directories don't exist anymore.