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

From: Jon Masters
Date: Wed Jul 18 2012 - 02:57:49 EST


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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/