[ovs-dev] [Single DP 04/15] Use ODP ports in dpif layer and below.

Ben Pfaff blp at nicira.com
Mon Oct 22 22:24:55 UTC 2012


On Thu, Oct 18, 2012 at 12:51:49PM -0700, Justin Pettit wrote:
> @@ -89,7 +91,7 @@ struct flow {
>      uint8_t arp_tha[6];         /* ARP/ND target hardware address. */
>      uint8_t nw_ttl;             /* IP TTL/Hop Limit. */
>      uint8_t nw_frag;            /* FLOW_FRAG_* flags. */
> -    uint8_t zeros[2];           /* Must be zero. */
> +    uint8_t zeros[0];           /* Must be zero. */

I'd get rid of this zeros[] member entirely.  It looks weird and it
causes various "sparse" warnings:

    ../lib/flow.c:595:11: warning: memset with byte count of 0
    ../tests/test-bundle.c:139:15: warning: memset with byte count of 0
    ../tests/test-multipath.c:63:19: warning: memset with byte count of 0

Thanks,

Ben.



More information about the dev mailing list