[PATCH v2 0/5] X86/XEN: Merge x86_init.paging.pagetable_setup_start and x86_init.paging.pagetable_setup_done setup functions and document its semantic

From: Attilio Rao
Date: Tue Aug 21 2012 - 16:36:09 EST


Currently the definition of x86_init.paging.pagetable_setup_start and
x86_init.paging.pagetable_setup_done is twisted and not really well
defined (in terms of prototypes desired). More specifically:
pagetable_setup_start:
* cleans up the boot time page table in the x86_32 case
* it is a nop for the XEN case
* it is a nop on x86_64

pagetable_setup_done:
* it is a nop on x86_32
* sets up accessor functions for pagetable manipulation, for the
XEN case
* it is a nop on x86_64

Most of this logic can be skipped by creating a new setup function that can
handle pagetable setup and pre/post operations on it. This means the above
mentioned functions will be removed and only one will be used for the whole
operation.
The new function must be called only once, during boot-time setup and
after the direct mapping for physical memory is available.

Differences with v1:
- The patch serie is re-arranged in a way that it helps reviews, following
a plan by Thomas Gleixner
- The PVOPS nomenclature is not used as it is not correct
- The front-end message is adjusted with feedback by Thomas Gleixner,
Stefano Stabellini and Konrad Rzeszutek Wilk


Attilio Rao (5):
X86/XEN: Remove the base argument from
x86_init.paging.pagetable_setup_start
X86/XEN: Rename pagetable_setup_start() setup functions into
pagetable_init()
X86/XEN: Allow setup function x86_init.paging.pagetable_init to setup
kernel pagetables
X86/XEN: Move content of xen_pagetable_setup_done() into
xen_pagetable_init() and retire now unused
x86_init.paging.pagetable_setup_done
X86/XEN: Add few lines explaining simple semantic for
x86_init.paging.pagetable_init setup function

arch/x86/include/asm/pgtable_types.h | 6 ++----
arch/x86/include/asm/x86_init.h | 11 +++++++----
arch/x86/kernel/setup.c | 4 +---
arch/x86/kernel/x86_init.c | 4 +---
arch/x86/mm/init_32.c | 11 ++++-------
arch/x86/xen/mmu.c | 18 +++++++-----------
6 files changed, 22 insertions(+), 32 deletions(-)

--
1.7.2.5

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