On Tue, Aug 21, 2018 at 03:49:25PM -0700, William Tu wrote:
> On Tue, Aug 21, 2018 at 9:22 AM Ben Pfaff <[email protected]> wrote:
> >
> > This test only worked if each OpenFlow port was assigned a particular
> > datapath port number: p1 to port 3, p2 to port 2, p3 and p4 to port 1.
> > This happened consistently on little-endian architectures because of the
> > use of a particular hash function, but on big-endian architectures it
> > failed because the hash function was different.
> >
> > This commit fixes the problem by adding the non-dummy ports separately.
> > (Dummy ports try to take the datapath port number corresponding to their
> > name, when it is available.)  This does result in swapping a couple of
> > datapaths port numbers, so that p1 has port 1, p2 has port 2, and the
> > erspan ports have port 3, hence the size of the patch.
> >
> > Reported-by: James Page <[email protected]>
> > Reported-at: 
> > https://mail.openvswitch.org/pipermail/ovs-dev/2018-August/351382.html
> > Signed-off-by: Ben Pfaff <[email protected]>
> 
> Thanks for the fix!
> Acked-by: William Tu <[email protected]>

Thanks, applied and backported.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to