dpdk.git
8 years agoixgbe: restore imissed stat counter
Robin Jarry [Tue, 15 Dec 2015 16:05:01 +0000 (17:05 +0100)]
ixgbe: restore imissed stat counter

This counter was left unmodified. Restore it in ixgbe_dev_stats_get.

The ierrors counter still includes imissed for ixgbe. This behaviour is
not consistent amongst all drivers. Another patch may be needed to unify
the meaning of the ierrors counter.

Fixes: 5e50ad1c1b63 ("ixgbe: add specific stats")

Signed-off-by: Robin Jarry <robin.jarry@6wind.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: fix ABI change announce for RETA configuration
Nelio Laranjeiro [Tue, 15 Dec 2015 14:15:12 +0000 (15:15 +0100)]
doc: fix ABI change announce for RETA configuration

Replace "entries" by "queues", it clarifies the case.

Fixes: bd3cea78abd8 ("doc: announce ABI change for RETA configuration")

Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
8 years agodoc: announce ABI change for extending filtering
Rahul Lakkireddy [Tue, 15 Dec 2015 14:10:53 +0000 (19:40 +0530)]
doc: announce ABI change for extending filtering

Current filtering support will be enhanced to accommodate support
for Chelsio T5 hardware filtering support.

Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agoscripts: fix ABI validator when revision is a tag
Panu Matilainen [Tue, 15 Dec 2015 13:55:15 +0000 (15:55 +0200)]
scripts: fix ABI validator when revision is a tag

Commit 9cbae2aa64eb managed to break the only previously supported
case where a tag is used as a revision, due to git show output
differing between tags and other objects. The hash is on the last
line of the output in both cases though so just grab that.

Fixes: 9cbae2aa64eb ("scripts: support any git revisions as ABI validation range")

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
8 years agodoc: improve Linux guide layout
John McNamara [Tue, 15 Dec 2015 13:34:08 +0000 (13:34 +0000)]
doc: improve Linux guide layout

Fixed Linux Getting Started Guide rst layout to improve
rendering in PDF.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: fix -n option in FreeBSD guide
John McNamara [Tue, 15 Dec 2015 13:47:07 +0000 (13:47 +0000)]
doc: fix -n option in FreeBSD guide

Fix EAL usage to indicate that -n is, now, optional.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: improve FreeBSD guide layout
John McNamara [Tue, 15 Dec 2015 11:53:41 +0000 (11:53 +0000)]
doc: improve FreeBSD guide layout

Fixed FreeBSD Getting Started Guide rst layout to improve
rendering in PDF.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: remove unused references from faq
John McNamara [Tue, 15 Dec 2015 11:12:01 +0000 (11:12 +0000)]
doc: remove unused references from faq

The faq refers to Linux*, with an asterisk, without any equivalent
note or footnote. This is a legacy from older versions of the docs.
This update removes it.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: clean up index files
John McNamara [Tue, 15 Dec 2015 10:10:49 +0000 (10:10 +0000)]
doc: clean up index files

Remove **Contents** and |Today| from the rst doc index files since
these are already added automatically to PDF files and are of
little value to the Html files where the Contents is shown in a
sidebar.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: fix rendering of design section
John McNamara [Tue, 15 Dec 2015 09:55:51 +0000 (09:55 +0000)]
doc: fix rendering of design section

Fix minor rst doc issues in the contributing/design.rst doc to correct
rendering of notes and code blocks.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: fix spellings
John McNamara [Tue, 15 Dec 2015 09:38:59 +0000 (09:38 +0000)]
doc: fix spellings

Fix various spellings in rst docs.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: add statistics fixes in release notes
Harry van Haaren [Tue, 15 Dec 2015 12:10:30 +0000 (12:10 +0000)]
doc: add statistics fixes in release notes

This patch updates the release notes to include the changes
made (by me), which were not appropriately documented at the time.
Hence, this patch fixes a number of missing docs,

Fixes: e81a315e5dc5 ("ixgbe: add MAC short packet discard count to Rx errors")
Fixes: 48dd1a82a615 ("ixgbe: remove mac fault counts from Rx errors")
Fixes: 256ff05a9cae ("ixgbe: fix Rx errors statistics for UDP checksum")

Also, the CRC byte removal was not added to release notes, so
Fixes: 156c5a8cf913 ("e1000: remove CRC size from byte counters")
Fixes: c03fcee9abbd ("ixgbe: remove CRC size from byte counters")
Fixes: 0834d1524dee ("i40e: remove CRC size from byte counters")

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoethdev: remove old flow director symbols
Thomas Monjalon [Tue, 15 Dec 2015 10:27:37 +0000 (11:27 +0100)]
ethdev: remove old flow director symbols

The API has been removed but the symbols were still declared in the map.

Fixes: a421b86a4a02 ("ethdev: remove old flow director API")

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agoeal: remove zombie symbols
Thomas Monjalon [Tue, 15 Dec 2015 09:26:20 +0000 (10:26 +0100)]
eal: remove zombie symbols

test_mp_secondary was initially added by mistake.
rte_snprintf has been removed.

