No, I can fix it. I just wanted to make sure.
On Tue, Jul 2, 2013 at 4:31 PM, Andy Zhou <az...@nicira.com> wrote: > Sorry, I misread the jhash2 prototype. Do you need v2 patch? > > > On Tue, Jul 2, 2013 at 4:05 PM, Jesse Gross <je...@nicira.com> wrote: >> >> On Tue, Jul 2, 2013 at 3:58 PM, Andy Zhou <az...@nicira.com> wrote: >> > diff --git a/datapath/tunnel.c b/datapath/tunnel.c >> > index 9102786..18c3622 100644 >> > --- a/datapath/tunnel.c >> > +++ b/datapath/tunnel.c >> > @@ -186,7 +186,9 @@ u16 ovs_tnl_get_src_port(struct sk_buff *skb) >> > int low; >> > int high; >> > unsigned int range; >> > - u32 hash = OVS_CB(skb)->flow->hash; >> > + struct sw_flow_key *pkt_key = OVS_CB(skb)->pkt_key; >> > + u32 hash = jhash2((const u32 *)pkt_key, >> > + 0, sizeof(*pkt_key) / sizeof(u32)); >> >> Unless I'm missing something, doesn't this swap the values for length >> and initval? > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
