lkml.org 
[lkml]   [2013]   [Oct]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 02/11] asmlinkage: Make __iowrite32_copy visible

* Andi Kleen <andi@firstfloor.org> wrote:

> From: Andi Kleen <ak@linux.intel.com>
>
> This is a assembler function on x86, so it should be visible.
>
> Cc: x86@kernel.org
> Signed-off-by: Andi Kleen <ak@linux.intel.com>
> ---
> include/linux/io.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

NAK for this and all the other patches, for two technical reasons:

- Please squash most of these trivial patches into a single patch
that also describes the context completely, at least the x86 and
core kernel ones that I'd apply within a single maintenance
domain.

- Please use the customary changelog style we use in the kernel:

" Current code does (A), this has a problem when (B).
We can improve this doing (C), because (D)."

Just saying 'because of LTO' is not enough, obviously. You need to
talk about and explain things Andi. Being cryptic, mysterious and
curt might work with some ladies, but it's outright harmful on lkml.

Thanks,

Ingo


\
 
 \ /
  Last update: 2013-10-26 12:41    [W:0.111 / U:0.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site