From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier Matz Subject: Re: [PATCH v2 6/7] virtio: fix pci accesses for ppc64 in legacy mode Date: Fri, 20 May 2016 14:18:27 +0200 Message-ID: <573F0093.6030501@6wind.com> References: <1463143859-3105-1-git-send-email-olivier.matz@6wind.com> <1463479192-2488-1-git-send-email-olivier.matz@6wind.com> <1463479192-2488-7-git-send-email-olivier.matz@6wind.com> <000301d1b1ae$cebf1470$6c3d3d50$@linux.vnet.ibm.com> <573EFEEF.4040800@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=gbk Content-Transfer-Encoding: 7bit Cc: david.marchand@6wind.com, yuanhan.liu@linux.intel.com, huawei.xie@intel.com To: Chao Zhu , dev@dpdk.org Return-path: Received: from mail.droids-corp.org (zoll.droids-corp.org [94.23.50.67]) by dpdk.org (Postfix) with ESMTP id 06C56ADE3 for ; Fri, 20 May 2016 14:18:37 +0200 (CEST) In-Reply-To: <573EFEEF.4040800@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" On 05/20/2016 02:11 PM, Olivier Matz wrote: > Maybe I could replace in the patch: > #ifdef RTE_ARCH_PPC_64 > By something like : > #if defined(RTE_ARCH_PPC_64) && (RTE_BYTE_ORDER == RTE_LITTLE_ENDIAN) Hmm it's probably useless since we already do a rte_be_to_cpu_*(), so it should do nothing on BE targets.