netdev - Re: ❌ FAIL: Test report for kernel 5.10.0-rc6 (mainline.kernel.org)

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: <ygnhpn3ijbyb.fsf@nvidia.com>
Date:   Thu, 10 Dec 2020 10:35:08 +0200
From:   Vlad Buslov <vladbu@...dia.com>
To:     Jakub Kicinski <kuba@...nel.org>
CC:     Eric Dumazet <edumazet@...gle.com>,
        Saeed Mahameed <saeed@...nel.org>,
        Jianlin Shi <jishi@...hat.com>,
        CKI Project <cki-project@...hat.com>,
        netdev <netdev@...r.kernel.org>, <skt-results-master@...hat.com>,
        Yi Zhang <yi.zhang@...hat.com>,
        Memory Management <mm-qe@...hat.com>,
        Jan Stancek <jstancek@...hat.com>,
        Jianwen Ji <jiji@...hat.com>, Hangbin Liu <haliu@...hat.com>,
        Ondrej Moris <omoris@...hat.com>,
        Ondrej Mosnacek <omosnace@...hat.com>,
        Changhui Zhong <czhong@...hat.com>,
        Xiong Zhou <xzhou@...hat.com>,
        Rachel Sibley <rasibley@...hat.com>,
        David Arcari <darcari@...hat.com>
Subject: Re: ❌ FAIL: Test report for kernel 5.10.0-rc6
 (mainline.kernel.org)

On Thu 10 Dec 2020 at 00:22, Jakub Kicinski <kuba@...nel.org> wrote:
> On Wed, 9 Dec 2020 22:54:40 +0200 Vlad Buslov wrote:
>> > Yes, I think the patch I sent should fix this, ETH_P_ARP should not be
>> > dropped ;)
>> >
>> > I am testing this before offical patch submission.
>> 
>> Your patch fixed TC geneve tests for me, but some of more complex OVS
>> tests are still failing. I'll try to provide details tomorrow.
>
> Does a revert of Eric's patch fix it? For OvS is could also well be:
> 9c2e14b48119 ("ip_tunnels: Set tunnel option flag when tunnel metadata is present")

The tests pass with Eric's commit reverted.