[ovs-dev] [multipath 8/7] flow: Increase number of registers to 4.
Justin Pettit
jpettit at nicira.com
Fri Dec 17 22:29:06 UTC 2010
Looks good.
--Justin
On Dec 16, 2010, at 4:01 PM, Ben Pfaff wrote:
> Requested-by: Teemu Koponen <koponen at nicira.com>
> ---
> Here's another one for this series that I forgot about until now.
>
> diff --git a/lib/flow.h b/lib/flow.h
> index 03ad32c..ee460e3 100644
> --- a/lib/flow.h
> +++ b/lib/flow.h
> @@ -32,7 +32,7 @@ struct flow_wildcards;
> struct ofp_match;
> struct ofpbuf;
>
> -#define FLOW_N_REGS 3
> +#define FLOW_N_REGS 4
> BUILD_ASSERT_DECL(FLOW_N_REGS <= NXM_NX_MAX_REGS);
>
> struct flow {
> @@ -49,13 +49,12 @@ struct flow {
> uint8_t dl_dst[6]; /* Ethernet destination address. */
> uint8_t nw_proto; /* IP protocol or low 8 bits of ARP opcode. */
> uint8_t nw_tos; /* IP ToS (DSCP field, 6 bits). */
> - uint32_t reserved; /* Reserved for 64-bit packing. */
> };
>
> /* Assert that there are FLOW_SIG_SIZE bytes of significant data in "struct
> * flow", followed by FLOW_PAD_SIZE bytes of padding. */
> #define FLOW_SIG_SIZE (40 + FLOW_N_REGS * 4)
> -#define FLOW_PAD_SIZE 4
> +#define FLOW_PAD_SIZE 0
> BUILD_ASSERT_DECL(offsetof(struct flow, nw_tos) == FLOW_SIG_SIZE - 1);
> BUILD_ASSERT_DECL(sizeof(((struct flow *)0)->nw_tos) == 1);
> BUILD_ASSERT_DECL(sizeof(struct flow) == FLOW_SIG_SIZE + FLOW_PAD_SIZE);
> --
> 1.7.1
>
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
More information about the dev
mailing list