netdev - Re: [RFC PATCH net-next] net: mvpp2: mvpp2_flow_get_hek

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20180717093801.6bddd5b5@bootlin.com>
Date:   Tue, 17 Jul 2018 09:38:01 +0200
From:   Maxime Chevallier <maxime.chevallier@...tlin.com>
To:     kbuild test robot <lkp@...el.com>
Cc:     kbuild-all@...org, netdev@...r.kernel.org,
        Antoine Tenart <antoine.tenart@...tlin.com>,
        kbuild test robot <fengguang.wu@...el.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH net-next] net: mvpp2: mvpp2_flow_get_hek_fields()
 can be static

Hi,

On Tue, 17 Jul 2018 09:18:12 +0800
kbuild test robot <lkp@...el.com> wrote:

>Fixes: d33ec4525007 ("net: mvpp2: add an RSS classification step for each flow")
>Signed-off-by: kbuild test robot <fengguang.wu@...el.com>
>---
> mvpp2_cls.c |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.c b/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.c
>index 723d0ba..fc9ec51 100644
>--- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.c
>+++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.c
>@@ -618,7 +618,7 @@ static int mvpp2_port_rss_hash_opts_set(struct mvpp2_port *port, int flow_type,
> 	return 0;
> }
> 
>-u16 mvpp2_flow_get_hek_fields(struct mvpp2_cls_flow_entry *fe)
>+static u16 mvpp2_flow_get_hek_fields(struct mvpp2_cls_flow_entry *fe)
> {
> 	u16 hash_opts = 0;
> 	int n_fields, i, field;

That was a valid patch since it seems I forgot to make
another function static in the RSS series.

However this function is used in debugfs, so this patch isn't valid
anymore.

Sorry about that, but please don't apply this.

Thanks,

Maxime

-- 
Maxime Chevallier, Bootlin (formerly Free Electrons)
Embedded Linux and kernel engineering
https://bootlin.com