On Wed, Apr 11, 2018 at 12:15:28PM +0000, Stokes, Ian wrote: > > Seems the doc just wasn't updated when they settled for a way of > > addressing this issue. > > Agreed, I've validated with a Mellanox Connect X3 pro card, 6 bytes > was expected. > > Thanks for this Marcelo, I see Timothy Redaelli (from Red Hat also) > has submitted a similar patch. > > https://mail.openvswitch.org/pipermail/ovs-dev/2018-April/345885.html
Oh, wasn't aware of that. Sorry Timothy, and thanks. > > There was a request to fix the commit message for that patch however. > > @Timothy: As this patch has the correct commit message I can merge > this to dpdk_merge if there are no objections rather than you > spinning a v2 for yours? > > Thanks > Ian > > > > > Thanks, > > Marcelo > > > > > > > > Regards, > > > Billy. > > > > > > > -----Original Message----- > > > > From: [email protected] [mailto:ovs-dev- > > > > [email protected]] On Behalf Of Marcelo Ricardo Leitner > > > > Sent: Monday, April 9, 2018 6:21 PM > > > > To: [email protected] > > > > Cc: [email protected]; [email protected] > > > > Subject: [ovs-dev] [PATCH] 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 <[email protected]> > > > > --- > > > > Documentation/howto/dpdk.rst | 4 ++-- > > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/Documentation/howto/dpdk.rst > > > > b/Documentation/howto/dpdk.rst index > > > > 79b626c76d0dd45381bd75ab867b7815ca941208..69e692f40d500cf65d59d1979 > > > > e07afa6f99cf903 100644 > > > > --- a/Documentation/howto/dpdk.rst > > > > +++ b/Documentation/howto/dpdk.rst > > > > @@ -53,9 +53,9 @@ with multiple ports. Using a PCI device like above > > > > won't work. Instead, below usage is suggested:: > > > > > > > > $ ovs-vsctl add-port br0 dpdk-p0 -- set Interface dpdk-p0 > > type=dpdk \ > > > > - options:dpdk-devargs="class=eth,mac=00:11:22:33:44:55:01" > > > > + options:dpdk-devargs="class=eth,mac=00:11:22:33:44:55" > > > > $ ovs-vsctl add-port br0 dpdk-p1 -- set Interface dpdk-p1 > > type=dpdk \ > > > > - options:dpdk-devargs="class=eth,mac=00:11:22:33:44:55:02" > > > > + options:dpdk-devargs="class=eth,mac=00:11:22:33:44:56" > > > > > > > > Note: such syntax won't support hotplug. The hotplug is supposed to > > > > work with future DPDK release, v18.05. > > > > -- > > > > 2.14.3 > > > > > > > > _______________________________________________ > > > > dev mailing list > > > > [email protected] > > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > _______________________________________________ > > dev mailing list > > [email protected] > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
