Thanks.
On Tue, Jul 2, 2013 at 5:08 PM, Jesse Gross <je...@nicira.com> wrote: > 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