Fixes: 9d41beed24b0 ("lib: provide initial versioning")
Fixes: 3185322809c1 ("eal: remove rte_snprintf")

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agodoc: announce ABI change for link speed
Thomas Monjalon [Tue, 15 Dec 2015 07:14:55 +0000 (08:14 +0100)]
doc: announce ABI change for link speed

A rework was prepared by Marc Sune:
http://dpdk.org/ml/archives/dev/2015-October/026037.html
The goal is to retrieve the supported link speed of a device
and to allow 100G devices while having a consistent API.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
Acked-by: Olga Shern <olgas@mellanox.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: Jan Viktorin <viktorin@rehivetech.com>
Acked-by: Matej Vido <matejvido@gmail.com>
8 years agodoc: announce ABI change for tunnel filtering
Jingjing Wu [Tue, 10 Nov 2015 03:49:46 +0000 (11:49 +0800)]
doc: announce ABI change for tunnel filtering

Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agodoc: announce ABI change for flow director
Jingjing Wu [Tue, 10 Nov 2015 03:11:26 +0000 (11:11 +0800)]
doc: announce ABI change for flow director

Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
Acked-by: Andrey Chilikin <andrey.chilikin@intel.com>
8 years agodoc: announce ABI change for RETA configuration
Nelio Laranjeiro [Mon, 9 Nov 2015 16:48:20 +0000 (17:48 +0100)]
doc: announce ABI change for RETA configuration

Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
Acked-by: Olga Shern <olgas@mellanox.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
8 years agodoc: announce ABI change for cmdline buffer size
Nelio Laranjeiro [Mon, 9 Nov 2015 16:48:19 +0000 (17:48 +0100)]
doc: announce ABI change for cmdline buffer size

Current buffer size are not enough for a few testpmd commands.

Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
Acked-by: Olga Shern <olgas@mellanox.com>
8 years agodoc: fix release notes for 2.2
John McNamara [Mon, 14 Dec 2015 18:51:16 +0000 (18:51 +0000)]
doc: fix release notes for 2.2

Fix grammar, spelling and formatting of DPDK 2.2 release notes.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agomlx: fix typos and inaccuracies in docs
Olga Shern [Mon, 14 Dec 2015 18:28:19 +0000 (20:28 +0200)]
mlx: fix typos and inaccuracies in docs

Signed-off-by: Olga Shern <olgas@mellanox.com>
8 years agodoc: add patch submit cheatsheet
Harry van Haaren [Mon, 14 Dec 2015 10:03:48 +0000 (10:03 +0000)]
doc: add patch submit cheatsheet

This patch adds the patch submission cheatsheet to
the contributers guide. Both HTML and PDF docs show
the cheatsheet on its own page.

Right clicking the SVG image in the HTML doc allows
for viewing the image on its own, useful for printing
in high quality.

The exact appearance of of the cheatsheet will depend
on the default monospace font installed.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: add contributors guide
John McNamara [Mon, 14 Dec 2015 20:45:54 +0000 (20:45 +0000)]
doc: add contributors guide

Add a document to explain the DPDK patch submission and review process.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agoscripts: add help for build testing
Thomas Monjalon [Sun, 13 Dec 2015 02:00:48 +0000 (03:00 +0100)]
scripts: add help for build testing

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agoexamples/ip_pipeline: add sample configurations and scripts
Jasvinder Singh [Mon, 14 Dec 2015 14:20:42 +0000 (14:20 +0000)]
examples/ip_pipeline: add sample configurations and scripts

This patch includes the configuration and script files of the some
applications which can be built using DPDK Packet Framework. A
configuration file defines the application structure which include
packet processing stages (knowns as pipelines), their connectivity
and other parameters necessary to start and run the application. A
script file specifies CLI commands required for loading table entries
(rules/routes, etc). The configuration/script files for simple
applications such as l2 forwarding, l3 forwarding have been presented.
In addition, to demonstrate the use and inter-connectivity of various
pipeline modules (which are avilable in packet framework), a complex
packet processing workload i.e. edge router is considered. The
configuration of the pipeline stages used for upstream and downstream
flow processing has been specified separately in two configuration
files. All these configuration and script files don't affect the
compilation.

Signed-off-by: Jasvinder Singh <jasvinder.singh@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
8 years agobonding: fix reordering of IP fragments
Andriy Berestovskyy [Tue, 8 Dec 2015 14:47:03 +0000 (15:47 +0100)]
bonding: fix reordering of IP fragments

Fragmented IPv4 packets have no TCP/UDP headers, so we hashed
random data introducing reordering of the fragments.

Signed-off-by: Andriy Berestovskyy <aber@semihalf.com>
Acked-by: Declan Doherty <declan.doherty@intel.com>
8 years agobonding: fix LACP mempool size
Andriy Berestovskyy [Tue, 8 Dec 2015 13:23:05 +0000 (14:23 +0100)]
bonding: fix LACP mempool size

The following messages might appear after some idle time:
"PMD: Failed to allocate LACP packet from pool"

The fix ensures the mempool size is greater than the sum
of TX descriptors.

Signed-off-by: Andriy Berestovskyy <aber@semihalf.com>
Acked-by: Declan Doherty <declan.doherty@intel.com>
8 years agodoc: add known VF issue when PF is reset
Helin Zhang [Mon, 14 Dec 2015 01:12:34 +0000 (09:12 +0800)]
doc: add known VF issue when PF is reset

