[PATCH] [14/50] x86: Minor code-style cleanups to change_page_attr

From: Andi Kleen
Date: Fri Sep 21 2007 - 18:36:52 EST



Remove a one liner function and expand into parent.
No functional changes.

Signed-off-by: Andi Kleen <ak@xxxxxxx>

---
arch/i386/mm/pageattr.c | 7 +------
arch/x86_64/mm/pageattr.c | 7 +------
2 files changed, 2 insertions(+), 12 deletions(-)

Index: linux/arch/x86_64/mm/pageattr.c
===================================================================
--- linux.orig/arch/x86_64/mm/pageattr.c
+++ linux/arch/x86_64/mm/pageattr.c
@@ -93,11 +93,6 @@ static void flush_kernel_map(void *arg)
__flush_tlb_all();
}

-static inline void flush_map(struct list_head *l)
-{
- on_each_cpu(flush_kernel_map, l, 1, 1);
-}
-
/* both protected by init_mm.mmap_sem */
static int full_flush;
static LIST_HEAD(deferred_pages);
@@ -271,7 +266,7 @@ void global_flush_tlb(void)
list_replace_init(&deferred_pages, &arg.l);
up_read(&init_mm.mmap_sem);

- flush_map(&arg);
+ on_each_cpu(flush_kernel_map, &arg, 1, 1);

list_for_each_entry_safe(pg, next, &arg.l, lru) {
list_del(&pg->lru);
Index: linux/arch/i386/mm/pageattr.c
===================================================================
--- linux.orig/arch/i386/mm/pageattr.c
+++ linux/arch/i386/mm/pageattr.c
@@ -238,11 +238,6 @@ __change_page_attr(struct page *page, pg
return 0;
}

-static inline void flush_map(struct list_head *l)
-{
- on_each_cpu(flush_kernel_map, l, 1, 1);
-}
-
/*
* Change the page attributes of an page in the linear mapping.
*
@@ -284,7 +279,7 @@ void global_flush_tlb(void)
full_flush = 0;
list_replace_init(&df_list, &arg.l);
spin_unlock_irq(&cpa_lock);
- flush_map(&arg);
+ on_each_cpu(flush_kernel_map, &arg, 1, 1);
list_for_each_entry_safe(pg, next, &arg.l, lru) {
list_del(&pg->lru);
clear_bit(PG_arch_1, &pg->flags);
-
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/