[ovs-dev] [patch v6 02/10] flow: Enhance parse_ipv6_ext_hdrs.

Justin Pettit jpettit at ovn.org
Wed Jun 6 06:36:34 UTC 2018


> On Apr 8, 2018, at 7:53 PM, Darrell Ball <dlu998 at gmail.com> wrote:
> 
> bool
> parse_ipv6_ext_hdrs(const void **datap, size_t *sizep, uint8_t *nw_proto,
> -                    uint8_t *nw_frag)
> +                    uint8_t *nw_frag,
> +                    const struct ovs_16aligned_ip6_frag **frag_hdr)

I know I previously acked this patch, but I do think this function could use a comment describing it, since it is a public function, and the arguments and return type aren't immediately obvious.

Thanks,

--Justin




More information about the dev mailing list