[ovs-dev] [PATCH] Use struct nicira_header nxh in struct nx_set_flow_format

Ben Pfaff blp at nicira.com
Tue Feb 28 02:37:19 UTC 2012


On Tue, Feb 28, 2012 at 10:59:44AM +0900, Simon Horman wrote:
> On Mon, Feb 27, 2012 at 04:51:45PM -0800, Chris Wright wrote:
> > * Simon Horman (horms at verge.net.au) wrote:
> > > This makes struct nx_set_flow_format consistent with
> > > other similar structures.
> > > 
> > > Signed-off-by: Simon Horman <horms at verge.net.au>
> > 
> > Looks good to me.
> > 
> > Acked-by: Chris Wright <chrisw at sous-sol.org>
> > 
> > BTW, struct nx_flow_mod_table_id vendor/subtype would default to arch-Endian,
> > (which seems like an oversight otherwise could get the same treatment).
> 
> Hi Chris,
> 
> I took a look over the code and that seems like an oversight to me.

I already applied the previous version.



More information about the dev mailing list