vpp.git
5 years agoNAT: convert remaining clib_warning to nat_log_* 57/15157/3
Juraj Sloboda [Fri, 5 Oct 2018 11:06:36 +0000 (13:06 +0200)]
NAT: convert remaining clib_warning to nat_log_*

Change-Id: Ie999ab852cc3775ec90820624d97be746d2590a4
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agonode_cli: Give the user a hint as to the problem. 78/14978/4
Paul Vinciguerra [Tue, 25 Sep 2018 17:02:07 +0000 (10:02 -0700)]
node_cli: Give the user a hint as to the problem.

tested with:
DBGvpp# show node foo
show node: unknown node name: 'foo'

DBGvpp# show node error-drop
node error-drop, type internal, state active, index 543
node function variants:
    ...

DBGvpp# show node error-drop bar
show node: unknown input 'bar'

Change-Id: I896cee9e60028a189dce83666fa4d32a14983a7b
Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
5 years agosession: fix local session tagging and cleanup 53/15153/3
Florin Coras [Fri, 5 Oct 2018 02:03:05 +0000 (19:03 -0700)]
session: fix local session tagging and cleanup

Change-Id: I0d42a0c71fea7dd669fb1fe5ded7e6e944245c7d
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agovhost_user: Add test for interface states and events 41/15141/2
Juraj Sloboda [Thu, 4 Oct 2018 13:15:16 +0000 (15:15 +0200)]
vhost_user: Add test for interface states and events

Change-Id: I2c330945bb0b07f649f574a055bfbea455e5d0b3
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agoOn interface deletion - make sure any vlan qinq table is cleared 51/15151/2
John Lo [Thu, 4 Oct 2018 20:22:21 +0000 (16:22 -0400)]
On interface deletion - make sure any vlan qinq table is cleared

Change-Id: Ie9bb1cfce2369b448ca55a21b2f8b8bbbb81c59c
Signed-off-by: John Lo <loj@cisco.com>
5 years agoImprove ip_mroute_add_del documentation 50/15150/2
Ian Wells [Thu, 4 Oct 2018 19:31:11 +0000 (12:31 -0700)]
Improve ip_mroute_add_del documentation

Change-Id: Ieb882cc67c37bf591b6c737234263c7c35416e6d
Signed-Off-By: Ian Wells <iawells@cisco.com>
5 years agovxlan:ip4 decap:remove access to tunnel object 98/15098/5
Eyal Bari [Tue, 2 Oct 2018 09:23:06 +0000 (12:23 +0300)]
vxlan:ip4 decap:remove access to tunnel object

store local/remote addresses + vrf + vni in hash key
store complete decap info in hash value (sw_if_index + next_index +
error)

this removes the need to access the tunnel object when matching both
unicast and mcast.

