From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 40D60326C for ; Mon, 3 Apr 2017 16:23:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1491229393; x=1522765393; h=subject:to:references:cc:from:message-id:date: mime-version:in-reply-to:content-transfer-encoding; bh=6lhh5mAyA5bTqxcipecuqElU8d13VrHZZaQY4l6u8ns=; b=jVEVVA/z4Ov7MwBW3nFGRk0PNVnXWaLUS7IT0L0vRZ28yL6N4w6XH1Jk At9BKyeup1iUiHXosErVMwcmfLaOlQ==; Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 Apr 2017 07:23:12 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,270,1486454400"; d="scan'208";a="84066071" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.122]) ([10.237.220.122]) by fmsmga006.fm.intel.com with ESMTP; 03 Apr 2017 07:23:11 -0700 To: Jerin Jacob , "Legacy, Allain" References: <1491227915-9410-1-git-send-email-jerin.jacob@caviumnetworks.com> <70A7408C6E1BFB41B192A929744D8523968FFB96@ALA-MBC.corp.ad.wrs.com> <20170403142204.jj3dgw4xxfc4pv4i@localhost.localdomain> Cc: "dev@dpdk.org" From: Ferruh Yigit Message-ID: Date: Mon, 3 Apr 2017 15:23:10 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <20170403142204.jj3dgw4xxfc4pv4i@localhost.localdomain> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH] net/avp: fix build with non x86 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: Mon, 03 Apr 2017 14:23:13 -0000 On 4/3/2017 3:22 PM, Jerin Jacob wrote: > On Mon, Apr 03, 2017 at 02:10:41PM +0000, Legacy, Allain wrote: >>> -----Original Message----- >>> From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com] >>> Sent: Monday, April 03, 2017 9:59 AM >>> To: dev@dpdk.org >>> Cc: YIGIT, FERRUH; Legacy, Allain; Jerin Jacob >>> Subject: [dpdk-dev] [PATCH] net/avp: fix build with non x86 >>> >>> sys/io.h is an x86 specific header file and AVP driver is not using any x86 >>> specific IO operation. Hence removing the sys/io.h reference to fix the build >>> issue on non x86 platforms. >>> >>> Fixes: 5567f9689cf8 ("net/avp: support driver registration") >>> >>> Signed-off-by: Jerin Jacob >> >> In addition to this, since we do not support AVP devices on anything but x86_64 perhaps it would be appropriate to avoid compilation altogether on other architectures. That is, by disabling CONFIG_RTE_LIBRTE_AVP_PMD by default and only enabling for x86_64 on icc/gcc/clang for linuxapp? > > If that is the case then Please send a patch to fix it appropriately. +1