From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 2/2] eal: remove useless header inclusion Date: Mon, 27 Jul 2015 00:18:40 +0200 Message-ID: <8723827.jzxt2xTp0g@xps13> References: <1437635732-29113-1-git-send-email-david.marchand@6wind.com> <1437635732-29113-3-git-send-email-david.marchand@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: David Marchand Return-path: Received: from mail-wi0-f173.google.com (mail-wi0-f173.google.com [209.85.212.173]) by dpdk.org (Postfix) with ESMTP id 64EA1C494 for ; Mon, 27 Jul 2015 00:19:56 +0200 (CEST) Received: by wicmv11 with SMTP id mv11so117707105wic.0 for ; Sun, 26 Jul 2015 15:19:56 -0700 (PDT) In-Reply-To: <1437635732-29113-3-git-send-email-david.marchand@6wind.com> 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-07-23 09:15, David Marchand: > --- a/app/test-pmd/cmdline.c > +++ b/app/test-pmd/cmdline.c > @@ -84,7 +84,6 @@ > #include > #include > #include > -#include This one is "needed" to check bypass support. The bypass support should be managed inside the ixgbe driver, and the specific code in testpmd must be removed. Waiting this cleanup, the inclusion will be moved in the #ifdef BYPASS. Note: this ugly bypass code was already signaled few times. If it's not reworked soon (without ifdef), it could be dropped.