of
TCP segments that it sends to OvS.
This patch is not intended for upstreaming, but rather was produced
in response to requests for an updated version of the initial TSO RFC
patch posted here:
https://mail.openvswitch.org/pipermail/ovs-dev/2016-June/316414.html
Signed-off-by: Mark Kavanagh
of a mempool can fail (as per rte_mempool_create),
including:
- insufficient memory available
- mempool already exists
- other memory allocation error
Update the error log to reflect this fact, and return rte_errno
in the event of error, instead of ENOMEM.
Signed-off-by: Mark Kavanagh
Tested-by: Sugesh Chandran <sugesh.chand...@intel.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
v4->v3:
- rebase to HEAD of master
- add 'acked-by' tags for S. Chandran, D. Ball
- add 'tested-by' tag for S. Chandran
v3->v2:
pport for jumbo frames")
Reported-by: Aaron Conole <acon...@redhat.com>
Reported-by: Vipin Varghese <vipin.vargh...@intel.com>
Reviewed-by: Aaron Conole <acon...@redhat.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
v3->v2:
- enable scatte
Added missing 'signed-off-by' tag.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
This RFC introduces an approach for implementing jumbo frame support for
OvS-DPDK with multi-segment mbufs.
== Overview ==
Currently, jumbo frame support for OvS-DPDK is implemented by increasing
the size of mbufs within a mempool, such that each mbuf within the pool is
large enough to contain
This RFC introduces an approach for implementing jumbo frame support for
OvS-DPDK with multi-segment mbufs.
== Overview ==
Currently, jumbo frame support for OvS-DPDK is implemented by increasing
the size of mbufs within a mempool, such that each mbuf within the pool is
large enough to contain
is termed a segment, hence
the name.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
V2->V1: add missing 'signed-off-by' tag; no code changes
lib/dpdk.c | 6 ++
lib/netdev-dpdk.c| 54 ++--
lib/netdev-dpdk.
is termed a segment, hence
the name.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/dpdk.c | 6 ++
lib/netdev-dpdk.c| 54 ++--
lib/netdev-dpdk.h| 1 +
vswitchd/vswitch.xml | 19 ++
4
pport for jumbo frames")
Reported-by: Vipin Varghese <vipin.vargh...@intel.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/netdev-dpdk.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index 8
pport for jumbo frames")
Reported-by: Aaron Conole <acon...@redhat.com>
Reported-by: Vipin Varghese <vipin.vargh...@intel.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
V2->v1:
- add 'reported-by' tag for Aaron Conole
- change VLOG_INFO to VL
This RFC introduces an approach for implementing jumbo frame support for
OvS-DPDK with multi-segment mbufs.
== Overview ==
Currently, jumbo frame support for OvS-DPDK is implemented by increasing
the size of mbufs within a mempool, such that each mbuf within the pool is
large enough to contain
Currently, jumbo frame support for OvS-DPDK is implemented by increasing
the size of mbufs within a mempool, such that each mbuf within the pool
is large enough to contain an entire jumbo frame of a user-defined size.
Typically, for each user-defined MTU 'requested_mtu', a new mempool is
created,
This RFC introduces an approach for implementing jumbo frame support for
OvS-DPDK with multi-segment mbufs.
== Overview ==
Currently, jumbo frame support for OvS-DPDK is implemented by increasing
the size of mbufs within a mempool, such that each mbuf within the pool is
large enough to contain
Currently, jumbo frame support for OvS-DPDK is implemented by increasing
the size of mbufs within a mempool, such that each mbuf within the pool
is large enough to contain an entire jumbo frame of a user-defined size.
Typically, for each user-defined MTU 'requested_mtu', a new mempool is
created,
netdev_dpdk_detach() declares a 'port_id' variable, of type uint8_t.
This variable should instead be of type dpdk_port_t.
Fixes: bb37956ac ("netdev-dpdk: Use uint8_t for port_id.")
CC: Ilya Maximets <i.maxim...@samsung.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.c
netdev_dpdk_detach() declares a 'port_id' variable, of type uint8_t.
This variable should instead be of type dpdk_port_t.
Fixes: bb37956ac ("netdev-dpdk: Use uint8_t for port_id.")
CC: Ilya Maximets <i.maxim...@samsung.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.c
IOMMU support. To that end, if IOMMU
support is required, this field should be set to true when setting
other global parameters on init (such as "dpdk-socket-mem", for
example).
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
v2->v1:
- rebase to HEAD of master
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Acked-by: Maxime Coquelin <max
advantage of
the vHost IOMMU feature introduced in DPDK v17.11.
---
v2->v1: - refactor vHost IOMMU enablement mechanism (use a global config
value, instead of the previous per-port approach).
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: vHost IOMMU supp
>v1: - refactor vHost IOMMU enablement mechanism (use a global config
value, instead of the previous per-port approach).
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: vHost IOMMU support
.travis/linux-build.sh | 2 +-
Documentation/faq/releas
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Acked-by: Maxime Coquelin <max
IOMMU support. To that end, if IOMMU
support is required, this field should be set to true when setting
other global parameters on init (such as "dpdk-socket-mem", for
example).
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
v2->v1:
- rebase to HEAD of master
PDK v17.11
[1] 3728b3b ("Merge branch 'dpdk_merge' of https://github.com/ist...;)
[2] https://patchwork.ozlabs.org/series/13829/mbox/
Mark Kavanagh (4):
netdev-dpdk: simplify mbuf sizing
lib/dp-packet: init specific mbuf fields to 0
lib/dp-packet: fix dp_packet_put_uninit for multi-seg mbufs
..@intel.com rebased]
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/dp-packet.c | 3 +++
lib/netdev-dpdk.c | 4
2 files changed, 7 insertions(+)
diff --git a/lib/dp-packet.c b/lib/dp-packet.c
index 443c225..5078211 100644
--- a/lib/dp-packet.c
+++ b/lib/dp-packet.
ovs-vsctl set Open_vSwitch . other_config:dpdk-multi-seg-mbufs=true
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
NEWS | 1 +
lib/dpdk.c | 7 +++
lib/netdev-dpdk.c| 43 ---
lib/netdev-dpdk.h| 1 +
From: Michael Qiu <qiud...@chinac.com>
When doing packet clone, if packet source is from DPDK driver,
multi-segment must be considered, and copy the segment's
data one by one.
Signed-off-by: Michael Qiu <qiud...@chinac.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.c
packet and copies the data to each segment.
Signed-off-by: Michael Qiu <qiud...@chinac.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/netdev-dpdk.c | 55 +++
1 file changed, 51 insertions(+), 4 deletions(-)
length of the entire
mbuf [2] by 'size'.
[1] In the case of a single-segment mbuf, this is the mbuf itself.
[2] This is stored in the first mbuf of an mbuf chain.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/dp-packet.c | 16
1 file changed, 16 inse
From: Michael Qiu <qiud...@chinac.com>
When a packet is from DPDK source, and it contains
multiple segments, data_len is not equal to the
packet size. This patch fixes this issue.
Co-authored-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Co-authored-by: Przemyslaw Lal <przemyslaw
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Acked-by: Maxime Coquelin <max
of
the vHost IOMMU feature introduced in DPDK v17.11.
Note that the previous RFC version of this patch (Acked x2) is here:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340731.html.
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: add support for vhost IOMMU
\
options:vhost-iommu-support=true
Note that support for this feature is only implemented for vhost
user client ports (since vhost user ports are considered deprecated).
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Acked-by: Maxime Coquelin <maxime.coque...@r
\
options:vhost-iommu-support=true
Note that support for this feature is only implemented for vhost
user client ports (since vhost user ports are considered deprecated).
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
Documentation/topics/dpdk/vhost-user.rs
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
.travis/linux-bu
host IOMMU feature.
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: add support for vhost IOMMU feature
.travis/linux-build.sh | 2 +-
Documentation/faq/releases.rst | 1 +
Documentation/intro/install/dpdk.rst | 10 -
Documentation/t
m particularly interested in comments on patch 2, which adds support
for the vhost IOMMU feature.
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: add support for vhost IOMMU feature
.travis/linux-build.sh | 2 +-
Documentation/faq/releases.rst |
(since vhost user ports are considered deprecated).
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
Documentation/topics/dpdk/vhost-user.rst | 19 +++
NEWS | 1 +
lib/netdev-dpdk.c
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
.travis/linux-bu
Adapted from an idea by Michael Qiu <qiud...@chinac.com>:
https://patchwork.ozlabs.org/patch/777570/
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/dp-packet.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
i
From: Michael Qiu <qiud...@chinac.com>
When a packet is from DPDK source, and it contains
multiple segments, data_len is not equal to the
packet size. This patch fixes this issue.
Co-authored-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Co-authored-by: Przemyslaw Lal <przemyslaw
From: Michael Qiu <qiud...@chinac.com>
Currently, when doing packet copy, lots of DPDK mbuf's info
will be missed, like packet type, ol_flags, etc.
Those information is very important for DPDK to do
packets processing.
Co-authored-by: Mark Kavanagh <mark.b.kavan...@intel.com>
[
.11
[1] 159cc1f ("datapath-windows: Correct endianness for deleting zone.")
[2] https://mail.openvswitch.org/pipermail/ovs-dev/2017-December/341786.html
---
v4: - restructure patchset
- account for 128B ARM cacheline when sizing mbufs
Mark Kavanagh (5):
netdev-dpdk: fix mbuf sizin
From: Michael Qiu <qiud...@chinac.com>
When doing packet clone, if packet source is from DPDK driver,
multi-segment must be considered, and copy the segment's
data one by one.
Co-authored-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Signed-off-by: Michael Qiu <qiud...@chinac.
).
Currently, in dp_packet_reset_packet(), that order is reversed.
Swap the order of same to resolve.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/dp-packet.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index b
-dpdk: round up mbuf_size to cache_line_size")
CC: Santosh Shukla <santosh.shu...@caviumnetworks.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/netdev-dpdk.c | 46 ++
1 file changed, 30 insertions(+), 16 deletions(-)
diff
ovs-vsctl set Open_vSwitch . other_config:dpdk-multi-seg-mbufs=true
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
NEWS | 1 +
lib/dpdk.c | 7 +++
lib/netdev-dpdk.c| 52 +---
lib/netdev-dpdk.
packet and copies the data to each segment.
Co-authored-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Signed-off-by: Michael Qiu <qiud...@chinac.com>
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
lib/netdev-dpdk.c | 56 +++
IOMMU enablement mechanism (use a global config
value, instead of the previous per-port approach).
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: vHost IOMMU support
.travis/linux-build.sh | 2 +-
Documentation/faq/releases.rst | 1
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
- document DPDK v17.11 virtio driver bug
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
setting other global parameters on init
(such as "dpdk-socket-mem", for example). Changing the value at
runtime is not supported, and requires restarting the vswitch daemon.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
Documentation/topics/dpdk/vho
; disregard.
v2: - refactor vHost IOMMU enablement mechanism (use a global config
value, instead of the previous per-port approach).
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11 upgrade
netdev-dpdk: vHost IOMMU support
.travis/linux-build.sh | 2 +-
Documentation/faq
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
Acked-by: Maxime Coquelin <max
setting other global parameters on init
(such as "dpdk-socket-mem", for example). Changing the value at
runtime is not supported, and requires restarting the vswitch daemon.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
v4: - rebase to HEAD of master
- clarify th
setting other global parameters on init
(such as "dpdk-socket-mem", for example). Changing the value at
runtime is not supported, and requires restarting the vswitch daemon.
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
Documentation/topics/dpdk/vho
This commit adds support for DPDK v17.11:
- minor updates to accomodate DPDK API changes
- update references to DPDK version in Documentation
- update DPDK version in travis' linux-build script
- document DPDK v17.11 virtio driver bug
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
.
- remove mention of vhost IOMMU mode from
netdev_dpdk_vhost_client_configure() log.
v3: - erroneous; disregard.
v2: - refactor vHost IOMMU enablement mechanism (use a global config
value, instead of the previous per-port approach).
Mark Kavanagh (2):
netdev-dpdk: DPDK v17.11
Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
---
AUTHORS.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/AUTHORS.rst b/AUTHORS.rst
index 3d3a081..cf62418 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -217,7 +217,7 @@ Marcin Mirecki
58 matches
Mail list logo
