From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Wiles, Keith" <keith.wiles@intel.com>
Subject: Re: [PATCH v2] drivers/net:new PMD using tun/tap host
 interface
Date: Wed, 21 Sep 2016 23:55:58 +0000
Message-ID: <008395DB-AC42-48DD-AE46-27017BE009FA@intel.com>
References: <1473948649-14169-1-git-send-email-keith.wiles@intel.com>
 <17C91FD1-D21A-49E3-B093-71AA5C1D79C6@intel.com>
 <20160921021351.GY23158@yliu-dev.sh.intel.com> <4238923.20rfmTZejf@xps13>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Cc: "<dev@dpdk.org>" <dev@dpdk.org>, Yuanhan Liu
 <yuanhan.liu@linux.intel.com>, "pmatilai@redhat.com" <pmatilai@redhat.com>
To: Thomas Monjalon <thomas.monjalon@6wind.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mga06.intel.com (mga06.intel.com [134.134.136.31])
 by dpdk.org (Postfix) with ESMTP id BFA5C567A
 for <dev@dpdk.org>; Thu, 22 Sep 2016 01:56:00 +0200 (CEST)
In-Reply-To: <4238923.20rfmTZejf@xps13>
Content-Language: en-US
Content-ID: <82173011D5FFC4419FEC8B087F0C7640@intel.com>
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>


Regards,
Keith

> On Sep 21, 2016, at 4:24 AM, Thomas Monjalon <thomas.monjalon@6wind.com> =
wrote:
>=20
> 2016-09-21 10:13, Yuanhan Liu:
>> On Wed, Sep 21, 2016 at 01:32:15AM +0000, Wiles, Keith wrote:
>>> I only assumed this to work for Linux and not FreeBSD/NetBSD as the han=
dling of the opens on the tun device are different then linux. I also only =
added it to the common_linux configuration file.
>>=20
>> I didn't notice the common_linux file, but I have guessed so (that you
>> meant to enable linux only): the "ifdef __linux" stuff just confuses me
>> a bit.
>=20
> By the way, we have a macro for Linux environment: RTE_EXEC_ENV_LINUXAPP
> Please prefer it over __linux

I removed the __linux__ ifdef in v3, BTW I think we need to grep the code f=
or the __linux__ as I started with one of the other drivers. I will try to =
remember to grep the drivers when I am back home.