From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 275049267 for ; Fri, 13 Nov 2015 01:16:15 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 12 Nov 2015 16:16:13 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,284,1444719600"; d="scan'208";a="818695421" Received: from irsmsx106.ger.corp.intel.com ([163.33.3.31]) by orsmga001.jf.intel.com with ESMTP; 12 Nov 2015 16:16:12 -0800 Received: from irsmsx103.ger.corp.intel.com ([169.254.3.13]) by IRSMSX106.ger.corp.intel.com ([169.254.8.228]) with mapi id 14.03.0248.002; Fri, 13 Nov 2015 00:16:12 +0000 From: "Mcnamara, John" To: Thomas Monjalon , Stephen Hemminger Thread-Topic: [dpdk-dev] Coverity policy for upstream (base) drivers. Thread-Index: AQHRHZY9DW18tagHSkOV+3hXiPhHrZ6Y9QiAgAAfu2A= Date: Fri, 13 Nov 2015 00:16:10 +0000 Message-ID: References: <20151112140508.79489210@xeon-e3> <3288160.AgV0ATSvM7@xps13> In-Reply-To: <3288160.AgV0ATSvM7@xps13> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] Coverity policy for upstream (base) drivers. X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 13 Nov 2015 00:16:15 -0000 > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Thursday, November 12, 2015 10:19 PM > To: Stephen Hemminger > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] Coverity policy for upstream (base) drivers. >=20 > 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. >=20 > 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 >=20 > These directories don't exist anymore. Hi Thomas, The directories don't exist anymore but code from those directories is stil= l in the Coverity database from prior to the restructuring. There is a new rule to ignore the new base drivers: ignore_base_code /drivers/net/*/base/* Yes John --=20