linux-kernel - Re: [PATCH 08/36] AArch64: Kernel booting and initialisation

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: <50065E6B.3060602@jonmasters.org>
Date:	Wed, 18 Jul 2012 02:57:47 -0400
From:	Jon Masters <jonathan@...masters.org>
To:	Catalin Marinas <catalin.marinas@....com>
CC:	linux-kernel@...r.kernel.org, Arnd Bergmann <arnd@...db.de>,
	Will Deacon <will.deacon@....com>
Subject: Re: [PATCH 08/36] AArch64: Kernel booting and initialisation

On 07/06/2012 05:05 PM, Catalin Marinas wrote:

> +- CPU mode
> +  All forms of interrupts must be masked in PSTATE.DAIF (Debug, SError,
> +  IRQ and FIQ).
> +  The CPU must be in either EL2 (RECOMMENDED) or non-secure EL1.

Even though this stuff is likely to be replaced with the result of some
of the other standardization, I'd like it if you'd strongly consider
removing the "or non-secure EL1". If you give an inch, someone will take
a mile and build a system that enters other than in EL2. Or, something
to the effect of "the highest non-secure exception level implemented"
would be my preference if you don't want to specify.

> diff --git a/arch/aarch64/include/asm/setup.h b/arch/aarch64/include/asm/setup.h

> +#define COMMAND_LINE_SIZE 1024

Suggest at least the same size as x86 (2048) for command line params, or
we'll just be coming back later to change it :)

Jon.
--
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/