[ovs-dev] [PATCH v2 1/3] userspace datapath: Add OVS_HASH_L4_SYMMETRIC dp_hash algorithm
Ben Pfaff
blp at ovn.org
Tue Apr 17 19:56:55 UTC 2018
On Mon, Apr 16, 2018 at 04:26:26PM +0200, Jan Scheurich wrote:
> This commit implements a new dp_hash algorithm OVS_HASH_L4_SYMMETRIC in
> the netdev datapath. It will be used as default hash algorithm for the
> dp_hash-based select groups in a subsequent commit to maintain
> compatibility with the symmetry property of the current default hash
> selection method.
>
> Signed-off-by: Jan Scheurich <jan.scheurich at ericsson.com>
> Signed-off-by: Nitin Katiyar <nitin.katiyar at ericsson.com>
> Co-authored-by: Nitin Katiyar <nitin.katiyar at ericsson.com>
"sparse" says:
../lib/flow.c:2128:13: error: restricted ovs_be32 degrades to integer
../lib/flow.c:2128:35: error: restricted ovs_be16 degrades to integer
I think that it is saying that htons should be htonl here:
if (flow->packet_type != htons(PT_ETH)) {
/* Cannot hash non-Ethernet flows */
return 0;
}
GCC says:
../lib/flow.c: In function 'flow_hash_symmetric_l2.part.20':
../lib/flow.c:2138:25: error: 'fields.<Ue630>.vlan_tci' is used uninitialized in this function [-Werror=uninitialized]
fields.vlan_tci ^= flow->vlans[i].tci & htons(VLAN_VID_MASK);
^~
I think that it is right.
Thanks,
Ben.
More information about the dev
mailing list