[ovs-git] [openvswitch/ovs] 2dd018: netdev-dpdk: fix MAC address in port addr example

GitHub noreply at github.com
Mon Apr 30 15:41:30 UTC 2018


  Branch: refs/heads/branch-2.9
  Home:   https://github.com/openvswitch/ovs
  Commit: 2dd018b1685f8f62f2946d415005ca329656e618
      https://github.com/openvswitch/ovs/commit/2dd018b1685f8f62f2946d415005ca329656e618
  Author: Marcelo Ricardo Leitner <marcelo.leitner at gmail.com>
  Date:   2018-04-19 (Thu, 19 Apr 2018)

  Changed paths:
    M Documentation/howto/dpdk.rst

  Log Message:
  -----------
  netdev-dpdk: fix MAC address in port addr example

The MAC address is always 6-bytes long, never 7. The extra :01 and :02
doesn't belong in there as it doesn't mean selecting one port or
another.

Instead, use an incrementing MAC address, which is what usually happens
on such cards.

See-also: http://www.dpdk.org/ml/archives/dev/2018-April/094976.html
Fixes: 5e7588186839 ("netdev-dpdk: fix port addition for ports sharing same PCI id")
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner at gmail.com>
Signed-off-by: Ian Stokes <ian.stokes at intel.com>


  Commit: feb12dd41996cf921de93e1ece95ad05e524dfdf
      https://github.com/openvswitch/ovs/commit/feb12dd41996cf921de93e1ece95ad05e524dfdf
  Author: Tiago Lam <tiago.lam at intel.com>
  Date:   2018-04-19 (Thu, 19 Apr 2018)

  Changed paths:
    M Documentation/topics/dpdk/vhost-user.rst

  Log Message:
  -----------
  dpdk docs: Drop qemu-kvm for qemu-system-x86_64.

When explaining on how to add vhost-user ports to a guest, using
libvirt, point to the qemu-system-x86_64 binary by default, instead of
using qemu-kvm. The latter has been made obsolete and dropped from a
number of distributions (although it is still available on Fedora).

This has been verified on both a Fedora 27 image and a Ubuntu 16.04 LTS
image.

Signed-off-by: Tiago Lam <tiago.lam at intel.com>
Acked-by: Aaron Conole <aconole at redhat.com>
Signed-off-by: Ian Stokes <ian.stokes at intel.com>


  Commit: 66c9360fb9509211913441987d5b7b79aee38336
      https://github.com/openvswitch/ovs/commit/66c9360fb9509211913441987d5b7b79aee38336
  Author: Tiago Lam <tiago.lam at intel.com>
  Date:   2018-04-19 (Thu, 19 Apr 2018)

  Changed paths:
    M Documentation/topics/dpdk/vhost-user.rst

  Log Message:
  -----------
  dpdk docs: Drop file share in libvirt config.

When explaining on how to add vhost-user ports to a guest, using
libvirt, the following piece of configuration is used:
    <disk type='dir' device='disk'>
      <driver name='qemu' type='fat'/>
      <source dir='/usr/src/dpdk-stable-17.11.1'/>
      <target dev='vdb' bus='virtio'/>
      <readonly/>
    </disk>

This is used to facilitate sharing of a DPDK directory between the host
and the guest. However, for this to work selinux also needs to be
configured (or disabled).  Furthermore, if one is using Ubuntu, libvirtd
would need to be added to complain only in AppArmor. Instead, in [1] it
is advised to use wget to get the DPDK sources over the internet, which
avoids this differentiation. Thus, we drop this piece of configuration
here as well and keep the example configuration as simple as possible.

This has been verified on both a Fedora 27 image and a Ubuntu 16.04 LTS
image.

[1] http://docs.openvswitch.org/en/latest/topics/dpdk/vhost-user/#dpdk-in-the-guest

Signed-off-by: Tiago Lam <tiago.lam at intel.com>
Acked-by: Stephen Finucane <stephen at that.guru>
Signed-off-by: Ian Stokes <ian.stokes at intel.com>


  Commit: c77f6920deba6f22d7e9a3857408d996100029c2
      https://github.com/openvswitch/ovs/commit/c77f6920deba6f22d7e9a3857408d996100029c2
  Author: Kevin Traynor <ktraynor at redhat.com>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M lib/netdev-dpdk.c

  Log Message:
  -----------
  netdev-dpdk: Free mempool only when no in-use mbufs.

DPDK mempools are freed when they are no longer needed.
This can happen when a port is removed or a port's mtu
is reconfigured so that a new mempool is used.

It is possible that an mbuf is attempted to be returned
to a freed mempool from NIC Tx queues and this can lead
to a segfault.

In order to prevent this, only free mempools when they
are not needed and have no in-use mbufs. As this might
not be possible immediately, sweep the mempools anytime
a port tries to get a mempool.

