netdev - Re: [PATCH net-next] openvswitch: avoid deferred execution of recirc actions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAOrHB_DE-iTQQawmbthXwngMar5Pm+RVE6PED_trkMXv3P2t2Q@mail.gmail.com>
Date:   Tue, 13 Sep 2016 13:52:32 -0700
From:   pravin shelar <pshelar@....org>
To:     Lance Richardson <lrichard@...hat.com>
Cc:     Linux Kernel Network Developers <netdev@...r.kernel.org>,
        ovs dev <dev@...nvswitch.org>, sramamur@...ux.vnet.ibm.com
Subject: Re: [PATCH net-next] openvswitch: avoid deferred execution of recirc actions

On Tue, Sep 13, 2016 at 7:08 AM, Lance Richardson <lrichard@...hat.com> wrote:
> The ovs kernel data path currently defers the execution of all
> recirc actions until stack utilization is at a minimum.
> This is too limiting for some packet forwarding scenarios due to
> the small size of the deferred action FIFO (10 entries). For
> example, broadcast traffic sent out more than 10 ports with
> recirculation results in packet drops when the deferred action
> FIFO becomes full, as reported here:
>
>      http://openvswitch.org/pipermail/dev/2016-March/067672.html
>
> Since the current recursion depth is available (it is already tracked
> by the exec_actions_level pcpu variable), we can use it to determine
> whether to execute recirculation actions immediately (safe when
> recursion depth is low) or defer execution until more stack space is
> available.
>
> With this change, the deferred action fifo size becomes a non-issue
> for currently failing scenarios because it is no longer used when
> there are three or fewer recursions through ovs_execute_actions().
>
> Suggested-by: Pravin Shelar <pshelar@....org>
> Signed-off-by: Lance Richardson <lrichard@...hat.com>

Thanks for working on it.

Acked-by: Pravin B Shelar <pshelar@....org>