Regards, Keith > On Sep 21, 2016, at 4:24 AM, Thomas Monjalon <thomas.monjalon at 6wind.com> > wrote: > > 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 >>> handling of the opens on the tun device are different then linux. I also >>> only added it to the common_linux configuration file. >> >> 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. > > 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 for 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.