This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "".
The branch, master has been updated
via 9163719bd4c04321592dad9da7f26539f49c8b7a (commit)
from 7e9004e87bab2eacd82d449286a84ae7a8cb75ab (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 9163719bd4c04321592dad9da7f26539f49c8b7a
Author: Matias Elo <matias.elo(a)nokia.com>
Date: Thu Nov 9 15:14:32 2017 +0200
travis: add build only test stage
Add basic build only test stage to the beginning of test run. This way
potential build errors are caught early on without having to wait for the
longer test jobs to finish.
Signed-off-by: Matias Elo <matias.elo(a)nokia.com>
Reviewed-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov(a)linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org>
diff --git a/.travis.yml b/.travis.yml
index 6bd03a0c..b5c1b641 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -11,6 +11,9 @@ language: c
sudo: required
dist: trusty
group: deprecated-2017Q2
+stages:
+ - "build only"
+ - test
env:
global:
# COVERITY_SCAN_TOKEN
@@ -256,7 +259,7 @@ jobs:
- ./configure --prefix=$HOME/odp-install
--enable-user-guides
- sudo PATH="$PATH" LD_LIBRARY_PATH="$HOME/cunit-install/$CROSS_ARCH/lib:$LD_LIBRARY_PATH" make distcheck
- - stage: test
+ - stage: "build only"
env: TEST=doxygen
compiler: gcc
install:
@@ -282,6 +285,38 @@ jobs:
- echo ${TRAVIS_COMMIT_RANGE};
- ODP_PATCHES=`echo ${TRAVIS_COMMIT_RANGE} | sed 's/\.//'`;
- ./scripts/ci-checkpatches.sh ${ODP_PATCHES};
+ - stage: "build only"
+ env: CONF=""
+ compiler: gcc
+ install: true
+ script:
+ - ./bootstrap
+ - ./configure --enable-helper-linux
+ - make
+ - stage: "build only"
+ env: CONF=""
+ compiler: clang-3.8
+ install: true
+ script:
+ - ./bootstrap
+ - ./configure --enable-helper-linux
+ - make
+ - stage: "build only"
+ env: CROSS_ARCH="i386"
+ compiler: gcc
+ install: true
+ script:
+ - ./bootstrap
+ - ./configure --enable-helper-linux $CROSS
+ - make
+ - stage: "build only"
+ env: CROSS_ARCH="arm64"
+ compiler: gcc
+ install: true
+ script:
+ - ./bootstrap
+ - ./configure --enable-helper-linux $CROSS
+ - make
after_failure:
- cat config.log
-----------------------------------------------------------------------
Summary of changes:
.travis.yml | 37 ++++++++++++++++++++++++++++++++++++-
1 file changed, 36 insertions(+), 1 deletion(-)
hooks/post-receive
--
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "".
The branch, master has been updated
via 7e9004e87bab2eacd82d449286a84ae7a8cb75ab (commit)
via d593398b2d55faf1e07f124a4c807c006eb856ea (commit)
via 4af5396e77ff170c80ec3720028493ed2c9f4826 (commit)
from fc020907bee7ec2ba976bc02399f6fd47f110d65 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 7e9004e87bab2eacd82d449286a84ae7a8cb75ab
Author: Matias Elo <matias.elo(a)nokia.com>
Date: Wed Nov 8 14:40:46 2017 +0200
linux-gen: dpdk: call packet parse function only when necessary
Not calling packet_parse_layer() when packet input parsing is disabled
removes two unnecessary function calls from fast path.
Signed-off-by: Matias Elo <matias.elo(a)nokia.com>
Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org>
diff --git a/platform/linux-generic/pktio/dpdk.c b/platform/linux-generic/pktio/dpdk.c
index d7e9fcd4..b5a87b82 100644
--- a/platform/linux-generic/pktio/dpdk.c
+++ b/platform/linux-generic/pktio/dpdk.c
@@ -439,7 +439,7 @@ static inline int mbuf_to_pkt(pktio_entry_t *pktio_entry,
if (pktio_cls_enabled(pktio_entry))
copy_packet_cls_metadata(&parsed_hdr, pkt_hdr);
- else
+ else if (pktio_entry->s.config.parser.layer)
packet_parse_layer(pkt_hdr,
pktio_entry->s.config.parser.layer);
@@ -654,7 +654,7 @@ static inline int mbuf_to_pkt_zero(pktio_entry_t *pktio_entry,
if (pktio_cls_enabled(pktio_entry))
copy_packet_cls_metadata(&parsed_hdr, pkt_hdr);
- else
+ else if (pktio_entry->s.config.parser.layer)
packet_parse_layer(pkt_hdr,
pktio_entry->s.config.parser.layer);
commit d593398b2d55faf1e07f124a4c807c006eb856ea
Author: Matias Elo <matias.elo(a)nokia.com>
Date: Fri May 19 16:01:17 2017 +0300
linux-gen: dpdk: fix max_input_queues value for ixgbe devices
The RSS functionality is limited to max 16 queues on devices using ixgbe
driver even though dpdk rte_eth_dev_info may return a larger value.
Signed-off-by: Matias Elo <matias.elo(a)nokia.com>
Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org>
diff --git a/platform/linux-generic/pktio/dpdk.c b/platform/linux-generic/pktio/dpdk.c
index 3684ca5c..d7e9fcd4 100644
--- a/platform/linux-generic/pktio/dpdk.c
+++ b/platform/linux-generic/pktio/dpdk.c
@@ -1135,6 +1135,13 @@ static void dpdk_init_capability(pktio_entry_t *pktio_entry,
rte_eth_dev_info_get(pkt_dpdk->port_id, dev_info);
capa->max_input_queues = RTE_MIN(dev_info->max_rx_queues,
PKTIO_MAX_QUEUES);
+
+ /* ixgbe devices support only 16 rx queues in RSS mode */
+ if (!strncmp(dev_info->driver_name, IXGBE_DRV_NAME,
+ strlen(IXGBE_DRV_NAME)))
+ capa->max_input_queues = RTE_MIN((unsigned)16,
+ capa->max_input_queues);
+
capa->max_output_queues = RTE_MIN(dev_info->max_tx_queues,
PKTIO_MAX_QUEUES);
capa->set_op.op.promisc_mode = 1;
commit 4af5396e77ff170c80ec3720028493ed2c9f4826
Author: Matias Elo <matias.elo(a)nokia.com>
Date: Thu May 11 15:13:25 2017 +0300
linux-gen: dpdk: handle minimum rx burst size
Both ixgbe and i40e dpdk drivers have a minimum supported rx burst size of
four. Additionally, update driver names to match dpdk v17.08.
Signed-off-by: Matias Elo <matias.elo(a)nokia.com>
Reviewed-by: Bill Fischofer <bill.fischofer(a)linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov(a)linaro.org>
diff --git a/platform/linux-generic/include/odp_packet_dpdk.h b/platform/linux-generic/include/odp_packet_dpdk.h
index 5d80d84a..d4c4f02c 100644
--- a/platform/linux-generic/include/odp_packet_dpdk.h
+++ b/platform/linux-generic/include/odp_packet_dpdk.h
@@ -30,12 +30,13 @@ ODP_STATIC_ASSERT((DPDK_NB_MBUF % DPDK_MEMPOOL_CACHE_SIZE == 0) &&
, "DPDK mempool cache size failure");
#endif
-#define DPDK_IXGBE_MIN_RX_BURST 4
+/* Minimum RX burst size */
+#define DPDK_MIN_RX_BURST 4
/** Cache for storing packets */
struct pkt_cache_t {
/** array for storing extra RX packets */
- struct rte_mbuf *pkt[DPDK_IXGBE_MIN_RX_BURST];
+ struct rte_mbuf *pkt[DPDK_MIN_RX_BURST];
unsigned idx; /**< head of cache */
unsigned count; /**< packets in cache */
};
diff --git a/platform/linux-generic/pktio/dpdk.c b/platform/linux-generic/pktio/dpdk.c
index de9295c4..3684ca5c 100644
--- a/platform/linux-generic/pktio/dpdk.c
+++ b/platform/linux-generic/pktio/dpdk.c
@@ -45,6 +45,10 @@ ODP_STATIC_ASSERT(PKT_EXTRA_LEN >= sizeof(struct rte_mbuf),
"DPDK rte_mbuf won't fit in odp_packet_hdr_t.extra!");
#endif
+/* DPDK poll mode drivers requiring minimum RX burst size DPDK_MIN_RX_BURST */
+#define IXGBE_DRV_NAME "net_ixgbe"
+#define I40E_DRV_NAME "net_i40e"
+
static int disable_pktio; /** !0 this pktio disabled, 0 enabled */
/* Has dpdk_pktio_init() been called */
@@ -1249,10 +1253,16 @@ static int dpdk_open(odp_pktio_t id ODP_UNUSED,
pkt_dpdk->vdev_sysc_promisc = 1;
rte_eth_promiscuous_disable(pkt_dpdk->port_id);
- if (!strcmp(dev_info.driver_name, "rte_ixgbe_pmd"))
- pkt_dpdk->min_rx_burst = DPDK_IXGBE_MIN_RX_BURST;
+ /* Drivers requiring minimum burst size. Supports also *_vf versions
+ * of the drivers. */
+ if (!strncmp(dev_info.driver_name, IXGBE_DRV_NAME,
+ strlen(IXGBE_DRV_NAME)) ||
+ !strncmp(dev_info.driver_name, I40E_DRV_NAME,
+ strlen(I40E_DRV_NAME)))
+ pkt_dpdk->min_rx_burst = DPDK_MIN_RX_BURST;
else
pkt_dpdk->min_rx_burst = 0;
+
if (ODP_DPDK_ZERO_COPY) {
if (pool_entry->ext_desc != NULL)
pkt_pool = (struct rte_mempool *)pool_entry->ext_desc;
@@ -1372,10 +1382,10 @@ static int dpdk_recv(pktio_entry_t *pktio_entry, int index,
if (!pkt_dpdk->lockless_rx)
odp_ticketlock_lock(&pkt_dpdk->rx_lock[index]);
/**
- * ixgbe_pmd has a minimum supported RX burst size ('min_rx_burst'). If
- * 'num' < 'min_rx_burst', 'min_rx_burst' is used as rte_eth_rx_burst()
- * argument and the possibly received extra packets are cached for the
- * next dpdk_recv_queue() call to use.
+ * ixgbe and i40e drivers have a minimum supported RX burst size
+ * ('min_rx_burst'). If 'num' < 'min_rx_burst', 'min_rx_burst' is used
+ * as rte_eth_rx_burst() argument and the possibly received extra
+ * packets are cached for the next dpdk_recv_queue() call to use.
*
* Either use cached packets or receive new ones. Not both during the
* same call. */
-----------------------------------------------------------------------
Summary of changes:
platform/linux-generic/include/odp_packet_dpdk.h | 5 ++--
platform/linux-generic/pktio/dpdk.c | 33 ++++++++++++++++++------
2 files changed, 28 insertions(+), 10 deletions(-)
hooks/post-receive
--
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "".
The branch, 2.0 has been updated
via 6cd43041e55bd73a02ca202f835e590b3ad5c354 (commit)
from 65419c16086369f86dea068e54ce492526fa4f41 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 6cd43041e55bd73a02ca202f835e590b3ad5c354
Author: Kevin Wang <kevin.wang(a)arm.com>
Date: Fri Nov 10 17:12:32 2017 +0800
linux-gen: buffer: remove burst metadata from odp_buffer_hdr_t
The total packet meta data size was 7 cache lines for Linux-generic.
For scalable scheduler, the burst metadata is not required. So just
remove it for scalable scheduler. After the changes, the size of
the meta data is 4 cache lines for Linux-generic.
Signed-off-by: Kevin Wang <kevin.wang(a)arm.com>
Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli(a)arm.com>
Reviewed-by: Brian Brooks <brian.brooks(a)arm.com>
Reviewed-by: Yi He <yi.he(a)linaro.org>
diff --git a/platform/linux-generic/Makefile.am b/platform/linux-generic/Makefile.am
index 3c505495..56ef03c6 100644
--- a/platform/linux-generic/Makefile.am
+++ b/platform/linux-generic/Makefile.am
@@ -261,16 +261,9 @@ __LIB__libodp_linux_la_SOURCES = \
odp_pkt_queue.c \
odp_queue_if.c \
queue/subsystem.c \
- queue/generic.c \
- queue/scalable.c \
odp_rwlock.c \
odp_rwlock_recursive.c \
odp_schedule_if.c \
- schedule/generic.c \
- schedule/iquery.c \
- schedule/scalable.c \
- schedule/scalable_ordered.c \
- schedule/sp.c \
schedule/subsystem.c \
odp_shared_memory.c \
odp_sorted_list.c \
@@ -359,19 +352,29 @@ endif
pool/generic.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
buffer/generic.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
if ODP_SCHEDULE_SCALABLE
+__LIB__libodp_linux_la_SOURCES += schedule/scalable.c \
+ schedule/scalable_ordered.c
schedule/scalable.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
else
-schedule/generic.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
-endif
if ODP_SCHEDULE_SP
+__LIB__libodp_linux_la_SOURCES += schedule/sp.c
schedule/sp.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
-endif
+else
if ODP_SCHEDULE_IQUERY
+__LIB__libodp_linux_la_SOURCES += schedule/iquery.c
schedule/iquery.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
+else
+__LIB__libodp_linux_la_SOURCES += schedule/generic.c
+schedule/generic.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
+endif
endif
+endif
+
if ODP_SCHEDULE_SCALABLE
+__LIB__libodp_linux_la_SOURCES += queue/scalable.c
queue/scalable.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
else
+__LIB__libodp_linux_la_SOURCES += queue/generic.c
queue/generic.lo: AM_CFLAGS += -DIM_ACTIVE_MODULE
endif
diff --git a/platform/linux-generic/include/odp_buffer_internal.h b/platform/linux-generic/include/odp_buffer_internal.h
index 86f6208f..514e2479 100644
--- a/platform/linux-generic/include/odp_buffer_internal.h
+++ b/platform/linux-generic/include/odp_buffer_internal.h
@@ -73,6 +73,7 @@ struct odp_buffer_hdr_t {
/* Segments */
seg_entry_t seg[CONFIG_PACKET_SEGS_PER_HDR];
+#ifndef ODP_SCHEDULE_SCALABLE
/* Burst counts */
uint8_t burst_num;
uint8_t burst_first;
@@ -82,7 +83,7 @@ struct odp_buffer_hdr_t {
/* Burst table */
struct odp_buffer_hdr_t *burst[BUFFER_BURST_SIZE];
-
+#endif
/* --- Mostly read only data --- */
/* User context pointer or u64 */
-----------------------------------------------------------------------
Summary of changes:
platform/linux-generic/Makefile.am | 23 ++++++++++++----------
.../linux-generic/include/odp_buffer_internal.h | 3 ++-
2 files changed, 15 insertions(+), 11 deletions(-)
hooks/post-receive
--