From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f180.google.com (mail-wi0-f180.google.com [209.85.212.180]) by dpdk.org (Postfix) with ESMTP id 794375A58 for ; Fri, 6 Nov 2015 13:25:58 +0100 (CET) Received: by wikq8 with SMTP id q8so29240966wik.1 for ; Fri, 06 Nov 2015 04:25:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind_com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:mail-followup-to:references :mime-version:content-type:content-disposition :content-transfer-encoding:in-reply-to; bh=BZ4NsirV2abuEITa6vk9dlXd6es+R4GNw9QQBVX9REo=; b=mXr0p+4gr/bxcoqMSwBNRSg8BDG50DT1Y7F8TkazMqZmZjyT7mzjV34xIqLFcqXgvy ngRLiAka8+/YyvAOoeddXH8uj3pwwiq8QWzDIEAj14FBs1ww7MBl+Lo6aZONYZ2ZE+zi TJzYVVz6iNo0VeJ6IShT1LXoBXyc0s/z2I9FgIFoGado7WyI44m1nmRoz+oNqp83NW03 2cuPu3vQVEas0N85+9A/ehOtJRkzbw7TTbV66z7SkaHkZSpdF8YR+UVvqDMSIa7IHHTD DQ+I+QA7qUot5/acN0hydZzes2oNsfEU5/CaEyDLUBI6OtC+wG/vB8+nphm+a3TsZAjn 9Tgg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id :mail-followup-to:references:mime-version:content-type :content-disposition:content-transfer-encoding:in-reply-to; bh=BZ4NsirV2abuEITa6vk9dlXd6es+R4GNw9QQBVX9REo=; b=aeC5ZNnOXDDClhgJwnWYkRyiF6kWabiyEvfCC6xuDF/+VCRh8FcV49VuDLa3jR5klT PfXCLLeJgDRujqcvwAowdNsYYOlBekg/5taYEfJ4gAQey+jhly+IdAR0LNVkoXKf1N0R bvlVu3FcZlCl1ZOqE2gf91hJbcUcB4o3srqRORZbhuvIYN/uthniY94/eaLv2854vgwl 26lNqCcUmTjNiE8GcGf2Jk4hkTxOI68lzmsWnuD4fRroWlxJ/TrdHaiELCOlIh3gK+bM yzHKU/8qMh6dMdlBTenrA1xJDjtztpdH+G5ulq67fhGOE6J8VsCOWe7CuXRe5+30Cdeb b8uw== X-Gm-Message-State: ALoCoQmLjpzcNA0Fw2zYlagb3Ig2/xIHmTRnk8NQmvnwForsve9a9+E/w+ykhYZumfrZiI+2WZuA X-Received: by 10.194.122.230 with SMTP id lv6mr16547115wjb.2.1446812758231; Fri, 06 Nov 2015 04:25:58 -0800 (PST) Received: from 6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id kr10sm12010196wjc.25.2015.11.06.04.25.57 (version=TLSv1 cipher=RC4-SHA bits=128/128); Fri, 06 Nov 2015 04:25:57 -0800 (PST) Date: Fri, 6 Nov 2015 13:25:39 +0100 From: Adrien Mazarguil To: Bruce Richardson Message-ID: <20151106122539.GC4013@6wind.com> Mail-Followup-To: Bruce Richardson , dev@dpdk.org References: <20151106115244.GB13920@bricha3-MOBL3> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20151106115244.GB13920@bricha3-MOBL3> Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v3 2/4] ethdev: move error checking macros to header X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 06 Nov 2015 12:25:58 -0000 Hi Bruce, On Fri, Nov 06, 2015 at 11:52:44AM +0000, Bruce Richardson wrote: > +Adrien on To: line > > Email user/client fail on original. :-( > > ----- Forwarded message from Bruce Richardson ----- > > Date: Fri, 6 Nov 2015 11:49:05 +0000 > From: Bruce Richardson > To: Stephen Hemminger , Thomas Monjalon , dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 2/4] ethdev: move error checking macros to header > User-Agent: Mutt/1.5.23 (2014-03-12) > > On Thu, Nov 05, 2015 at 04:09:18PM +0100, Adrien Mazarguil wrote: > > Bruce is asking for a consensus about -pedantic, whether we want to do the > > extra effort to support it in DPDK. Since I like checking for -pedantic > > errors, it's enabled for mlx4 and mlx5 when compiling these drivers in > > debugging mode. There is currently no established rule in DPDK against this. > > > > I'm arguing that most C headers (C compiler, libc, most libraries, even the > > Linux kernel in uapi to an extent) provide standards compliant includes > > because they cannot predict or force particular compilation flags on > > user applications. > > > > If we consider DPDK as a system wide library, I think we should do it as > > well in all installed header files. If we choose not to, then we must > > document that our code is not standard, -pedantic is unsupported and I'll > > have to drop it from mlx4 and mlx5. > > > > -- > > Adrien Mazarguil > > 6WIND > > Hi Adrien, > > I'm trying to dig into this a bit more now, and try out using a static inline > function, but I'm having trouble getting DPDK to compile with the mlx drivers > turned on in the config. I'm trying to follow the instructions here: > http://dpdk.org/doc/guides/nics/mlx4.html, but it's not clearly called out what > requirements are for compilation vs requirements for running the PMD. > > I'm running Fedora 23, and installed the libibverbs-devel package, but when I > compile I get the following error: > > == Build drivers/net/mlx4 > CC mlx4.o > /home/bruce/ethdev-cleanup/drivers/net/mlx4/mlx4.c: In function ‘txq_cleanup’: > /home/bruce/ethdev-cleanup/drivers/net/mlx4/mlx4.c:886:37: error: storage size of ‘params’ isn’t known > struct ibv_exp_release_intf_params params; > ^ > compilation terminated due to -Wfatal-errors. > > Any suggestions on the fix for this? This is a known issue, libibverbs-devel package from Fedora 23 most likely does not support extended types and functions required by mlx4. You should remove the packages that come with your distribution and install libraries versions from Mellanox OFED as described in the next section: http://dpdk.org/doc/guides/nics/mlx4.html#getting-mellanox-ofed Note: no need to fully install OFED for compilation checks, you can extract an updated libibverbs package from the archive. -- Adrien Mazarguil 6WIND