linux-kernel - Re: [RFC,PATCH] arch/arm: compute and export NR

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: <CABqD9hYavCxA38GbBhbCmBT+PEQQsvhbF4U7xY8P8ygbFy+=Ag@mail.gmail.com>
Date:	Tue, 16 Aug 2011 16:44:56 -0500
From:	Will Drewry <wad@...omium.org>
To:	Arnd Bergmann <arnd@...db.de>
Cc:	linux-kernel@...r.kernel.org,
	Russell King <linux@....linux.org.uk>,
	Mikael Pettersson <mikpe@...uu.se>,
	"Eric W. Biederman" <ebiederm@...ssion.com>,
	Mike Frysinger <vapier@...too.org>,
	linux-arm-kernel@...ts.infradead.org
Subject: Re: [RFC,PATCH] arch/arm: compute and export NR_syscalls

On Tue, Aug 16, 2011 at 4:21 PM, Arnd Bergmann <arnd@...db.de> wrote:
> On Tuesday 16 August 2011 16:14:26 Will Drewry wrote:
>>
>> asm-exports.c is added instead of reusing asm-offsets.c to avoid a
>> variety of collisions (VM_EXEC, DMA_*, etc).  It is possible to use the
>> same calls.S mechanism but add NR_syscalls to asm-offsets.c.  However,
>> at inclusion time for generated/asm-offsets.h, conflicting defines will
>> need to be #undef'd if !__ASSEMBLY__ since it appears that the purpose
>> of asm-offsets.h is to safely bind C language definitions to assembly
>> and not the reverse.
>>
>> - Is this approach palatable?
>> - Should I resend only when paired with the other ftrace-needed patches?
>
> This seems overly complex, compared to a one-line change adding the symbol
> to asm/unistd.h. The only other architecture that uses an approach
> like the one you have posted is x86-64, and it's simpler there
> because it can easily be done in asm-offsets.c there without the need
> to create another helper.

Agreed!

I proposed this approach based solely on prior threads I've seen. E.g.,
- https://lkml.org/lkml/2007/6/1/427
  (don't just #define)
- https://lkml.org/lkml/2009/8/27/280
  (todo: x86-32 to move to x86-64)

If a single line #define is good enough, then it certainly works for me.

thanks!
will
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/