linux-kernel - Re: [PATCH v2 7/7] ia64: add __IGNORE* entries in asm/unistd.h

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: <CAK8P3a2E2KJApEcwpJgqs9KHuyr-mzE8Y6F+D1E0akpXt8VK_Q@mail.gmail.com>
Date:   Fri, 28 Sep 2018 14:09:30 +0200
From:   Arnd Bergmann <arnd@...db.de>
To:     Firoz Khan <firoz.khan@...aro.org>
Cc:     linux-ia64@...r.kernel.org, Tony Luck <tony.luck@...el.com>,
        Fenghua Yu <fenghua.yu@...el.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        gregkh <gregkh@...uxfoundation.org>,
        Philippe Ombredanne <pombredanne@...b.com>,
        Kate Stewart <kstewart@...uxfoundation.org>,
        y2038 Mailman List <y2038@...ts.linaro.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        linux-arch <linux-arch@...r.kernel.org>,
        Deepa Dinamani <deepa.kernel@...il.com>,
        Marcin Juszkiewicz <marcin.juszkiewicz@...aro.org>
Subject: Re: [PATCH v2 7/7] ia64: add __IGNORE* entries in asm/unistd.h

On Thu, Sep 27, 2018 at 4:16 PM Firoz Khan <firoz.khan@...aro.org> wrote:
>
> Add __IGNORE* entries for perf_event_open, seccomp, pkey_
> mprotect, pkey_alloc, pkey_free, statx, io_pgetevents and
> rseq in the file asm/unistd.h as it is correct to keep
> __IGNORE* entry in non uapi header asm/unistd.h while uapi/
> asm/unistd.h must hold information only useful for user
> space applications.
>
> Signed-off-by: Firoz Khan <firoz.khan@...aro.org>
> ---

This patch looks wrong: We don't want to ignore the fact that the
syscalls are missing here, we should add them instead. I'd leave
that for a follow-up though, we may want to add further ones that
are currently marked as __IGNORE.

      Arnd