From: [email protected]
Date: Sat, 19 Oct 2019 16:08:40 +0800

> @@ -432,13 +432,9 @@ int ovs_flow_tbl_flush(struct flow_table *flow_table)
>  static u32 flow_hash(const struct sw_flow_key *key,
>                    const struct sw_flow_key_range *range)
>  {
> -     int key_start = range->start;
> -     int key_end = range->end;
> -     const u32 *hash_key = (const u32 *)((const u8 *)key + key_start);
> -     int hash_u32s = (key_end - key_start) >> 2;
> -
> +     const u32 *hash_key = (const u32 *)((const u8 *)key + range->start);
>       /* Make sure number of hash bytes are multiple of u32. */
> -     BUILD_BUG_ON(sizeof(long) % sizeof(u32));
> +     int hash_u32s = range_n_bytes(range) >> 2;

Please place an empty line between the last local variable declaration and
the start of comments and code in the body of the function.

Thank you.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to