however for mcast handling it requires 3 hash lookups:
* one failed unicast lookup (by src+dst addrs)
* lookup by mcast(dst) addr .
* unicast lookup (tunnel local ip as dst + pkt's src addr)

where previously it needed 2:
* lookup by src to find unicast tunnel + compare dst to local addr
(failing for mcast)
* lookup by mcast to find the mcast tunnel

Change-Id: I7a3485d130a54194b8f7e2df0431258db36eceeb
Signed-off-by: Eyal Bari <ebari@cisco.com>
5 years agovcl: fix test throughput computation 49/15149/2
Florin Coras [Thu, 4 Oct 2018 17:07:22 +0000 (10:07 -0700)]
vcl: fix test throughput computation

Change-Id: I78260cd1a412e93a5d6686888b6ea17f52245a4a
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agoSupport reassembly for fragments coming to ip4-local node 70/15070/5
Juraj Sloboda [Tue, 2 Oct 2018 09:13:53 +0000 (11:13 +0200)]
Support reassembly for fragments coming to ip4-local node

Change-Id: I3aa4708c1c3cdda344f282d56b617677080eaaa1
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agoacl-plugin: tuplemerge: avoid batch-resize of the applied entries vector - VPP-1352 42/15142/2
Andrew Yourtchenko [Thu, 4 Oct 2018 13:11:01 +0000 (15:11 +0200)]
acl-plugin: tuplemerge: avoid batch-resize of the applied entries vector - VPP-1352

If the number of rules within a given partition exceeds the limit,
the split_partition() might get called, in which we calculate
the relaxed mask, create a new partition with that mask and
attempt to reallocate some entries from the overcrowded partition.

The non-TM code was pre-expanding the vector with rules by
the number of rules in the new ACL being applied - which
caused the split_partition() to iterate over the rules
filled with zeroes. Most of the time it is benign, but
if a newly created relaxed partition is such that these
entries can be "relocated", then the code attempts to
do so, which does not end well.

Change-Id: I2dbf3ccd29ff97277b21cdb11c4424ff0915c3b7
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
5 years agoudp_echo: fix cut-through server mode 28/15128/9
Florin Coras [Wed, 3 Oct 2018 21:29:10 +0000 (14:29 -0700)]
udp_echo: fix cut-through server mode

Change-Id: I20dd2071c936eb2c1870d45860553007c8e20587
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agoMFIB-API: when programming an (S,G) fix the grp-len 46/15146/2
Neale Ranns [Thu, 4 Oct 2018 15:36:56 +0000 (08:36 -0700)]
MFIB-API: when programming an (S,G) fix the grp-len

Change-Id: I97052d29080501de53fe95499c860faf0765e81d
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
5 years agoclib_count_equal_*: don't read of the end of a small array and init data only if... 25/15125/2
Neale Ranns [Wed, 3 Oct 2018 18:13:27 +0000 (14:13 -0400)]
clib_count_equal_*: don't read of the end of a small array and init data only if used (VPP-1429)

Change-Id: I8afa57ecca590698d3430746968aa0a5b0070469
Signed-off-by: Neale Ranns <nranns@cisco.com>
5 years agoVPP-1430: remove unnecessary padding 34/15134/3
Marco Varlese [Thu, 4 Oct 2018 09:52:44 +0000 (11:52 +0200)]
VPP-1430: remove unnecessary padding

A copy/paste mistake caused to add padding to the buffer used to send
data.

Change-Id: I04d8dfec4ae195927be1675c5ce8b30230d5b376
Signed-off-by: Marco Varlese <marco.varlese@suse.com>
5 years agoFix handling of mpls_tunnel_add_del API 27/15127/2
John Lo [Wed, 3 Oct 2018 20:32:44 +0000 (16:32 -0400)]
Fix handling of mpls_tunnel_add_del API

Handles next_hop_weight, next_hop_preference and next_hop_table_id
properly in vl_api_mpls_tunnel_add_del_t_handler(). Also updated
vpp_api_test and custom_dump handling of related APIs.

Change-Id: I89e211bfa3b9d8ead396b24b3c46b7c7f6511f44
Signed-off-by: John Lo <loj@cisco.com>
5 years agovmxnet3: coverity woe 30/15130/3
Steven [Wed, 3 Oct 2018 22:25:13 +0000 (15:25 -0700)]
vmxnet3: coverity woe

Coverity complains that hb might be null when !prev_b0 is true. It may be right, but not likely.
To make the code more explicit when it parses a complete packet, set got_packet = 1 and test
got_packet instead of prev_b0 which is somewhat obscure.

Use vlib_buffer_free_one rather than vlib_buffer_free_no_next in output.c which takes care of
chain buffers.

Change-Id: I6e0587605fdc622c3f55fedd74c474666631ed91
Signed-off-by: Steven <sluong@cisco.com>
5 years agoLISP ONLY: maintain the make-plugin.sh script 29/15129/1
Dave Barach [Wed, 3 Oct 2018 21:41:29 +0000 (17:41 -0400)]
LISP ONLY: maintain the make-plugin.sh script

Make sure it's invoked in .../src/plugins

Change-Id: I85a05dbd863f3254c017962f635d505482548979
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agoLISP ONLY: maintain the emacs lisp plugin generator 26/15126/1
Dave Barach [Wed, 3 Oct 2018 20:10:04 +0000 (16:10 -0400)]
LISP ONLY: maintain the emacs lisp plugin generator

Add a generator for CMakeLists.txt, remove autotools input generator.

Add missing extern declarations for node registrations in <plugin-name>.h.

Remove extern from the <plugin-name>_main_t declaration in
<plugin-name>.c, so it won't come up undefined at runtime.

Reposition forward node declaration relative to CLIB_MARCH_VARIANT
conditional, to avoid multiple definition / "<plugin-name>_node
changed size from X to Y" warnings.

Change-Id: I8cc00bca5c40fa3ea00549411a3a9027f97a4826
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agotest framework: add shell-style wildcard matching to filenames 19/15119/4
Andrew Yourtchenko [Wed, 3 Oct 2018 09:38:31 +0000 (11:38 +0200)]
test framework: add shell-style wildcard matching to filenames

Currently, one can either use "*" to denote all filenames,
or to give the specific file name to run the tests in.
This commit adds the possibility to run all tests
matching the shell wildcard, e.g. TEST="test_acl_plugin*"
will execute all ACL plugin testcases.

Change-Id: I9048a601958947c7b757c3dfd57b19cdd8a1e3c0
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
5 years agoipsec: add missing ipv6 ah code & ipv6 tests 69/15069/6
Klement Sekera [Wed, 26 Sep 2018 09:19:00 +0000 (11:19 +0200)]
ipsec: add missing ipv6 ah code & ipv6 tests

Change-Id: I89e90193ded1beb6cb0950c15737f9467efac1c3
Signed-off-by: Klement Sekera <ksekera@cisco.com>
5 years agosession: fix first segment manager cleanup 17/15117/2
Florin Coras [Wed, 3 Oct 2018 08:00:57 +0000 (01:00 -0700)]
session: fix first segment manager cleanup

Change-Id: I750db7617ff9fd36647743094e0fa855e7dada86
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agomemif: include interface name in logs 22/15122/2
Juraj Sloboda [Wed, 3 Oct 2018 14:00:21 +0000 (16:00 +0200)]
memif: include interface name in logs

Change-Id: I67f58aa93c721dbd518cc2ddcea330006693662f
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agoVPP-1440: clean up coverity warnings 24/15124/3
Dave Barach [Wed, 3 Oct 2018 15:45:06 +0000 (11:45 -0400)]
VPP-1440: clean up coverity warnings

Change-Id: Ie315d4e83c0575d82aee0369ef50e8dd2dad6a2a
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agoVPP-1426: CLI crash on 'set int feature' 15/15115/5
Paul Vinciguerra [Wed, 3 Oct 2018 02:02:16 +0000 (19:02 -0700)]
VPP-1426: CLI crash on 'set int feature'

Change-Id: Ib2221a43347c66940b6b5f71914a451ca4627d2f
Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
5 years agosock api: fix registrations and client reads 14/15114/5
Florin Coras [Wed, 3 Oct 2018 01:29:25 +0000 (18:29 -0700)]
sock api: fix registrations and client reads

- When clients connect, instead of returing registration indicies return
handles. By convention socket registrations will have the MSB set to 1.
This makes it easy to distinguish them from shm registrations.
- Fix client reads to allow for messages larger than 4kB (needed for the
creat reply wherein the message table is provided).

Change-Id: I7bc0a072d066dffbf2e3ad9ba3ed50291231af9d
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agostat_client.c coverity error. 20/15120/2
Ole Troan [Wed, 3 Oct 2018 10:46:54 +0000 (12:46 +0200)]
stat_client.c coverity error.

Change-Id: Ic072beb8882981425f436936f239e8c8bf229a99
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agovmxnet3: error handling 11/15111/3
Steven [Tue, 2 Oct 2018 21:36:13 +0000 (14:36 -0700)]
vmxnet3: error handling

outbound -- if we have to drop the packet due to no descriptor space is available, drop the
whole packet, not fragments.
inbound -- check and drop error packets

Change-Id: Ida1d32e61521bafd67f714d729ad53cd7c487dc6
Signed-off-by: Steven <sluong@cisco.com>
5 years agofix format error in show logging config output 10/15110/3
Jerome Tollet [Tue, 2 Oct 2018 20:54:30 +0000 (22:54 +0200)]
fix format error in show logging config output

Change-Id: Idc41a219db185b524f497b096eb71892b5f9ebf8
Signed-off-by: Jerome Tollet <jtollet@cisco.com>
5 years agojvppgen: update headers 12/15112/1
Bernhard M. Wiedemann [Tue, 2 Oct 2018 23:16:19 +0000 (01:16 +0200)]
jvppgen: update headers

headers of generated files should correctly tell
about where it cames from to not confuse readers.

Change-Id: If667fa4aab188978156982adf3fa13474bd93e9e
Signed-off-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
5 years agoEnabled untagged vs default functionality 86/15086/3
Mike Bly [Mon, 24 Sep 2018 17:13:06 +0000 (10:13 -0700)]
Enabled untagged vs default functionality
Removed 0-tags attribute for default-sub-if config
Moved default-sub-if check before untagged

Change-Id: I68043445aa2f79846e0743567b9015257fd87f8d
Signed-off-by: Mike Bly <mbly@ciena.com>
5 years agoVPP-1440: clean up coverity warnings 09/15109/2
Dave Barach [Tue, 2 Oct 2018 20:33:56 +0000 (16:33 -0400)]
VPP-1440: clean up coverity warnings

Change-Id: Ic6823fb617ecae547a5f0e28b1e037848e40f682
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agoStats: vpp_prometheus_export fixes. 08/15108/3
Ole Troan [Tue, 2 Oct 2018 20:00:29 +0000 (22:00 +0200)]
Stats: vpp_prometheus_export fixes.

- Crash on /sys/nodes if per-node-counters off
- Require patterns parameters.

Change-Id: If4fddefb17504a92ba462c7af91e373601b66a5c
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agoPAPI: Use UNIX domain sockets instead of shared memory 10/13910/9
Ole Troan [Thu, 2 Aug 2018 09:58:12 +0000 (11:58 +0200)]
PAPI: Use UNIX domain sockets instead of shared memory

Adds support for running the API purely across Unix domain sockets.
Usage: vpp = VPP(use_socket=True)

Change-Id: Iafc1301e03dd3edc3f4d702dd6c0b98d3b50b69e
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agovcl: fix coverity warning 03/15103/2
Florin Coras [Tue, 2 Oct 2018 15:42:06 +0000 (08:42 -0700)]
vcl: fix coverity warning

Change-Id: I08b0244ba8b0e8a463224b39e1f7b878ba8e4e6f
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agoip4-local: classify protos that skip csum and src check 90/15090/3
Florin Coras [Tue, 2 Oct 2018 04:59:18 +0000 (21:59 -0700)]
ip4-local: classify protos that skip csum and src check

Makes it easy to add protocols that skip L4 checks.

Change-Id: I80ff95685bfa17e29c5800d9c96153cdc7739be2
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agovhost-user: add support for vlib_log API 95/15095/2
Jerome Tollet [Tue, 2 Oct 2018 09:41:11 +0000 (11:41 +0200)]
vhost-user: add support for vlib_log API

Change-Id: I0af68f6b41d0024aa64b93a8b18e2d179bf939b0
Signed-off-by: Jerome Tollet <jtollet@cisco.com>
Signed-off-by: Damjan Marion <damarion@cisco.com>
5 years agoFix JVPP enum _host_to_net_ translation (VPP-1438) 76/15076/3
Michal Cmarada [Mon, 1 Oct 2018 14:26:31 +0000 (16:26 +0200)]
Fix JVPP enum _host_to_net_ translation (VPP-1438)

- use ordinal value of enumeration instead of accessing
  its value directly.

Change-Id: I6327a8f7668852846907a92c2713de9ff90c1c75
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
5 years agoBFD: add get echo source API (VPP-1367) 96/15096/2
Matus Fabian [Tue, 2 Oct 2018 10:22:18 +0000 (03:22 -0700)]
BFD: add get echo source API (VPP-1367)

Change-Id: I45f540128e038ddb17ba96ce30965e8f7c732067
Signed-off-by: Matus Fabian <matfabia@cisco.com>
5 years agoIPsec: add API for SPDs dump (VPP-1363) 92/15092/2
Matus Fabian [Tue, 2 Oct 2018 08:13:25 +0000 (01:13 -0700)]
IPsec: add API for SPDs dump (VPP-1363)

Change-Id: I36e6878712c394de629a9182d2af24c53a8f811d
Signed-off-by: Matus Fabian <matfabia@cisco.com>
5 years agovhost-user: Interface state updates 72/15072/2
Juraj Sloboda [Mon, 1 Oct 2018 10:42:07 +0000 (12:42 +0200)]
vhost-user: Interface state updates

Fix inconsistencies between admin and link interface states
Admin down should imply link down:
  link_up = admin_up && link_ready

Change-Id: I4d668d82d035b5d2ae508727f34f1722a0c3e677
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agovhost_user: Fix setting MTU using uninitialized variable 71/15071/2
Juraj Sloboda [Fri, 28 Sep 2018 10:04:26 +0000 (12:04 +0200)]
vhost_user: Fix setting MTU using uninitialized variable

Change-Id: I0caa5fd584e3785f237d08f3d3be23e9bfee7605
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agoUpdate code to compute checksum for buffer chains 15/15015/2
Juraj Sloboda [Wed, 26 Sep 2018 12:25:32 +0000 (14:25 +0200)]
Update code to compute checksum for buffer chains

Compute ICMP checksum for buffer chains
Fix checksum function for buffer chains

Change-Id: I39b845b94a63c3ab5fc9f6f9ab36cadbc67c104f
Signed-off-by: Juraj Sloboda <jsloboda@cisco.com>
5 years agotls: fix disconnects for sessions with pending data 64/15064/5
Florin Coras [Sat, 29 Sep 2018 17:23:06 +0000 (10:23 -0700)]
tls: fix disconnects for sessions with pending data

TLS can enqueue events to itself when app session queue cannot be
entirely drained. If a pending disconnect is handled before any such
event, session layer may try to dequeue data on deallocated sessions.

Change-Id: I5bfc4d53ce95bc16b6a01e1b0e644aafa1ca311b
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agotcp: fix close wait timeout with no fin 79/15079/3
Florin Coras [Mon, 1 Oct 2018 15:32:04 +0000 (08:32 -0700)]
tcp: fix close wait timeout with no fin

Change-Id: Icba9b0dc6dcb4b72288f966728201812d8d12144
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agotcp: accept fins if in recovery 89/15089/2
Florin Coras [Tue, 2 Oct 2018 01:31:02 +0000 (18:31 -0700)]
tcp: accept fins if in recovery

Change-Id: I0c9c055fcc3d681c4032228a90cc81f484e200f0
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agoFix documentation about sw_interface_set_l2_bridge 87/15087/1
Yichen Wang [Mon, 1 Oct 2018 18:15:25 +0000 (11:15 -0700)]
Fix documentation about sw_interface_set_l2_bridge

Change-Id: I53e0e7d2cf1e3c52794830b33fa93d1582b74769
Signed-off-by: Yichen Wang <yicwang@cisco.com>
5 years agoSource VRF Select 77/15077/2
Neale Ranns [Mon, 1 Oct 2018 08:42:13 +0000 (01:42 -0700)]
Source VRF Select

match against a packet's source address to determine
the VRF for the subsequent destination address lookup.

Change-Id: I48ee0ef54dcb891f0ec7f879e4d3b925a0ed0081
Signed-off-by: Neale Ranns <nranns@cisco.com>
5 years agoRename device class from PPPPOE to PPPoE 81/15081/2
Paul Vinciguerra [Mon, 1 Oct 2018 15:36:26 +0000 (08:36 -0700)]
Rename device class from PPPPOE to PPPoE

Change-Id: Ia20eee6cef360b1acd768101df0713f0005a7d14
Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
5 years agodpdk: updated makefile to enable QAT cryptodev 78/15078/2
Radu Nicolau [Mon, 1 Oct 2018 14:57:31 +0000 (15:57 +0100)]
dpdk: updated makefile to enable QAT cryptodev

Change-Id: I2d70f2f29f3af10b6e43dde235fdf90a4596eb35
Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
5 years agoAdd adjacency counters to the stats segment 35/15035/4
Neale Ranns [Fri, 28 Sep 2018 12:00:57 +0000 (05:00 -0700)]
Add adjacency counters to the stats segment

Change-Id: I6b59df939c9daf40e261d73d19f500bd90abe6ff
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
5 years agovat: fix coverity warning for show_threads 82/15082/2
Mohsin Kazmi [Mon, 1 Oct 2018 15:41:08 +0000 (17:41 +0200)]
vat: fix coverity warning for show_threads

Change-Id: I8c226b91cc45f6f345444afd8d168e1292371582
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agoAPI / CLI event-log tracing 73/15073/2
Dave Barach [Mon, 1 Oct 2018 13:25:32 +0000 (09:25 -0400)]
API / CLI event-log tracing

Add an "elog trace [api][cli][barrier]" debug CLI command. Removed the
barrier elog test command. Remove unused reliable multicast code.

Change-Id: Ib3ecde901b7c49fe92b313d0087cd7e776adcdce
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agoFix the path error inside vcl socket_test.sh 12/14912/4
Yalei Wang [Thu, 20 Sep 2018 08:46:29 +0000 (16:46 +0800)]
Fix the path error inside vcl socket_test.sh

Change-Id: I5bcfbd436839e7a6dd82dc57cdb3b7fb6200a69c
Signed-off-by: Yalei Wang <william.wangyalei@huawei.com>
Signed-off-by: Dave Wallace <dwallacelf@gmail.com>
5 years agothread: Add show threads api 65/14765/10
Mohsin Kazmi [Tue, 11 Sep 2018 18:27:09 +0000 (20:27 +0200)]
thread: Add show threads api

Change-Id: I3124238ab4d43bcef5590bad33a4ff0b5d8b7d15
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agomroute routers in the stats segment 46/15046/3
Neale Ranns [Fri, 7 Sep 2018 16:37:07 +0000 (09:37 -0700)]
mroute routers in the stats segment

Change-Id: I798e4fb6470ae9e763f8de1c290ff0fc3c0b7f9e
Signed-off-by: Neale Ranns <nranns@cisco.com>
5 years agoIGMP: handle (*,G) report with no source addresses 66/15066/2
Neale Ranns [Wed, 26 Sep 2018 13:51:39 +0000 (06:51 -0700)]
IGMP: handle (*,G) report with no source addresses

Change-Id: I363370b9d4a27b992bad55c48fc930a2fbea2165
Signed-off-by: Neale Ranns <nranns@cisco.com>
5 years agoSCTP: fix overflow issue with timestamp 20/15020/9
Marco Varlese [Thu, 27 Sep 2018 14:43:57 +0000 (16:43 +0200)]
SCTP: fix overflow issue with timestamp

Change-Id: I03bb47a2baa4375b7bf9347d95c4cc8de37fe510
Signed-off-by: Marco Varlese <marco.varlese@suse.com>
5 years agoSupport dynamic dual/quad loop selection on aarch64 05/14905/11
Lijian Zhang [Mon, 10 Sep 2018 09:13:56 +0000 (17:13 +0800)]
Support dynamic dual/quad loop selection on aarch64

Currently, there are three variants available on aarch64, qdf24xx, thunderx2t99, and cortex-a72.
-DCLIB_N_PREFETCHES is passed to source code to select dual/quad implementation.
Besides, different compiler options are applied on these critical functions.

gcc-7.3.0 reports ICE(internal compiler error) with -mtune=thunderx2t99,
so -mtune=thunderx2t99 is enabled only when gcc version is greater than 7.3.0

Cavium ThunderX2, Impermenter 0x43, Part 0x0af
    -march=armv8-a+crc+crypto -mtune=thunderx2t99
Qualcomm Centriq 2400, Impermenter 0x51, Part 0xc00
    -march=armv8.1-a+crc+crypto -mtune=qdf24xx
Cortex-A72, Impermenter 0x41, Part 0xd08
    -march=armv8-a+crc+crypto -mtune=cortex-a72

Change-Id: Id5649c6325c1e642d0fd42535e3908793b13e02a
Signed-off-by: Lijian Zhang <Lijian.Zhang@arm.com>
Reviewed-by: Sirshak Das <sirshak.das@arm.com>
Reviewed-by: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
5 years agovlib: barrier sync elog tracing improvements 36/15036/4
Dave Barach [Sun, 30 Sep 2018 19:54:06 +0000 (15:54 -0400)]
vlib: barrier sync elog tracing improvements

Change-Id: I2d3b8d5a7192ff68bee443a99346ecb807b2d833
Signed-off-by: Damjan Marion <damarion@cisco.com>
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agovom: Fix the l2 port type in bridge domain 19/15019/2
Mohsin Kazmi [Thu, 27 Sep 2018 13:00:32 +0000 (15:00 +0200)]
vom: Fix the l2 port type in bridge domain

Change-Id: I8aba11142daeacce892ec529c2229ec54ee427e2
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agovcl: handle old events before blocking in epoll 54/15054/2
Florin Coras [Fri, 28 Sep 2018 21:59:37 +0000 (14:59 -0700)]
vcl: handle old events before blocking in epoll

Change-Id: I3e62c787882d93c3bfb398ed0d04ef56b3a60b2b
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agosession: lock msg queues while generating control events 48/15048/4
Florin Coras [Fri, 28 Sep 2018 17:35:45 +0000 (10:35 -0700)]
session: lock msg queues while generating control events

Change-Id: I229ce9c306a97956c87ccb8f7a1c5c9dac94ab31
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agogitignore: build/external/downloads 55/15055/1
Florin Coras [Fri, 28 Sep 2018 22:03:16 +0000 (15:03 -0700)]
gitignore: build/external/downloads

Change-Id: I041764a5546352086aa9803e4bbeb9dff5f8d69a
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agobond: tx performance enhancement part deux 29/15029/4
Steven [Fri, 28 Sep 2018 03:06:26 +0000 (20:06 -0700)]
bond: tx performance enhancement part deux

- Reduce per packet cost by buffering the output packet buffer indexes in the queue and
process the queue outside the packet processing loop.
- Move unnecessary variable initialization outside of the while loop.
- There is no need to save the old interface if tracing is not enabled.

Test result for 256 bytes packet comparison. Other packet size shows similar improvement.

With the patch
--------------
BondEthernet0-output             active              52836        13526016               0          1.71e1          256.00
BondEthernet0-tx                 active              52836        13526016               0          2.68e1          256.00
TenGigabitEthernet6/0/0-output   active              52836         6762896               0          9.17e0          127.99
TenGigabitEthernet6/0/0-tx       active              52836         6762896               0          6.97e1          127.99
TenGigabitEthernet6/0/1-output   active              52836         6763120               0          9.40e0          128.00
TenGigabitEthernet6/0/1-tx       active              52836         6763120               0          7.00e1          128.00
bond-input                       active              52836        13526016               0          1.76e1          256.00

Without the patch
-----------------
BondEthernet0-output             active              60858        15579648               0          1.73e1          256.00
BondEthernet0-tx                 active              60858        15579648               0          2.94e1          256.00
TenGigabitEthernet6/0/0-output   active              60858         7789626               0          9.29e0          127.99
TenGigabitEthernet6/0/0-tx       active              60858         7789626               0          7.01e1          127.99
TenGigabitEthernet6/0/1-output   active              60858         7790022               0          9.31e0          128.00
TenGigabitEthernet6/0/1-tx       active              60858         7790022               0          7.10e1          128.00
bond-input                       active              60858        15579648               0          1.77e1          256.00

Change-Id: Ib6d73a63ceeaa2f1397ceaf4c5391c57fd865b04
Signed-off-by: Steven <sluong@cisco.com>
5 years agoMAP: Add check for well known ports. 40/15040/3
Ole Troan [Fri, 28 Sep 2018 12:28:00 +0000 (14:28 +0200)]
MAP: Add check for well known ports.

And more unit-tests.

Change-Id: I4667d82d928b7ba8d96b5a5648d464115b3ed216
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agoIP ttl check in ip4-input missing for single packet path. 39/15039/1
Ole Troan [Fri, 28 Sep 2018 12:27:24 +0000 (14:27 +0200)]
IP ttl check in ip4-input missing for single packet path.

Change-Id: Idc17b2f8794d37cd3242a97395ab56bd633ca575
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agostats: Split stat_segment and stats code in preparation for deprecation. 97/14997/6
Ole Troan [Wed, 26 Sep 2018 13:26:42 +0000 (15:26 +0200)]
stats: Split stat_segment and stats code in preparation for deprecation.

Split the stat_segment.c code from stats.c.
Rename stats.[ch] to prepare for removing (19.01?)
In addition stats.api can be removed.

Since the stats aggregation for the stat segment does not use the API, that
part is now done on the main thread. (Old stats aggregator is also left in place).

Change-Id: I9867429f4fc547b1a7ab7f88bc4f3625428d681b
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agoRevert "japi: Move Java API binding to cmake" 26/15026/2
Damjan Marion [Thu, 27 Sep 2018 18:02:28 +0000 (18:02 +0000)]
Revert "japi: Move Java API binding to cmake"

components:  Development
CMake Error at /usr/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find JNI (missing: JAVA_AWT_INCLUDE_PATH)
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.10/Modules/FindJNI.cmake:310 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:37 (find_package)
-- Configuring incomplete, errors occurred!
See also "/scratch/vpp-tmp/build-root/build-vpp-native/japi/CMakeFiles/CMakeOutput.log".
See also "/scratch/vpp-tmp/build-root/build-vpp-native/japi/CMakeFiles/CMakeError.log".
Makefile:640: recipe for target 'japi-configure' failed
make[2]: *** [japi-configure] Error 1
make[2]: Leaving directory '/scratch/vpp-tmp/build-root'

This reverts commit a33db2528485ac39e8987bec7b138dbdad21dc5a.

Change-Id: I8a818ce70a8379656c1134432e22db418a4690fe
Signed-off-by: Damjan Marion <damarion@cisco.com>
5 years agoMAP: Correctly print ea-bits-len in map commands. 13/15013/4
Ole Troan [Thu, 27 Sep 2018 11:54:30 +0000 (13:54 +0200)]
MAP: Correctly print ea-bits-len in map commands.

Change-Id: I17b19875547cd6056aaf8f8fde439caa8e42d274
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agoMAP: 64-bit DMR 12/15012/4
Ole Troan [Thu, 27 Sep 2018 11:53:30 +0000 (13:53 +0200)]
MAP: 64-bit DMR

Change-Id: I261e11f1006039e4a4019fec2fd3271763ae476a
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agoTrivial: Cleanup some typos. 50/14950/2
Paul Vinciguerra [Mon, 24 Sep 2018 12:25:00 +0000 (05:25 -0700)]
Trivial: Cleanup some typos.

This is a new commit for code under a different maintainer.

Change-Id: I79fa403fec6a312238a9a4b18b35dbcafaa05439
Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
5 years agoAdd logging to vfio code 17/15017/2
Damjan Marion [Thu, 27 Sep 2018 12:54:49 +0000 (14:54 +0200)]
Add logging to vfio code

Change-Id: Iacdff0a5006a0617e3c9559ce7b258f26e256abb
Signed-off-by: Damjan Marion <damarion@cisco.com>
5 years agodpdk_plugin: fix mlx5 build and runtime issues 80/14980/3
Sirshak Das [Sat, 22 Sep 2018 19:57:26 +0000 (14:57 -0500)]
dpdk_plugin: fix mlx5 build and runtime issues

There are issues with VPP finding and linking the mlx5 shared glue
library which was built by default if mlx5 was enabled.

Runtime Errors this patch fixes:

net_mlx5: cannot load glue library: librte_pmd_mlx5_glue.so.18.05.0:
cannot open shared object file: No such file or directory

net_mlx5: cannot initialize PMD due to missing run-time dependency on
rdma-core libraries (libibverbs, libmlx5)

This patch introduces additional config parameter to disable glue
library building and instead statically link ibverbs and mlx5
libraries to the PMD and dpdk_plugin.

Change-Id: I0b2f67652a57854c778e991780903fb15706ace8
Signed-off-by: Sirshak Das <sirshak.das@arm.com>
Reviewed-by: Lijian Zhang <Lijian.Zhang@arm.com>
5 years agolibmemif: external region bugfix 09/15009/4
Jakub Grajciar [Thu, 27 Sep 2018 08:26:35 +0000 (10:26 +0200)]
libmemif: external region bugfix

Change-Id: I7dbece80dda917ff78f53f8f4074132895aed316
Signed-off-by: Jakub Grajciar <jgrajcia@cisco.com>
5 years agomemif: Memif Test Case 90/14890/10
Jakub Grajciar [Wed, 19 Sep 2018 11:36:16 +0000 (13:36 +0200)]
memif: Memif Test Case

Change-Id: Ic0d5fc6ccbd496afcc870b908ef799af7c804c30
Signed-off-by: Jakub Grajciar <jgrajcia@cisco.com>
5 years ago"show log": print wall-clock time 16/15016/2
Dave Barach [Thu, 27 Sep 2018 12:46:15 +0000 (08:46 -0400)]
"show log": print wall-clock time

Change-Id: I36f42c03f778955dd543da6c878be090d0443922
Signed-off-by: Dave Barach <dave@barachs.net>
5 years agotcp: use scaled window for new connects 24/15024/3
Florin Coras [Thu, 27 Sep 2018 16:19:29 +0000 (09:19 -0700)]
tcp: use scaled window for new connects

Change-Id: Idf83fce8ca176e57b323e3741034e3223f1d195a
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agoadd cmake build option to build openssl async 07/15007/3
Ping Yu [Thu, 27 Sep 2018 12:49:33 +0000 (08:49 -0400)]
add cmake build option to build openssl async

This code is orignally in automake, but it is missing in cmake.
Thus add it to make openssl async work in cmake build system

Change-Id: Ie69ee9c2099273e51ce13ccab27bdd2619db4814
Signed-off-by: Ping Yu <ping.yu@intel.com>
5 years agoavf: may crash if failed to allocate buffers to ring 03/15003/3
Steven [Wed, 26 Sep 2018 17:56:46 +0000 (10:56 -0700)]
avf: may crash if failed to allocate buffers to ring

In avf_rxq_refill, we invoke vlib_buffer_alloc_to_ring which may fill buffers from the
end of the ring and continue to the beginning of the ring. If we fill some in the end and
continue to fill some in the beginning, but does not have enough buffers to fill the
whole request, n_alloc returns a value which is not equal to n_refill to indicate partial
refill. We don't like partial refill and invoke vlib_buffer_free to get rid of
the buffers that just got refilled. However, vlib_buffer_free API is to free the buffers
from the slot continuously. It does not know how to free some from rxq->bufs[slot], and then
continue to free the rest when it reaches the end of the ring.

The fix is to use vlib_buffer_free_from_ring which is smart enough to figure that stuff
out.

Change-Id: I93c28e0b0d8d8f22c321d1a5912e00c27b4e2e8d
Signed-off-by: Steven <sluong@cisco.com>
5 years agoSCTP: fix advanced debug output 11/15011/2
Marco Varlese [Thu, 27 Sep 2018 09:58:34 +0000 (11:58 +0200)]
SCTP: fix advanced debug output

Change-Id: Ied63ebaec8e19189f8b1ab2a7f6d7474c3f2ad6e
Signed-off-by: Marco Varlese <marco.varlese@suse.com>
5 years agoTest typo fixes 10/15010/2
juraj.linkes [Wed, 19 Sep 2018 12:59:43 +0000 (14:59 +0200)]
Test typo fixes

Renamed one acl testcase class which was copy-pasted and fixed one
inheritance issue.

Renamed one bihash testcase class.

Change-Id: I70d911ee7872d8d9ddba96c7f721dd099f0152f7
Signed-off-by: juraj.linkes <juraj.linkes@pantheon.tech>
5 years agofix typo in vpp-ext-deps rpm packaging 14/15014/1
Damjan Marion [Thu, 27 Sep 2018 12:04:43 +0000 (14:04 +0200)]
fix typo in vpp-ext-deps rpm packaging

Change-Id: Ia23a876cefbfd32d6f543a77dfec57a4aa5676ae
Signed-off-by: Damjan Marion <damarion@cisco.com>
5 years agoIPIP and IPv6 fragmentation 84/14984/6
Ole Troan [Wed, 19 Sep 2018 10:38:51 +0000 (12:38 +0200)]
IPIP and IPv6 fragmentation

- Error where ICMPv6 error code doesn't reset VLIB_TX = -1
  Leading to crash for ICMP generated on tunnelled packets
- Missed setting VNET_BUFFER_F_LOCALLY_ORIGINATED, so
  IP in IPv6 packets never got fragmented.
- Add support for fragmentation of buffer chains.
- Remove support for inner fragmentation in frag code itself.

Change-Id: If9a97301b7e35ca97ffa5c0fada2b9e7e7dbfb27
Signed-off-by: Ole Troan <ot@cisco.com>
5 years agovom: Add ip punt redirect 77/14977/4
Mohsin Kazmi [Tue, 25 Sep 2018 16:26:52 +0000 (18:26 +0200)]
vom: Add ip punt redirect

Change-Id: I5c7b925b22b53f18b0c45374883a6a4bff580c75
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agovnet: show interface rx-placement may display garbage characters 02/15002/2
Steven [Wed, 26 Sep 2018 17:44:54 +0000 (10:44 -0700)]
vnet: show interface rx-placement may display garbage characters

Before
------
DBGvpp# sh int rx-mode
sh int rx-mode
Thread 1 (vpp_wk_0^@):
  node vmxnet3-input:
    vmxnet3-0/b/0/0 queue 0 (polling)
Thread 2 (vpp_wk_1^@):
  node vmxnet3-input:
    vmxnet3-0/13/0/0 queue 0 (polling)
DBGvpp#

After
-----
DBGvpp# sh int rx-placement
sh int rx-placement
Thread 1 (vpp_wk_0):
  node vmxnet3-input:
    vmxnet3-0/b/0/0 queue 0 (polling)
Thread 2 (vpp_wk_1):
  node vmxnet3-input:
    vmxnet3-0/13/0/0 queue 0 (polling)
DBGvpp#

Change-Id: I5910d502757054c3942fac9d20c5104e95fc6b56
Signed-off-by: Steven <sluong@cisco.com>
5 years agovmxnet3: delete interface causes a crash 01/15001/2
Steven [Wed, 26 Sep 2018 17:14:18 +0000 (10:14 -0700)]
vmxnet3: delete interface causes a crash

Need to free the buffers starting from ring->consume+1, not ring->consume
Make use of the cool API vlib_buffer_free_from_ring
Fix a memory leak in vmxnet3_txq_init

Change-Id: I0a539ea75211408d84ce433d97a0a0aec5a9618d
Signed-off-by: Steven <sluong@cisco.com>
5 years agoacl-plugin: fix the stateful ICMP handling and add testcases 04/15004/2
Andrew Yourtchenko [Wed, 26 Sep 2018 19:03:06 +0000 (21:03 +0200)]
acl-plugin: fix the stateful ICMP handling and add testcases

The stateful ICMP/ICMPv6 handling got broken.
Fix that and introduce testcases to catch in the future.

Change-Id: Ie602e72d6ac613d64ab0bf6693b6d75afb1a9552
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
5 years agoMPLS tunnel dump: use sw_if_index not tunnel_index 95/14995/2
Neale Ranns [Wed, 26 Sep 2018 12:07:25 +0000 (05:07 -0700)]
MPLS tunnel dump: use sw_if_index not tunnel_index

Change-Id: I6c0d5aec6ee96a0d40358f0e09a0901b22265063
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
5 years agoperftool: fix c2cpel 81/14981/2
Florin Coras [Tue, 25 Sep 2018 23:27:51 +0000 (16:27 -0700)]
perftool: fix c2cpel

Change-Id: I0f39477bbf88d490409fdcd5f58df55cfe2ec531
Signed-off-by: Florin Coras <fcoras@cisco.com>
5 years agoforget to assign spd_index to config 91/14991/2
Kingwel Xie [Wed, 26 Sep 2018 09:07:09 +0000 (05:07 -0400)]
forget to assign spd_index to config

Change-Id: I35dcb987edf11097f34a633ac36d87cecd12088f
Signed-off-by: Kingwel Xie <kingwel.xie@ericsson.com>
5 years agojapi: Move Java API binding to cmake 52/14952/9
Mohsin Kazmi [Thu, 13 Sep 2018 07:59:50 +0000 (09:59 +0200)]
japi: Move Java API binding to cmake

Change-Id: Ia7ddd9f9e2e2d9c133967593c3e187fe42501eac
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agorpm: Fix the spec file to look into right directories 92/14992/3
Mohsin Kazmi [Wed, 26 Sep 2018 08:49:41 +0000 (10:49 +0200)]
rpm: Fix the spec file to look into right directories

Change-Id: If526e9f17226d7c371a8a98b9ac932196dab547a
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agomismatch bi0 and b0 when finalizing ip reassembly. 90/14990/2
Kingwel Xie [Wed, 26 Sep 2018 08:59:52 +0000 (04:59 -0400)]
mismatch bi0 and b0 when finalizing ip reassembly.
bi0 always points to the first buffer in chain but b0 could be any fragment in a ip packet.
It causes a vnet_feature_next function call to the wrong buffer
and crashes sometime when there is >1 feature nodes under ip-unicast.
The fix is simple, makes b0 and bi0 both pointed to the first buffer

Change-Id: I7de36e68fb42b050fa63201abd98aeb6ba2e0cd3
Signed-off-by: Kingwel Xie <kingwel.xie@ericsson.com>
5 years agodpdk: fix QSFP+ module info 96/14996/2
Damjan Marion [Wed, 26 Sep 2018 13:28:17 +0000 (15:28 +0200)]
dpdk: fix QSFP+ module info

Change-Id: I89c7df778e66a5d2147190dc99445405d81964e5
Signed-off-by: Damjan Marion <damarion@cisco.com>
5 years agoModify return values of L2 bridge API handlers 51/14951/4
Alexander Chernavin [Mon, 24 Sep 2018 09:42:01 +0000 (05:42 -0400)]
Modify return values of L2 bridge API handlers

Change-Id: Ie6a6dde1fdc0e4fa8560682072a69876867a88d3
Signed-off-by: Alexander Chernavin <achernavin@netgate.com>
5 years agoNAT44: endpoint-dependent mode session timeout improvement (VPP-1423) 85/14985/2
Matus Fabian [Wed, 26 Sep 2018 08:08:00 +0000 (01:08 -0700)]
NAT44: endpoint-dependent mode session timeout improvement (VPP-1423)

Change-Id: Ib62e503f4eb5d72431288de32f417a4553df4e0c
Signed-off-by: Matus Fabian <matfabia@cisco.com>
5 years agobond: tx perf improvements 59/14959/4
Damjan Marion [Fri, 21 Sep 2018 16:11:33 +0000 (18:11 +0200)]
bond: tx perf improvements

Change-Id: I0c3f2add35ad9fc11308b7a2a2c69ffd8472dd2e
Signed-off-by: Damjan Marion <damarion@cisco.com>
5 years agoitf: dump interface rx-placement 51/14751/5
Mohsin Kazmi [Mon, 10 Sep 2018 16:11:00 +0000 (18:11 +0200)]
itf: dump interface rx-placement

Change-Id: I7dd48c8a2c77f82a2c1aa8311b062f5f0bc4e3fd
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
5 years agoFIB: recusrive paths must lock the table to prevent its deletion 42/14942/4
Neale Ranns [Tue, 11 Sep 2018 02:14:07 +0000 (19:14 -0700)]
FIB: recusrive paths must lock the table to prevent its deletion

Change-Id: I958bf057be751dc7b3a0d93080021b3addc405b7
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
5 years agoL2 BD: introduce a BD interface on which to send UU packets 89/14689/5
Neale Ranns [Wed, 5 Sep 2018 16:13:57 +0000 (09:13 -0700)]
L2 BD: introduce a BD interface on which to send UU packets

Change-Id: I21ad6b04c19c8735d057174b1f260a59f2812241
Signed-off-by: Neale Ranns <nranns@cisco.com>