Add a known issue to warn that PF reset event/request is not
handled by any VF drivers till 2.2, and which should be fixed
in the future.

Reported-by: David Marchand <david.marchand@6wind.com>
Signed-off-by: Helin Zhang <helin.zhang@intel.com>
8 years agoexamples: skip unsupported ethtool and l3fwd-power on FreeBSD
Yong Liu [Mon, 14 Dec 2015 02:32:52 +0000 (10:32 +0800)]
examples: skip unsupported ethtool and l3fwd-power on FreeBSD

Userspace ethtool and l3fwd-power not support FreeBSD.
So skip them when build samples.

Signed-off-by: Marvin Liu <yong.liu@intel.com>
8 years agoversion: 2.2.0-rc4
Thomas Monjalon [Mon, 14 Dec 2015 00:40:20 +0000 (01:40 +0100)]
version: 2.2.0-rc4

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agoexamples/ip_pipeline: update release notes
Jasvinder Singh [Wed, 4 Nov 2015 17:03:46 +0000 (17:03 +0000)]
examples/ip_pipeline: update release notes

This patch updates the release notes with the features
added to ip_pipeline application.

Signed-off-by: Jasvinder Singh <jasvinder.singh@intel.com>
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
8 years agoexamples/vhost: fix guide
Bernard Iremonger [Thu, 10 Dec 2015 11:23:04 +0000 (11:23 +0000)]
examples/vhost: fix guide

Fix typos in sample console commands.
Remove "user@target:~$" prefixes in command lines.

Add --socket-mem 1024 to vhost-switch command lines.
Without the --socket-mem parameter all hugepages will be allocated
to vhost-switch, leaving no memory for the VM.

Add --disable-hw-vlan-filter parameter to testpmd command lines.
Without this parameter testpmd fails on startup in the VM.

Increase value of --sock-mem parameter to testpmd as original
value was too small.

