[lkml]   [2019]   [Jan]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 1/3] riscv: Introduce huge page support for 32/64bit kernel
On 01/11/2019 07:18 AM, Paul Walmsley wrote:
> Hello Alexandre,
> On Mon, 10 Dec 2018, Alexandre Ghiti wrote:
>> This patch implements both 4MB huge page support for 32bit kernel
>> and 2MB/1GB huge pages support for 64bit kernel.
>> Signed-off-by: Alexandre Ghiti <>
> This patch introduces a few minor checkpatch and strict checkpatch
> warnings, when tested against both v4.20 and v5.0-rc1 checkpatch versions.
> Could you fix those? The other two patches in the series look OK from
> that point of view.

Yes, I had noted those warnings:

- the SPDX warning is weird, in all *.c files, the script seems to
accept only
  "// SPDX-License-Identifier: GPL-2.0" and not "/*
SPDX-License-Identifier: GPL-2.0 */".
  But on *.h files, this the contrary, maybe I missed something, I am going
  to take a look at the script
  Note that kernel/ftrace.c and kernel/perf_event.c raise the same
strange warning.

- I'll change the "printk(KERN_ERR" into a pr_err.

- I'll ignore the last warning regarding the new file.

Thanks for your feedback Paul,


> - Paul
> _______________________________________________
> linux-riscv mailing list

 \ /
  Last update: 2019-01-11 15:00    [W:0.031 / U:1.884 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site