Fixes: 8d38823bdf8b ("netdev-dpdk: fix memory leak")
Cc: mark.b.kavanagh81 at gmail.com
Cc: Ilya Maximets <i.maximets at samsung.com>
Reported-by: Venkatesan Pradeep <venkatesan.pradeep at ericsson.com>
Signed-off-by: Kevin Traynor <ktraynor at redhat.com>
Signed-off-by: Ian Stokes <ian.stokes at intel.com>


  Commit: a7fb0a4ee109e7ca4a56b506b52ce35ef0025d00
      https://github.com/openvswitch/ovs/commit/a7fb0a4ee109e7ca4a56b506b52ce35ef0025d00
  Author: Kevin Traynor <ktraynor at redhat.com>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M lib/netdev-dpdk.c

  Log Message:
  -----------
  netdev-dpdk: Add mempool reuse/free debug.

There is debug when a new mempool is created, but not
when it is reused or freed. Add these as it is very
difficult to debug mempool issues from logs without
them.

Signed-off-by: Kevin Traynor <ktraynor at redhat.com>
Signed-off-by: Ian Stokes <ian.stokes at intel.com>


  Commit: acd8f9d266c709faf594a3acf82422d3bf2defef
      https://github.com/openvswitch/ovs/commit/acd8f9d266c709faf594a3acf82422d3bf2defef
  Author: Ian Stokes <ian.stokes at intel.com>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M Documentation/howto/sflow.rst

  Log Message:
  -----------
  docs: Fix sflow documentation url and markup.

The link url link for the blog in sflow documentation causes make
check-docs to fail with a broken link warning. Fix this by correcting
the url address. Also use correct markup for note regarding the
configuration of sflow.

CC: Stephen Finucane <stephen at that.guru>
Fixes: 198c5d3d0 ("doc: Add sFlow cookbook from website")
Signed-off-by: Ian Stokes <ian.stokes at intel.com>
Acked-by: Stephen Finucane <stephen at that.guru>


  Commit: 376c64d1ee952a3aca113b6609c908f8df7710f1
      https://github.com/openvswitch/ovs/commit/376c64d1ee952a3aca113b6609c908f8df7710f1
  Author: Ian Stokes <ian.stokes at intel.com>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M Documentation/internals/contributing/documentation-style.rst

  Log Message:
  -----------
  docs: Fix style guide url in DocumentationStyle.rst.

The link used for IBM Style Guide is no longer valid. As there is no
longer a valid link via redbooks remove the url to avoid make
check-docs failing.

Cc: Stephen Finucane <stephen at that.guru>
Fixes: 26ea2d409 ("docs: Add writing guide")
Signed-off-by: Ian Stokes <ian.stokes at intel.com>
Acked-by: Stephen Finucane <stephen at that.guru>


  Commit: bc6d54cc475b8276027badcc2694f448850d8f49
      https://github.com/openvswitch/ovs/commit/bc6d54cc475b8276027badcc2694f448850d8f49
  Author: Ian Stokes <ian.stokes at intel.com>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M Documentation/internals/contributing/documentation-style.rst
    M Documentation/intro/install/documentation.rst

  Log Message:
  -----------
  docs: Fix sphinx urls.

Update dead url links for sphinx documentation to avoid
make check-docs failing.

Cc: Stephen Finucane <stephen at that.guru>
Fixes: 26ea2d409 ("docs: Add writing guide")
Fixes: 73c76b447 ("doc: Add info on building documentation")
Signed-off-by: Ian Stokes <ian.stokes at intel.com>
Acked-by: Stephen Finucane <stephen at that.guru>


  Commit: 11e972ee2e7aa822bc289a3ae0aec944b67bfa58
      https://github.com/openvswitch/ovs/commit/11e972ee2e7aa822bc289a3ae0aec944b67bfa58
  Author: Ian Stokes <ian.stokes at intel.com>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M Documentation/ref/index.rst

  Log Message:
  -----------
  docs: Fix urls in index.rst.

This patch prepends 'www' to openvswitch urls in index.rst. Without this
make check-docs fails when verifying url liveness. Also remove url
referencing ovsdb-server(5) as these are no longer accessible.

Cc: Stephen Finucane <stephen at that.guru>
Fixes: 4f6ec357c ("doc: Populate 'ref' section")
Signed-off-by: Ian Stokes <ian.stokes at intel.com>
Acked-by: Stephen Finucane <stephen at that.guru>


  Commit: 1b4482c7f6c1b7cd2dba56a2976aea2f915adc46
      https://github.com/openvswitch/ovs/commit/1b4482c7f6c1b7cd2dba56a2976aea2f915adc46
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-04-30 (Mon, 30 Apr 2018)

  Changed paths:
    M Documentation/howto/dpdk.rst
    M Documentation/howto/sflow.rst
    M Documentation/internals/contributing/documentation-style.rst
    M Documentation/intro/install/documentation.rst
    M Documentation/ref/index.rst
    M Documentation/topics/dpdk/vhost-user.rst
    M lib/netdev-dpdk.c

  Log Message:
  -----------
  Merge branch 'dpdk_merge_2_9' of https://github.com/istokes/ovs into HEAD


Compare: https://github.com/openvswitch/ovs/compare/2a9f1d7531b7...1b4482c7f6c1


More information about the git mailing list