From: tip-bot for Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
To: linux-tip-commits@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, hpa@zytor.com, mingo@redhat.com,
rusty@rustcorp.com.au, xiaoguangrong@cn.fujitsu.com,
jens.axboe@oracle.com, jeremy@goop.org,
akpm@linux-foundation.org, tglx@linutronix.de, mingo@elte.hu
Subject: [tip:x86/mm] x86: Reuse the boot-time mappings of fixed_addresses
Date: Fri, 4 Sep 2009 07:59:09 GMT [thread overview]
Message-ID: <tip-d9fd2e4645eeb87415618fd0a8f5c992e6b43b37@git.kernel.org> (raw)
In-Reply-To: <4A90AADE.20307@gmail.com>
Commit-ID: d9fd2e4645eeb87415618fd0a8f5c992e6b43b37
Gitweb: http://git.kernel.org/tip/d9fd2e4645eeb87415618fd0a8f5c992e6b43b37
Author: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
AuthorDate: Sun, 23 Aug 2009 10:35:10 +0800
Committer: Ingo Molnar <mingo@elte.hu>
CommitDate: Fri, 4 Sep 2009 09:35:17 +0200
x86: Reuse the boot-time mappings of fixed_addresses
Some fixed_addresses items are only used when system boot,
after boot, they are free but no way to use, like early ioremap
area. They are wasted for us, we can reuse them after system
boot.
In this patch, we put them in permanent kmap's area and expand
vmalloc's address range. In boot time, reserve them in
permanent_kmaps_init() to avoid multiple used, after system
boot, we unreserved them then user can use it.
Signed-off-by: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
Acked-by: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Cc: Jens Axboe <jens.axboe@oracle.com>
Cc: linux-mm@kvack.org
Cc: Andrew Morton <akpm@linux-foundation.org>
LKML-Reference: <4A90AADE.20307@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
arch/x86/include/asm/fixmap.h | 2 ++
arch/x86/include/asm/pgtable_32_types.h | 4 ++--
arch/x86/mm/init_32.c | 8 ++++++++
include/linux/highmem.h | 2 ++
mm/highmem.c | 26 ++++++++++++++++++++++++++
5 files changed, 40 insertions(+), 2 deletions(-)
diff --git a/arch/x86/include/asm/fixmap.h b/arch/x86/include/asm/fixmap.h
index 7b2d71d..604f135 100644
--- a/arch/x86/include/asm/fixmap.h
+++ b/arch/x86/include/asm/fixmap.h
@@ -142,6 +142,8 @@ extern void reserve_top_address(unsigned long reserve);
#define FIXADDR_BOOT_SIZE (__end_of_fixed_addresses << PAGE_SHIFT)
#define FIXADDR_START (FIXADDR_TOP - FIXADDR_SIZE)
#define FIXADDR_BOOT_START (FIXADDR_TOP - FIXADDR_BOOT_SIZE)
+#define FIXMAP_REUSE (__end_of_fixed_addresses - \
+ __end_of_permanent_fixed_addresses)
extern int fixmaps_set;
diff --git a/arch/x86/include/asm/pgtable_32_types.h b/arch/x86/include/asm/pgtable_32_types.h
index 5e67c15..328b8af 100644
--- a/arch/x86/include/asm/pgtable_32_types.h
+++ b/arch/x86/include/asm/pgtable_32_types.h
@@ -37,8 +37,8 @@ extern bool __vmalloc_start_set; /* set once high_memory is set */
#define LAST_PKMAP 1024
#endif
-#define PKMAP_BASE ((FIXADDR_BOOT_START - PAGE_SIZE * (LAST_PKMAP + 1)) \
- & PMD_MASK)
+#define PKMAP_BASE ((FIXADDR_BOOT_START - PAGE_SIZE * (LAST_PKMAP - \
+ FIXMAP_REUSE + 1)) & PMD_MASK)
#ifdef CONFIG_HIGHMEM
# define VMALLOC_END (PKMAP_BASE - 2 * PAGE_SIZE)
diff --git a/arch/x86/mm/init_32.c b/arch/x86/mm/init_32.c
index 3cd7711..595e485 100644
--- a/arch/x86/mm/init_32.c
+++ b/arch/x86/mm/init_32.c
@@ -410,8 +410,16 @@ static void __init permanent_kmaps_init(pgd_t *pgd_base)
pmd = pmd_offset(pud, vaddr);
pte = pte_offset_kernel(pmd, vaddr);
pkmap_page_table = pte;
+ kmaps_reserve(LAST_PKMAP-FIXMAP_REUSE, LAST_PKMAP-1);
}
+static int __init permanent_kmaps_unreserve(void)
+{
+ kmaps_unreserve(LAST_PKMAP-FIXMAP_REUSE, LAST_PKMAP-1);
+ return 0;
+}
+late_initcall(permanent_kmaps_unreserve);
+
static void __init add_one_highpage_init(struct page *page, int pfn)
{
ClearPageReserved(page);
diff --git a/include/linux/highmem.h b/include/linux/highmem.h
index 211ff44..984c4c9 100644
--- a/include/linux/highmem.h
+++ b/include/linux/highmem.h
@@ -41,6 +41,8 @@ unsigned int nr_free_highpages(void);
extern unsigned long totalhigh_pages;
void kmap_flush_unused(void);
+void kmaps_reserve(int start, int end);
+void kmaps_unreserve(int start, int end);
#else /* CONFIG_HIGHMEM */
diff --git a/mm/highmem.c b/mm/highmem.c
index 25878cc..a481fa7 100644
--- a/mm/highmem.c
+++ b/mm/highmem.c
@@ -85,6 +85,32 @@ static DECLARE_WAIT_QUEUE_HEAD(pkmap_map_wait);
do { spin_unlock(&kmap_lock); (void)(flags); } while (0)
#endif
+void kmaps_reserve(int start, int end)
+{
+ int i;
+
+ lock_kmap();
+ for (i = start; i <= end; i++) {
+ BUG_ON(pkmap_count[i]);
+ pkmap_count[i] = -1;
+ }
+ unlock_kmap();
+}
+
+void kmaps_unreserve(int start, int end)
+{
+ int i;
+
+ lock_kmap();
+ for (i = start; i <= end; i++) {
+ BUG_ON(pkmap_count[i] != -1);
+ pkmap_count[i] = 0;
+ }
+
+ flush_tlb_kernel_range(PKMAP_ADDR(start), PKMAP_ADDR(end));
+ unlock_kmap();
+}
+
static void flush_all_zero_pkmaps(void)
{
int i;
prev parent reply other threads:[~2009-09-04 8:00 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-23 2:35 [PATCH] x86: reuse the boot-time mappings of fixed_addresses Xiao Guangrong
2009-08-29 11:00 ` Ingo Molnar
2009-08-29 18:16 ` H. Peter Anvin
2009-08-31 8:26 ` Ingo Molnar
2009-09-01 0:29 ` Jeremy Fitzhardinge
2009-09-04 7:33 ` Ingo Molnar
2009-09-04 15:47 ` Jeremy Fitzhardinge
2009-09-04 16:15 ` H. Peter Anvin
2009-09-04 18:55 ` Ingo Molnar
2009-09-04 7:59 ` tip-bot for Xiao Guangrong [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=tip-d9fd2e4645eeb87415618fd0a8f5c992e6b43b37@git.kernel.org \
--to=xiaoguangrong@cn.fujitsu.com \
--cc=akpm@linux-foundation.org \
--cc=hpa@zytor.com \
--cc=jens.axboe@oracle.com \
--cc=jeremy@goop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tip-commits@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mingo@redhat.com \
--cc=rusty@rustcorp.com.au \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox