linux-kernel - Re: [PATCH v3 00/31] AArch64 Linux kernel port

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: <20120907232537.GA5098@quad.lixom.net>
Date:	Fri, 7 Sep 2012 16:25:37 -0700
From:	Olof Johansson <olof@...om.net>
To:	Catalin Marinas <catalin.marinas@....com>
Cc:	linux-arch@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	linux-kernel@...r.kernel.org, Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH v3 00/31] AArch64 Linux kernel port

On Fri, Sep 07, 2012 at 05:26:35PM +0100, Catalin Marinas wrote:
> This is the 3rd version of patches implementing Linux kernel support for
> the 64-bit ARM architecture. Most of the review comments for the
> previous release have been implemented. There are a few left (like
> generic timer dts specification) but the main goal was to freeze the
> user ABI.
> 
> The Linux kernel patches are available on this tree:
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/cmarinas/linux-aarch64.git upstream
> 
> Main changes since the previous version:
> - ptrace interface now using regsets for TLS and hardware debug
>   registers.
> - sys_personality fixes. PER_LINUX is the default for all tasks (32 or
>   64-bit) and there is no compat_sys_personality defined.
> - Kconfig clean-up (unused symbols, AARCH32_EMULATION removed in favour
>   of COMPAT).
> - Clean-up of the __cpuc_* function prefixes.
> - CPU information table implemented in C rather than assembly.
> - __bad_* function calls replaced with BUILD_BUG.
> - Inline get_user/put_user (code cleaner, slightly bigger kernel Image).
> - DMA API implementation simplified, defaulting to swiotlb. It will be
>   extended as hardware becomes available.
> - Arch-specific code in the generic timer patch moved to arch/arm64/.
> - Comment fixes and clarifications.

I see that a good amount of the comments I had on v2 have neither been replied
to nor addressed.

Please address them where appropriate, ideally before merging. I'll be happy to
be told I'm wrong in some of the cases too, but completely ignoring feedback
isn't cool.

However, most of them were nits, and I've had a look-through of the rest
of the patches, feel free to add:

Acked-by: Olof Johansson <olof@...om.net>

on the patches in this series.

-Olof
--
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/