netdev - Re: mlx5 stable backport help

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: <33b67a2b9a67bc9fe307edc0a3eeeab875598a09.camel@mellanox.com>
Date:   Wed, 20 Feb 2019 20:01:00 +0000
From:   Saeed Mahameed <saeedm@...lanox.com>
To:     "davem@...emloft.net" <davem@...emloft.net>
CC:     "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        Or Gerlitz <ogerlitz@...lanox.com>
Subject: Re: mlx5 stable backport help

On Tue, 2019-02-19 at 22:36 -0800, David Miller wrote:
> Saeed and Or,
> 
> I need help backporting two changes for -stable.  Namely:

Hi Dave Thanks for trying, 

> 
> 6707f74be8621ae067d2cf1c4485900e2742c20f ("net/mlx5e: Update hw flows
> when encap source mac changed")

The fixes tag of the above commit is wrong and it should have been

commit e32ee6c78efa6a32bff782bbe7a9970b018996ca 
Author: Eli Britstein <elibr@...lanox.com>
Date:   Mon Dec 3 17:09:54 2018 +0200

    net/mlx5e: Support tunnel encap over tagged Ethernet

Before this patch we didn't have the support for "tunnel encap over
tagged Ethernet" which introduced the whole route_dev logic to generate
the encap headers, so the patch should only exist in -rc, let's just
skip it.

> 218d05ce326f9e1b40a56085431fa1068b43d5d9 ("net/mlx5e: Don't overwrite
> pedit action when multiple pedit used")
> 

I tried and this patch applies smoothly on 4.19.

let me know if you need further assistance.

Thanks,
Saeed

> For example, with the first one, not only has the code moved into
> tc_tun.c but also we require the addition of
> get_route_and_out_devs(),
> the passing in of the &route_dev, and also all of the new
> dst_is_lag_dev logic.  That's like 4 or 5 commits worth.
> 
> Thanks.