linux-kernel - Re: [RESEND PATCH 3/4] futex: Remove unused or redundant includes

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] [day] [month] [year] [list]

Message-ID: <87pn8tpu5h.fsf@nanos.tec.linutronix.de>
Date:   Sat, 18 Jul 2020 02:02:34 +0200
From:   Thomas Gleixner <tglx@...utronix.de>
To:     André Almeida <andrealmeid@...labora.com>,
        linux-kernel@...r.kernel.org, peterz@...radead.org
Cc:     mingo@...hat.com, dvhart@...radead.org, kernel@...labora.com,
        krisman@...labora.com
Subject: Re: [RESEND PATCH 3/4] futex: Remove unused or redundant includes

André Almeida <andrealmeid@...labora.com> writes:
> I just retested compiling tip/locking/core, and it seems something
> changed the last time I wrote the patch. Removing <linux/compat.h> will
> indeed break the compilation now, the patch bellow should fix it. Sorry
> for the noise.

I just folded it back manually and force pushed the result. Manually
because the below:

> =======================================
>
> diff --git a/kernel/futex.c b/kernel/futex.c
> index f483bc52dbac..4616d4ad609d 100644
>
> --- a/kernel/futex.c
>
> +++ b/kernel/futex.c
>
> @@ -31,6 +31,7 @@
>
>  31 *  "The futexes are also cursed."
>  32 *  "But they come in a choice of three flavours!"
>  33 */
> +34#include <linux/compat.h>
>  35#include <linux/jhash.h>
>  36#include <linux/pagemap.h>
>  37#include <linux/syscalls.h>

does not really qualify as a valid patch.