From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 00/19] cleanup duplicate includes Date: Sun, 16 Jul 2017 17:34:24 +0200 Message-ID: <2260742.dob8HrkYnO@xps> References: <20170711185546.26138-1-stephen@networkplumber.org> 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 out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 75DF925E5 for ; Sun, 16 Jul 2017 17:34:32 +0200 (CEST) In-Reply-To: <20170711185546.26138-1-stephen@networkplumber.org> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 11/07/2017 20:55, Stephen Hemminger: > Using a simple script (in first patch), discovered that there was > lots of #include file sloppiness in DPDK. This patch series > fixes everything but base drivers and examples. > > Stephen Hemminger (19): > devtools: add simple script to find duplicate includes > eal: remove duplicate includes > cmdline: remove duplicate include of errno.h > distributor: remove duplicate include of rte_compat.h > lpm: remove duplicate include of errno.h > cuckoo_hash: remove duplicate include of rte_log.h > timer: remove duplicate include of rte_per_lcore.h > bnxt: remove duplicate include of unistd.h > ixgbe: remove duplicate include of rte_atomic.h > i40e: remove duplicate include of rte_atomic.h > virtio: remove duplicated includes > cxgbe: remove duplicated includes > vmxnet3: remove duplicated include of rte_atomic.h > dpaa2: remove duplicated includes > fsmlc: remove duplicated includes > app/proc-info: remove duplicate include > app/test-pmd: remove duplicate includes > test/test-pipeline: remove duplicate include of rte_per_lcore.h > test/test: remove duplicate includes Squashed a bit and applied with an additional patch for examples, some more duplicates in .h files removed, and without the perl script. Note: my shell script is pending for approval :) Thanks