Fixes: d0dff9ba445e ("doc: sample application user guide"
Fixes: 9bc23cb8209c ("doc: add vhost-user to sample guide")
Fixes: 43866bf71d58 ("doc: fix vhost sample parameter")

Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
Acked-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
8 years agovhost: note the ABI changes
Yuanhan Liu [Mon, 7 Dec 2015 03:25:52 +0000 (11:25 +0800)]
vhost: note the ABI changes

Note the ABI changes and update the ABI version to 2.

Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
8 years agoszedata2: note limited current support in guide
Matej Vido [Tue, 24 Nov 2015 12:07:13 +0000 (13:07 +0100)]
szedata2: note limited current support in guide

Add note to the documentation that only x86_64 versions of the external
libraries are currently provided.

Signed-off-by: Matej Vido <matejvido@gmail.com>
8 years agomlx: add known issues in release notes
Olga Shern [Sat, 12 Dec 2015 22:13:25 +0000 (00:13 +0200)]
mlx: add known issues in release notes

Signed-off-by: Olga Shern <olgas@mellanox.com>
8 years agomlx4: update features and fixes in release notes
Olga Shern [Sat, 12 Dec 2015 22:13:24 +0000 (00:13 +0200)]
mlx4: update features and fixes in release notes

Signed-off-by: Olga Shern <olgas@mellanox.com>
8 years agomlx5: update documentation
Olga Shern [Sat, 12 Dec 2015 19:43:24 +0000 (21:43 +0200)]
mlx5: update documentation

- Update features, limitations, configuration and prerequisites sections.
- Add a note to describe RSS behavior differences with librte_pmd_mlx4 in
  testpmd.

Signed-off-by: Olga Shern <olgas@mellanox.com>
Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
8 years agomlx4: update documentation
Olga Shern [Sat, 12 Dec 2015 16:20:51 +0000 (18:20 +0200)]
mlx4: update documentation

-Split "Features" and "Limitations" sections.
-Update limitations with missing information.
-Update prerequisites with supported MLNX_OFED release, firmware and
 CPU architectures.
-Enhance usage example with openibd script.

Signed-off-by: Olga Shern <olgas@mellanox.com>
Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
8 years agoenic: add guide
John Daley [Sun, 13 Dec 2015 23:11:32 +0000 (15:11 -0800)]
enic: add guide

Signed-off-by: John Daley <johndale@cisco.com>
8 years agofm10k: add guide
Chen Jing D(Mark) [Thu, 10 Dec 2015 07:00:21 +0000 (15:00 +0800)]
fm10k: add guide

This documentation covers introductions and limitations on Intel
FM10000 series products.

Signed-off-by: Chen Jing D(Mark) <jing.d.chen@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoi40e: add X722 support to release notes
Helin Zhang [Thu, 12 Nov 2015 03:18:27 +0000 (11:18 +0800)]
i40e: add X722 support to release notes

Updated release notes about adding X722 support.

Signed-off-by: Helin Zhang <helin.zhang@intel.com>
8 years agoi40e: add base driver update to release notes
Jingjing Wu [Tue, 27 Oct 2015 08:34:36 +0000 (16:34 +0800)]
i40e: add base driver update to release notes

Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoe1000: add interrupt limitations in guide
Yong Liu [Fri, 30 Oct 2015 09:23:08 +0000 (17:23 +0800)]
e1000: add interrupt limitations in guide

Update interrupt related limitations for e1000em nic.
It only support one interrupt source and not support auto-clear.

Signed-off-by: Marvin Liu <yong.liu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoe1000: add base driver update to release notes
Wenzhuo Lu [Mon, 2 Nov 2015 06:13:40 +0000 (14:13 +0800)]
e1000: add base driver update to release notes

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: add NIC performance guide on Linux IA
Qian Xu [Wed, 18 Nov 2015 03:10:16 +0000 (11:10 +0800)]
doc: add NIC performance guide on Linux IA

Add a new guide doc as part of the Linux Getting Started Guide.

The document is a step-by-step guide on how to get high performance
with DPDK on an Intel platform.

It is designed for users who are not familiar with DPDK but would like
to get the best performance with NICs.

Signed-off-by: Qian Xu <qian.q.xu@intel.com>
8 years agodoc: increment minimum Linux kernel requirement
John McNamara [Wed, 25 Nov 2015 14:26:00 +0000 (14:26 +0000)]
doc: increment minimum Linux kernel requirement

Update the documentation to reflect that the minimum Linux kernel
requirement for DPDK 2.2 has increased from 2.6.33 to 2.6.34.

Compatibility with kernel 2.6.33 was dropped, after discussion on
the mailing list, in the following commit:
2e6e9e215703 ("igb_uio: use existing PCI macros")

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: link examples to API
John McNamara [Fri, 11 Dec 2015 13:41:51 +0000 (13:41 +0000)]
doc: link examples to API

Enable Doxygen parsing of DPDK example files so that the example
code is linked to and from the API docs. This allows the reader
to see examples of API usage in context by following a link.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: change theme of guides
John McNamara [Sun, 1 Nov 2015 23:29:13 +0000 (23:29 +0000)]
doc: change theme of guides

Change the Sphinx default theme from "alabaster" to the ReadTheDocs
theme. See for example:
    http://dpdk.readthedocs.org/en/latest/

This looks better for technical documentation and in particular
it has a 80 char wide verbatim block rendering.

Also add logos for the html and latex docs.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: remove dejavu font requirement
John McNamara [Sun, 1 Nov 2015 20:17:58 +0000 (20:17 +0000)]
doc: remove dejavu font requirement

Remove requirement for occasionally hard to find/install
DejaVuSansMono font. It isn't gnerally required anyway.
The default mono font is sufficient.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
8 years agodoc: turn off permalink symbols in html guides
John McNamara [Sun, 1 Nov 2015 16:44:30 +0000 (16:44 +0000)]
doc: turn off permalink symbols in html guides

Turn off the permalink symbol in the Sphinx generated html docs.
This is a distracting mouseover when reading the docs and links
to sections can still be obtained from the doc index.

Signed-off-by: John McNamara <john.mcnamara@intel.com>
8 years agodoc: add readme file
Stephen Hemminger [Wed, 2 Dec 2015 04:20:05 +0000 (20:20 -0800)]
doc: add readme file

This project is missing a proper README which is used in
other projects and some git visualization services.
Only a starting point, please feel free to edit.
To keep the file short and current, I avoided putting any specific
information about features and versions.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ip_pipeline: fix parsing error code
Fan Zhang [Fri, 11 Dec 2015 11:29:11 +0000 (11:29 +0000)]
examples/ip_pipeline: fix parsing error code

Coverity issue: 107136
Fixes: 7f64b9c004aa ("examples/ip_pipeline: rework config file syntax")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ip_pipeline: fix buffer overrun
Fan Zhang [Fri, 11 Dec 2015 11:29:10 +0000 (11:29 +0000)]
examples/ip_pipeline: fix buffer overrun

Coverity issue: 107127
Fixes: 7f64b9c004aa ("examples/ip_pipeline: rework config file syntax")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ip_pipeline: fix logically dead node
Fan Zhang [Fri, 11 Dec 2015 11:29:09 +0000 (11:29 +0000)]
examples/ip_pipeline: fix logically dead node

Coverity issue: 107109
Fixes: 7122d30131ad ("examples/ip_pipeline: rework flow classification pipeline")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ip_pipeline: fix copy-paste error
Fan Zhang [Fri, 11 Dec 2015 11:29:08 +0000 (11:29 +0000)]
examples/ip_pipeline: fix copy-paste error

Coverity issue: 107104
Fixes: 7f64b9c004aa ("examples/ip_pipeline: rework config file syntax")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ip_pipeline: fix buffer copy size
Fan Zhang [Fri, 11 Dec 2015 17:19:16 +0000 (17:19 +0000)]
examples/ip_pipeline: fix buffer copy size

Coverity issue: 107133
Fixes: eb32fe7c5574 ("examples/ip_pipeline: rework initialization parameters")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
8 years agoexamples/ip_pipeline: fix buffer size warning
Fan Zhang [Fri, 11 Dec 2015 11:29:06 +0000 (11:29 +0000)]
examples/ip_pipeline: fix buffer size warning

Coverity issue: 107102
Fixes: 7f64b9c004aa ("examples/ip_pipeline: rework config file syntax")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agovhost: fix logically dead code
Huawei Xie [Thu, 10 Dec 2015 17:57:20 +0000 (01:57 +0800)]
vhost: fix logically dead code

CID 107107 (#1 of 1): Logically dead code
Fixes: af4f2c5feb2e ("vhost: fix code style")

Signed-off-by: Huawei Xie <huawei.xie@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agovhost: fix missed unlock
Huawei Xie [Thu, 10 Dec 2015 17:57:19 +0000 (01:57 +0800)]
vhost: fix missed unlock

CID 107113 (#1 of 1): Missing unlock (LOCK)5. missing_unlock: Returning
without unlocking pfdset->fd_mutex.
Fixes: fbf7e07ca142 ("vhost: add select based event driven processing")

Signed-off-by: Huawei Xie <huawei.xie@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agovhost: fix missed break in switch
Huawei Xie [Thu, 10 Dec 2015 17:57:18 +0000 (01:57 +0800)]
vhost: fix missed break in switch

CID 107114 (#1 of 1): Missing break in switch
Fixes: 8f972312b8f4 ("vhost: support vhost-user")

Signed-off-by: Huawei Xie <huawei.xie@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agovhost: fix out-of-bounds read
Huawei Xie [Thu, 10 Dec 2015 17:57:16 +0000 (01:57 +0800)]
vhost: fix out-of-bounds read

CID 107126 (#1 OF 1): Out-of-bounds read
Fixes: 8f972312b8f4 ("vhost: support vhost-user")

Signed-off-by: Huawei Xie <huawei.xie@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agobnx2x: add maintainers
Rasesh Mody [Fri, 11 Dec 2015 06:36:41 +0000 (22:36 -0800)]
bnx2x: add maintainers

Add maintainers for BNX2X PMD.

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: add version
Rasesh Mody [Fri, 11 Dec 2015 06:36:42 +0000 (22:36 -0800)]
bnx2x: add version

Add BNX2X PMD version, print it as part of adapter info.
Adjusted print adapter info output formatting.

This patch versions BNX2X PMD at 1.0.0.

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: fix 32-bit build
Rasesh Mody [Fri, 11 Dec 2015 06:36:40 +0000 (22:36 -0800)]
bnx2x: fix 32-bit build

Compile tested.

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: fix build on FreeBSD
Rasesh Mody [Fri, 11 Dec 2015 06:36:39 +0000 (22:36 -0800)]
bnx2x: fix build on FreeBSD

Compile tested.

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: add license file
Rasesh Mody [Fri, 11 Dec 2015 06:36:38 +0000 (22:36 -0800)]
bnx2x: add license file

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: add guide
Rasesh Mody [Fri, 11 Dec 2015 06:36:37 +0000 (22:36 -0800)]
bnx2x: add guide

Signed-off-by: Harish Patil <harish.patil@qlogic.com>
Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: add periodic debug option
Harish Patil [Fri, 11 Dec 2015 06:36:36 +0000 (22:36 -0800)]
bnx2x: add periodic debug option

The periodic debug option is used to collect periodic
events like statistics, register access etc and won't
interfere with user-level messages.

Signed-off-by: Harish Patil <harish.patil@qlogic.com>
Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: fix build with clang
Harish Patil [Fri, 11 Dec 2015 06:36:35 +0000 (22:36 -0800)]
bnx2x: fix build with clang

Fix for the following clang build error:
drivers/net/bnx2x/elink.c:10384:41: error: shifting a
      negative signed value is undefined [-Werror,-Wshift-negative-value]
                vars->eee_status &= ~SHMEM_EEE_1G_ADV <<
                                    ~~~~~~~~~~~~~~~~~ ^

Signed-off-by: Harish Patil <harish.patil@qlogic.com>
8 years agobnx2x: update VF to support newer PF drivers
Harish Patil [Fri, 11 Dec 2015 06:36:34 +0000 (22:36 -0800)]
bnx2x: update VF to support newer PF drivers

SR-IOV is supported using bnx2x poll mode driver running as VF driver and
native linux driver running as PF (in host/hypervisor). There is no issue
while running with the PF driver which is at the base version as that of
PMD. However, there is a compatibility issue between newer out-of-box PF
drivers with older VF driver. So the newer VFs would also need to send
BNX2X_VF_TLV_PHYS_PORT_ID (among other TLVs) to differentiate between
newer and older VFs.

Signed-off-by: Harish Patil <harish.patil@qlogic.com>
8 years agobnx2x: skip Rx producer refresh when no packet
Rasesh Mody [Fri, 11 Dec 2015 18:31:54 +0000 (10:31 -0800)]
bnx2x: skip Rx producer refresh when no packet

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: fix Tx error check
Rasesh Mody [Fri, 11 Dec 2015 18:31:53 +0000 (10:31 -0800)]
bnx2x: fix Tx error check

Correct the error check in PMD transmit routine.

Signed-off-by: Rasesh Mody <rasesh.mody@qlogic.com>
8 years agobnx2x: fix restart
Chas Williams [Wed, 9 Dec 2015 22:11:04 +0000 (17:11 -0500)]
bnx2x: fix restart

If you stop and start the driver, the rx queue will have the previous
index values when programming the adapter.  Therefore, we should always
reset the queue indices when the rx ring is setup.  Note:  We need to
clear (write) the status block's completion queue index since it is
possibly in a read cache.

Tidy some init code to make it clearer what the defaults are.

Signed-off-by: Chas Williams <3chas3@gmail.com>
8 years agocxgbe: fix Rx queue setup error management
Rahul Lakkireddy [Fri, 11 Dec 2015 08:25:30 +0000 (13:55 +0530)]
cxgbe: fix Rx queue setup error management

When refilling freelists for the first time and if it fails, the rxq
is freed and returns ENOMEM. There is a check while freeing hardware rxq
to pass freelist context id if the freelist exists or 0xffff otherwise.
The error path is only reached if freelist exists. So, fix is to remove
the useless check for freelist existence.

Coverity issue: 107108
Fixes: 92c8a63223e5 ("cxgbe: add device configuration and Rx support")

Reported-by: John McNamara <john.mcnamara@intel.com>
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoxenvirt: fix build for 32-bit platform
Huawei Xie [Thu, 10 Dec 2015 15:15:58 +0000 (23:15 +0800)]
xenvirt: fix build for 32-bit platform

Reported-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>
Signed-off-by: Huawei Xie <huawei.xie@intel.com>
Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>
8 years agoeal/linux: fix Rx interrupt read error management
Cunming Liang [Fri, 11 Dec 2015 01:48:57 +0000 (09:48 +0800)]
eal/linux: fix Rx interrupt read error management

In eal_intr_proc_rxtx_intr, negative value may be used as argument to
a function expecting a positive value. If 'read' returns EAGAIN as
example, the bytes_read updates to a negative value which continue
be passed as argument for the next 'read'.

Coverity issue: 107115

Function read(fd, &buf, bytes_read) returns a negative number.
Assigning: signed variable bytes_read = read.
CID 107115 (#1 of 1): Argument cannot be negative
(NEGATIVE_RETURNS) bytes_read is passed to a parameter
that cannot be negative.
    bytes_read = read(fd, &buf, bytes_read);

Fixes: c9f3ec1a0f3f ("eal/linux: add Rx interrupt control function")

Signed-off-by: Cunming Liang <cunming.liang@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: David Marchand <david.marchand@6wind.com>
8 years agoexamples/netmap_compat: fix build on Suse 11
Michael Qiu [Fri, 11 Dec 2015 06:36:33 +0000 (14:36 +0800)]
examples/netmap_compat: fix build on Suse 11

 examples/netmap_compat/bridge/../lib/compat_netmap.c:880:41:
 error: ‘POLLRDNORM’ undeclared (first use in this function)

The root casue is POLLRDNORM is more-or-less nonstandard, and it
depends on macro "__USE_XOPEN".

In suse11 sp3, POLLRDNORM will not be defined because "__USE_XOPEN"
issue.

This patch add check if it is not defined, define it.

Fixes: 06371afe394d (examples/netmap_compat: import netmap compatibility example)

Signed-off-by: Michael Qiu <michael.qiu@intel.com>
8 years agoexamples/performance-thread: fix return sign
Ian Betts [Sat, 12 Dec 2015 08:12:08 +0000 (08:12 +0000)]
examples/performance-thread: fix return sign

The patch fixes an inverted return value in the
cond_destroy and cond_init APIs of the pthread shim
example.

These APIs are now demonstrated in the sample app by
having the mutexes and condition variables
explicitly destroyed before the appplication terminates.

Fixes: 433ba6228f9a77a9b5f4 ("add pthread_shim app")

Signed-off-by: Ian Betts <ian.betts@intel.com>
8 years agoscripts: fix relpath.sh output when $prefix is set in environment
Panu Matilainen [Fri, 11 Dec 2015 12:44:23 +0000 (14:44 +0200)]
scripts: fix relpath.sh output when $prefix is set in environment

When relpath.sh is called from install target with prefix set, eg
"make install DESTDIR=/tmp/dpdk-root prefix=/usr", the prefix
from the environment leaks to relpath.sh internal helper variable and
causes incorrect symlinks to be generated in sdk $(targetdir):

   include -> /usr../../../include/dpdk
   lib -> /usr../../../lib

Initialize the local variable to empty to avoid side-effects from
environment.

Fixes: 6b62a72a70d0 ("mk: install a standard cutomizable tree")

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>
8 years agombuf_offload: mark experimental state
Thomas Monjalon [Fri, 11 Dec 2015 23:26:24 +0000 (00:26 +0100)]
mbuf_offload: mark experimental state

Cryptodev was marked experimental and mbuf_offload depends on it.
The mbuf_offload library is one of the crypto area which requires
some discussions before having a stable API.

The experimental mark is also added to rte_cryptodev_configure()
to be sure one cannot miss it.

Fixes: 66874e55f550 ("cryptodev: mark experimental state")

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
8 years agoexamples/performance-thread: mark as experimental
Ian Betts [Thu, 10 Dec 2015 12:24:58 +0000 (12:24 +0000)]
examples/performance-thread: mark as experimental

This commit removes the performance thread example from
examples/Makefile, and marks the example as "experimental"
in the release note, and it its API headers files.

Signed-off-by: Ian Betts <ian.betts@intel.com>
8 years agoexamples/performance-thread: add pthread_shim app
Ian Betts [Tue, 8 Dec 2015 06:05:17 +0000 (06:05 +0000)]
examples/performance-thread: add pthread_shim app

This commit adds an example that illustrates how to implement
a pthread shim with the lthread subsystem included in the
performance thread example application.

Signed-off-by: Ian Betts <ian.betts@intel.com>
Acked-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
8 years agoexamples/performance-thread: add l3fwd-thread app
Ian Betts [Tue, 8 Dec 2015 06:05:16 +0000 (06:05 +0000)]
examples/performance-thread: add l3fwd-thread app

This commit adds an L3 forwarding application to the performace-thread
example.

Signed-off-by: Ian Betts <ian.betts@intel.com>
Acked-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
8 years agoexamples/performance-thread: add lthread subsystem
Ian Betts [Tue, 8 Dec 2015 06:05:15 +0000 (06:05 +0000)]
examples/performance-thread: add lthread subsystem

This commit adds the lightweight thread subsystem used by the
performance-thread sample applications.

Signed-off-by: Ian Betts <ian.betts@intel.com>
Acked-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
8 years agodoc: add guide for performance-thread example
Ian Betts [Tue, 8 Dec 2015 06:05:14 +0000 (06:05 +0000)]
doc: add guide for performance-thread example

This commit adds the sample application user guide for the
performance thread sample application.

Signed-off-by: Ian Betts <ian.betts@intel.com>
Acked-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
8 years agoexamples/vhost: reduce number of hugepages needed
Bernard Iremonger [Thu, 10 Dec 2015 15:49:17 +0000 (15:49 +0000)]
examples/vhost: reduce number of hugepages needed

Add #ifndef MAX_QUEUES to change MAX_QUEUES at compile time if needed.
Change MAX_QUEUES from 512 to 128 to reduce the number of hugepages
required by the vhost-switch program.

To change MAX_QUEUES add '-D MAX_QUEUES=512' to the EXTRA_CFLAGS variable,
before building the application.

Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
8 years agoexample/ip_pipeline: fix config file parsing defect
Fan Zhang [Thu, 10 Dec 2015 12:19:19 +0000 (12:19 +0000)]
example/ip_pipeline: fix config file parsing defect

Coverity issue: 120143

Fixes: 7f64b9c004aa ("examples/ip_pipeline: rework config file syntax")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
8 years agoexamples/l2fwd-keepalive: fix integer overflow
Remy Horton [Thu, 10 Dec 2015 09:50:07 +0000 (09:50 +0000)]
examples/l2fwd-keepalive: fix integer overflow

Fix Coverity warning with potential 32-bit integer multiplication overflow
where final result is expected to be 64-bit.

>>> CID 120144 (#1 of 1): Unintentional integer overflow (OVERFLOW_BEFORE_WIDEN)

Fixes: e64833f2273a ("examples/l2fwd-keepalive: add sample application")

Signed-off-by: Remy Horton <remy.horton@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ethtool: fix uninitialised variable
Remy Horton [Thu, 10 Dec 2015 09:50:06 +0000 (09:50 +0000)]
examples/ethtool: fix uninitialised variable

Fix Coverity warning with uninitialised field in structure being used.
Zero out all the other unused ones.

>>> CID 120413 (#1 of 1): Uninitialized scalar variable (UNINIT)

Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")

Signed-off-by: Remy Horton <remy.horton@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoapp/testpmd: swap MAC in IEEE1588 forwarding
Pablo de Lara [Thu, 10 Dec 2015 09:28:27 +0000 (09:28 +0000)]
app/testpmd: swap MAC in IEEE1588 forwarding

IEEE1588 forwarding mode uses same port to receive and transmit packets.
In this mode, the MAC address in the packets are not touched,
and therefore, packets could be sent with the same destination
MAC address as the MAC address of the port that is transmitting,
which causes the packet to be dropped in some NICs.

In order to avoid this scenario, MAC addresses are swapped
before packet is sent out.

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoaf_packet: set input port in mbuf
Pavel Krauz [Thu, 10 Dec 2015 10:25:05 +0000 (11:25 +0100)]
af_packet: set input port in mbuf

Fixes port number in mbuf structure when using with eth_af_packet

Signed-off-by: Pavel Krauz <pavel.krauz@anritsu.com>
8 years agoi40e: fix filtering with vlan input
Helin Zhang [Thu, 10 Dec 2015 10:44:22 +0000 (18:44 +0800)]
i40e: fix filtering with vlan input

It corrects the input set values to be set for vlan or
double vlan.

Fixes: 98f055707685 ("i40e: configure input fields for RSS or flow director")

Signed-off-by: Helin Zhang <helin.zhang@intel.com>
8 years agovfio: support iommu group zero
Stephen Hemminger [Wed, 9 Dec 2015 17:55:56 +0000 (09:55 -0800)]
vfio: support iommu group zero

The current implementation of VFIO will not with the new no-IOMMU mode
in 4.4 kernel. The original code assumed that IOMMU group zero would
never be used. Group numbers are assigned starting at zero, and up
until now the group numbers came from the hardware which is likely
to use group 0 for system devices that are not used with DPDK.

The fix is to allow 0 as a valid group and rearrange code
to split the return value from the group value.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
8 years agoexamples/vhost: fix strict aliasing
Pablo de Lara [Wed, 9 Dec 2015 14:26:49 +0000 (14:26 +0000)]
examples/vhost: fix strict aliasing

Fixes following error on gcc 4.4.7:

examples/vhost/main.c: In function ‘new_device’:
rte_ring.h:740: error:
    dereferencing pointer ‘mbuf.486’ does break strict-aliasing rules
examples/vhost/main.c:1503: note: initialized from here
...
rte_ring.h:740: error:
    dereferencing pointer ‘({anonymous})’ does break strict-aliasing rules
examples/vhost/main.c:1804: note: initialized from here

Fixes: d19533e8 ("examples/vhost: copy old vhost example")

Reported-by: Qian Xu <qian.q.xu@intel.com>
Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Acked-by: Huawei Xie <huawei.xie@intel.com>
8 years agoexamples/ip_pipeline: fix strict-aliasing
Michael Qiu [Wed, 9 Dec 2015 08:44:26 +0000 (16:44 +0800)]
examples/ip_pipeline: fix strict-aliasing

Compile ip_pipeline in CentOS 6.5 with kernel 2.6.32-431
GCC 4.4.7, will lead below error:

    pipeline_routing_be.c: In function ‘pipeline_routing_msg_req_arp_add_handler’:
    pipeline_routing_be.c:1817: error: dereferencing pointer ‘({anonymous})’
does break strict-aliasing rules

This because the code break strict-aliasing rule.
The patch solve this issue.

Fixes: 0ae7275810f1 (examples/ip_pipeline: add more functions to routing pipeline)

Signed-off-by: Michael Qiu <michael.qiu@intel.com>
Acked-by: Jasvinder Singh <jasvinder.singh@intel.com>
8 years agoexamples/ip_pipeline: fix build in old environments
Jasvinder Singh [Wed, 9 Dec 2015 11:12:53 +0000 (11:12 +0000)]
examples/ip_pipeline: fix build in old environments

This patch fixes build errors on linux kernels such as
SuSE 11-SP2/3(64 bits), etc.

Error Log:
error: implicit declaration of function 'WIFEXITED'
error: implicit declaration of function 'WEXITSTATUS'

Fixes: ed0b2d020159 ("examples/ip_pipeline: add more ports")

Signed-off-by: Jasvinder Singh <jasvinder.singh@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
8 years agoexamples/ip_pipeline: fix coverity warnings
Piotr Azarewicz [Wed, 9 Dec 2015 09:33:50 +0000 (10:33 +0100)]
examples/ip_pipeline: fix coverity warnings

The source and destination both are the arrays of cmdline_parse_ctx_t.
So the goal is to copy elements size of cmdline_parse_ctx_t not
cmdline_parse_ctx_t*.

CID 120412:  Code maintainability issues  (SIZEOF_MISMATCH)
Passing argument "&app->cmds[app->n_cmds]" of type "cmdline_parse_ctx_t *"
and argument "n_cmds * 8UL /* sizeof (cmdline_parse_ctx_t *) */"
to function "memcpy" is suspicious.
In this case, "sizeof (cmdline_parse_ctx_t *)" is equal to
"sizeof (cmdline_parse_ctx_t)", but this is not a portable assumption.

Coverity issue: 120412
Fixes: b4aee0fb9c6d ("examples/ip_pipeline: reconfigure thread binding dynamically")
Fixes: ea0908c4ab89 ("examples/ip_pipeline: add master pipeline")
Fixes: eb32fe7c5574 ("examples/ip_pipeline: rework initialization parameters")

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: Piotr Azarewicz <piotrx.t.azarewicz@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
8 years agoexamples/ethtool: remove dead code
Stephen Hemminger [Tue, 8 Dec 2015 16:51:09 +0000 (08:51 -0800)]
examples/ethtool: remove dead code

Remove dead code, and print better return for other errors.

>>>     CID 120411:  Control flow issues  (DEADCODE)
>>>     Execution cannot reach this statement: "return;".

Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Remy Horton <remy.horton@intel.com>
8 years agovhost: do not stall if guest is slow
Stephen Hemminger [Tue, 8 Dec 2015 00:50:21 +0000 (16:50 -0800)]
vhost: do not stall if guest is slow

When guest is booting (or any othertime guest is busy) it is possible
for the small receive ring (256) to get full. If this happens the
vhost library should just return normally. It's current behavior
of logging just creates massive log spew/overflow which could even
act as a DoS attack against host.

Reported-by: Nathan Law <nlaw@brocade.com>
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
8 years agovirtio: fix wrong queue index
Yuanhan Liu [Tue, 8 Dec 2015 03:05:12 +0000 (11:05 +0800)]
virtio: fix wrong queue index

We should provide VIRTIO_PCI_QUEUE_SEL with vq->vq_queue_idx,
but not vq->queue_id.

vq->queue_id is the queue id from rte_eth_rx/tx_queue_setup(),
which always starts from 0 no matter which queue it is. However,
for virtio, even number is for RX queue, and odd number is for
TX queue.

Fixes: 5382b188fb8a ("virtio: add queue release")

Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>