netdev - Re: [RFC PATCH] net: sched: fix unsued cpu variable

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: <54148C0B.8040500@intel.com>
Date:	Sat, 13 Sep 2014 11:25:15 -0700
From:	John Fastabend <john.r.fastabend@...el.com>
To:	John Fastabend <john.fastabend@...il.com>, netdev@...r.kernel.org
Subject: Re: [RFC PATCH] net: sched: fix unsued cpu variable

On 09/13/2014 11:13 AM, John Fastabend wrote:
> kbuild test robot reported an unused variable cpu in cls_u32.c
> after the patch below. This happens when PERF and MARK config
> variables are disabled
> 
> commit 459d5f626da75573e985a7197b0919c3b143146c
> Author: John Fastabend <john.fastabend@...il.com>
> Date:   Fri Sep 12 20:08:47 2014 -0700
> 
>     net: sched: make cls_u32 per cpu
> 
> One way to fix this is to use separate variables for perf
> and mark and define the cpu variable inside the ifdef
> logic.
> 
> I don't really like this though so I'm wondering if I can
> clean up the ifdef/endif logic to make this easier to read.
> So RFC patch until I convince myself there isn't a better
> way. Thought it might be worth sending out in case someone
> else has an idea.
> 
> Signed-off-by: John Fastabend <john.r.fastabend@...el.com>

kbuild robot kicked me another warning I missed I'll send out
a couple patches tonight or tomorrow morning to resolve them.

Thanks,
John
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html