[ovs-dev] Q on FLOW_SIG_SIZE and hashing

[Ravi.Kerur at telekom.com]

Yes I do. Attached complete diffs. The debugging information I mentioned earlier are with these diffs as well. I shifted to just flow struct diffs after I went through FLOW_WC_SEQ changes I had and thought it might not have an impact, as most of the checks are in ".c" and with build_assert_decl and compilation should have failed if I had missed something. Anyways let me know your inputs. 

Thanks
Ravi

-----Original Message-----
From: Ethan Jackson [mailto:ethan at nicira.com] 
Sent: Tuesday, February 21, 2012 2:31 PM
To: Kerur, Ravi
Cc: dev at openvswitch.org
Subject: Re: [ovs-dev] Q on FLOW_SIG_SIZE and hashing

> I have attached diffs which includes adding a member to struct flow and adjusting FLOW_SIG_SIZE accordingly. This is experimental so I haven't bothered to change FLOW_WC_SEQ...

Oh I'm sorry for the confusion, I thought you had a more involved
patch which makes the necessary changes demanded by FLOW_WC_SEQ.  The
code really does require those changes to work, simply adding the
field to the structure is insufficient.  The behavior you're seeing is
what I'd expect to see without the FLOW_WC_SEQ changes.  There may be
other changes necessary as well.

Ethan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ethan_diffs
Type: application/octet-stream
Size: 100775 bytes
Desc: ethan_diffs
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20120221/976e929c/attachment-0004.obj>