[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openvswitch-dev
Subject:    [ovs-dev] [PATCH] Use struct nicira_header nxh in struct nx_set_flow_format
From:       horms () verge ! net ! au (Simon Horman)
Date:       2012-02-28 2:54:16
Message-ID: 20120228025416.GA28596 () verge ! net ! au
[Download RAW message or body]

On Mon, Feb 27, 2012 at 06:37:19PM -0800, Ben Pfaff wrote:
> 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.

Sorry, I was confused.


[prev in list] [next in list] [prev in thread] [next in thread]