Re: [Qemu-devel] [PATCH] Remove qemu_host_page_bits

[Stefan Hajnoczi]

On Sat, Sep 17, 2011 at 12:25:30PM +0200, Stefan Weil wrote:
 It was introduced with commit 54936004fddc52c321cb3f9a9a51140e782bed5d
 as host_page_bits but never used.
 
 Signed-off-by: Stefan Weil w...@mail.berlios.de
 ---
  cpu-all.h |1 -
  exec.c|4 
  2 files changed, 0 insertions(+), 5 deletions(-)

Thanks, applied to the trivial patches tree:
http://repo.or.cz/w/qemu/stefanha.git/shortlog/refs/heads/trivial-patches

Stefan

[Qemu-devel] [PATCH] Remove qemu_host_page_bits

[Stefan Weil]

It was introduced with commit 54936004fddc52c321cb3f9a9a51140e782bed5d
as host_page_bits but never used.

Signed-off-by: Stefan Weil w...@mail.berlios.de
---
 cpu-all.h |1 -
 exec.c|4 
 2 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/cpu-all.h b/cpu-all.h
index 3532026..e8143cd 100644
--- a/cpu-all.h
+++ b/cpu-all.h
@@ -290,7 +290,6 @@ extern unsigned long reserved_va;
 
 /* ??? These should be the larger of unsigned long and target_ulong.  */
 extern unsigned long qemu_real_host_page_size;
-extern unsigned long qemu_host_page_bits;
 extern unsigned long qemu_host_page_size;
 extern unsigned long qemu_host_page_mask;
 
diff --git a/exec.c b/exec.c
index 3df6b23..639deae 100644
--- a/exec.c
+++ b/exec.c
@@ -183,7 +183,6 @@ typedef struct PageDesc {
 #define V_L1_SHIFT (L1_MAP_ADDR_SPACE_BITS - TARGET_PAGE_BITS - V_L1_BITS)
 
 unsigned long qemu_real_host_page_size;
-unsigned long qemu_host_page_bits;
 unsigned long qemu_host_page_size;
 unsigned long qemu_host_page_mask;
 
@@ -275,9 +274,6 @@ static void page_init(void)
 qemu_host_page_size = qemu_real_host_page_size;
 if (qemu_host_page_size  TARGET_PAGE_SIZE)
 qemu_host_page_size = TARGET_PAGE_SIZE;
-qemu_host_page_bits = 0;
-while ((1  qemu_host_page_bits)  qemu_host_page_size)
-qemu_host_page_bits++;
 qemu_host_page_mask = ~(qemu_host_page_size - 1);
 
 #if defined(CONFIG_BSD)  defined(CONFIG_USER_ONLY)
-- 
1.7.2.5