[ovs-dev] [PATCH] datapath: Backport "openvswitch: Fix mask generation for nested attributes."
Pravin Shelar
pshelar at nicira.com
Thu Sep 17 21:42:33 UTC 2015
On Tue, Sep 15, 2015 at 9:58 PM, Jesse Gross <jesse at nicira.com> wrote:
> Upstream commit:
> openvswitch: Fix mask generation for nested attributes.
>
> Masks were added to OVS flows in a way that was backwards compatible
> with userspace programs that did not generate masks. As a result, it is
> possible that we may receive flows that do not have a mask and we need
> to synthesize one.
>
> Generating a mask requires iterating over attributes and descending into
> nested attributes. For each level we need to know the size to generate the
> correct mask. We do this with a linked table of attribute types.
>
> Although the logic to handle these nested attributes was there in concept,
> there are a number of bugs in practice. Examples include incomplete links
> between tables, variable length attributes being treated as nested and
> missing sanity checks.
>
> Signed-off-by: Jesse Gross <jesse at nicira.com>
> Acked-by: Pravin B Shelar <pshelar at nicira.com>
> Signed-off-by: David S. Miller <davem at davemloft.net>
>
> Upstream: 982b5270 ("openvswitch: Fix mask generation for nested attributes.")
> Signed-off-by: Jesse Gross <jesse at nicira.com>
LGTM
Acked-by: Pravin B Shelar <pshelar at nicira.com>
More information about the dev
mailing list