[ovs-build] Errored: istokes/ovs#713 (mseg - 66a24c1)

Travis CI builds at travis-ci.org
Mon Oct 22 20:08:49 UTC 2018


Build Update for istokes/ovs
-------------------------------------

Build: #713
Status: Errored

Duration: 1 hr, 49 mins, and 51 secs
Commit: 66a24c1 (mseg)
Author: Tiago Lam
Message: dp-packet: Fix data_len handling multi-seg mbufs.

When a dp_packet is from a DPDK source, and it contains multi-segment
mbufs, the data_len is not equal to the packet size, pkt_len. Instead,
the data_len of each mbuf in the chain should be considered while
distributing the new (provided) size.

To account for the above dp_packet_set_size() has been changed so that,
in the multi-segment mbufs case, only the data_len on the last mbuf of
the chain and the total size of the packet, pkt_len, are changed. The
data_len on the intermediate mbufs preceeding the last mbuf is not
changed by dp_packet_set_size(). Furthermore, in some cases
dp_packet_set_size() may be used to set a smaller size than the current
packet size, thus effectively trimming the end of the packet. In the
multi-segment mbufs case this may lead to lingering mbufs that may need
freeing.

__dp_packet_set_data() now also updates an mbufs' data_len after setting
the data offset. This is so that both fields are always in sync for each
mbuf in a chain.

Co-authored-by: Michael Qiu <qiudayu at chinac.com>
Co-authored-by: Mark Kavanagh <mark.b.kavanagh at intel.com>
Co-authored-by: Przemyslaw Lal <przemyslawx.lal at intel.com>
Co-authored-by: Marcin Ksiadz <mksiadz at gmail.com>
Co-authored-by: Yuanhan Liu <yliu at fridaylinux.org>

Signed-off-by: Michael Qiu <qiudayu at chinac.com>
Signed-off-by: Mark Kavanagh <mark.b.kavanagh at intel.com>
Signed-off-by: Przemyslaw Lal <przemyslawx.lal at intel.com>
Signed-off-by: Marcin Ksiadz <mksiadz at gmail.com>
Signed-off-by: Yuanhan Liu <yliu at fridaylinux.org>
Signed-off-by: Tiago Lam <tiago.lam at intel.com>
Acked-by: Eelco Chaudron <echaudro at redhat.com>

View the changeset: https://github.com/istokes/ovs/compare/61306dd7fdf4...66a24c114698

View the full build log and details: https://travis-ci.org/istokes/ovs/builds/444620068?utm_medium=notification&utm_source=email

--

You can unsubscribe from build emails from the istokes/ovs repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=16644441&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20181022/d576fe8f/attachment.html>


More information about the build mailing list