lkml.org 
[lkml]   [2017]   [Mar]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 22/26] x86/mm: add sync_global_pgds() for configuration with 5-level paging

* Kirill A. Shutemov <kirill.shutemov@linux.intel.com> wrote:

> This basically restores slightly modified version of original
> sync_global_pgds() which we had before foldedl p4d was introduced.

Please read your changelogs, I saw several typos/grammar mistakes in earlier
patches. The one here is:

s/foldedl/folded

> + for (address = start; address <= end && address >= start;
> + address += PGDIR_SIZE) {

Please don't address col80 checkpatch warnings by breaking the line in such an
ugly way! Find another method, or just leave it slightly longer than 80 cols.

This one could probably be solved by:

s/address/addr

... which is the canonical variable name for such iterations anyway.

> + /* the pgt_lock only for Xen */

Please use whole sentences in comments, and please capitalize them properly.

I.e. here:


/* We acquire the pgt_lock only for Xen: */

> + BUG_ON(pgd_page_vaddr(*pgd)
> + != pgd_page_vaddr(*pgd_ref));

Ugly col80 artifact ...

Please review the rest of the series for similar patterns as well, and please only
post 5-10 patches in the next submission - we'll review and apply them step by
step.

Thanks,

Ingo

\
 
 \ /
  Last update: 2017-03-13 08:22    [W:0.175 / U:23.932 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site