On Fri, Aug 17, 2012 at 08:42:53PM +1200, Joe Stringer wrote:
> On 16 August 2012 06:43, Ben Pfaff wrote:
> > On Wed, Aug 15, 2012 at 12:28:47AM +1200, Joe Stringer wrote:
> >> In OpenFlow 1.1, we add support for OFPIT_WRITE_METADATA. This allows us to
> >> write to the metadata field. Internally
On 16 August 2012 06:43, Ben Pfaff wrote:
> On Wed, Aug 15, 2012 at 12:28:47AM +1200, Joe Stringer wrote:
>> In OpenFlow 1.1, we add support for OFPIT_WRITE_METADATA. This allows us to
>> write to the metadata field. Internally it is represented using
>> ofpact_metadata.
>>
>> We introduce NXAST_
On Wed, Aug 15, 2012 at 12:28:47AM +1200, Joe Stringer wrote:
> In OpenFlow 1.1, we add support for OFPIT_WRITE_METADATA. This allows us to
> write to the metadata field. Internally it is represented using
> ofpact_metadata.
>
> We introduce NXAST_WRITE_METADATA to handle writing to the metadata
Basically it looks good. One minor comment inline.
On Wed, Aug 15, 2012 at 12:28:47AM +1200, Joe Stringer wrote:
> diff --git a/lib/ofp-actions.h b/lib/ofp-actions.h
> index 272b8e8..a16c04a 100644
> --- a/lib/ofp-actions.h
> +++ b/lib/ofp-actions.h
...
> @@ -311,6 +311,15 @@ struct ofpact_fin_tim
In OpenFlow 1.1, we add support for OFPIT_WRITE_METADATA. This allows us to
write to the metadata field. Internally it is represented using ofpact_metadata.
We introduce NXAST_WRITE_METADATA to handle writing to the metadata field in
OpenFlow 1.0+. This structure reflects OFPIT_WRITE_METADATA.
Wh
