Hibernation: Pass CR3 in the image header on x86_64
authorRafael J. Wysocki <rjw@sisk.pl>
Thu, 18 Oct 2007 10:04:54 +0000 (03:04 -0700)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Thu, 18 Oct 2007 21:37:19 +0000 (14:37 -0700)
Since we already pass the address of restore_registers() in the image header,
we can also pass the value of the CR3 register from before the hibernation in
the same way.  This will allow us to avoid using init_level4_pgt page tables
during the restore.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Pavel Machek <pavel@ucw.cz>
Cc: Andi Kleen <ak@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/x86/kernel/suspend_64.c
arch/x86/kernel/suspend_asm_64.S

index 01fbfb018ca94590888eeb5bdd4dda25b41f4fdc..da10eef4c3ef6a7e6c2a5ce8e704b0198e255fd2 100644 (file)
@@ -156,6 +156,12 @@ extern int restore_image(void);
  */
 unsigned long restore_jump_address;
 
+/*
+ * Value of the cr3 register from before the hibernation (this value is passed
+ * in the image header).
+ */
+unsigned long restore_cr3;
+
 pgd_t *temp_level4_pgt;
 
 void *relocated_restore_code;
@@ -254,7 +260,8 @@ int pfn_is_nosave(unsigned long pfn)
 
 struct restore_data_record {
        unsigned long jump_address;
-       unsigned long control;
+       unsigned long cr3;
+       unsigned long magic;
 };
 
 #define RESTORE_MAGIC  0x0123456789ABCDEFUL
@@ -271,7 +278,8 @@ int arch_hibernation_header_save(void *addr, unsigned int max_size)
        if (max_size < sizeof(struct restore_data_record))
                return -EOVERFLOW;
        rdr->jump_address = restore_jump_address;
-       rdr->control = (restore_jump_address ^ RESTORE_MAGIC);
+       rdr->cr3 = restore_cr3;
+       rdr->magic = RESTORE_MAGIC;
        return 0;
 }
 
@@ -285,7 +293,7 @@ int arch_hibernation_header_restore(void *addr)
        struct restore_data_record *rdr = addr;
 
        restore_jump_address = rdr->jump_address;
-       return (rdr->control == (restore_jump_address ^ RESTORE_MAGIC)) ?
-                       0 : -EINVAL;
+       restore_cr3 = rdr->cr3;
+       return (rdr->magic == RESTORE_MAGIC) ? 0 : -EINVAL;
 }
 #endif /* CONFIG_HIBERNATION */
index 40a209e0525cce8fb7d1a8e8acf7318fea3e9813..48344b666d2c2e60f56d3393d82ecc48e0b9388f 100644 (file)
@@ -39,6 +39,9 @@ ENTRY(swsusp_arch_suspend)
        /* save the address of restore_registers */
        movq    $restore_registers, %rax
        movq    %rax, restore_jump_address(%rip)
+       /* save cr3 */
+       movq    %cr3, %rax
+       movq    %rax, restore_cr3(%rip)
 
        call swsusp_save
        ret
@@ -60,6 +63,7 @@ ENTRY(restore_image)
 
        /* prepare to jump to the image kernel */
        movq    restore_jump_address(%rip), %rax
+       movq    restore_cr3(%rip), %rbx
 
        /* prepare to copy image data to their original locations */
        movq    restore_pblist(%rip), %rdx
@@ -98,9 +102,7 @@ done:
 
 ENTRY(restore_registers)
        /* go back to the original page tables */
-       movq    $(init_level4_pgt - __START_KERNEL_map), %rax
-       addq    phys_base(%rip), %rax
-       movq    %rax, %cr3
+       movq    %rbx, %cr3
 
        /* Flush TLB, including "global" things (vmalloc) */
        movq    mmu_cr4_features(%rip), %rax