Commit af727902 authored by Jeff Dike's avatar Jeff Dike Committed by Linus Torvalds

[PATCH] uml: fix host LDT lookup initialization locking, try 2

Add some locking to host_ldt_entries to prevent racing when reading LDT
information from the host.

The locking is somewhat more careful than my previous attempt.  Now, only
the check of host_ldt_entries is locked.  The lock is dropped immediately
afterwards, and if the LDT needs initializing, that (and the memory
allocations needed) proceed outside the lock.

Also fixed some style violations.
Signed-off-by: default avatarJeff Dike <jdike@addtoit.com>
Cc: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent dae9f8e6
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
#include "linux/slab.h" #include "linux/slab.h"
#include "linux/types.h" #include "linux/types.h"
#include "linux/errno.h" #include "linux/errno.h"
#include "linux/spinlock.h"
#include "asm/uaccess.h" #include "asm/uaccess.h"
#include "asm/smp.h" #include "asm/smp.h"
#include "asm/ldt.h" #include "asm/ldt.h"
...@@ -386,23 +387,33 @@ static long do_modify_ldt_skas(int func, void __user *ptr, ...@@ -386,23 +387,33 @@ static long do_modify_ldt_skas(int func, void __user *ptr,
return ret; return ret;
} }
short dummy_list[9] = {0, -1}; static DEFINE_SPINLOCK(host_ldt_lock);
short * host_ldt_entries = NULL; static short dummy_list[9] = {0, -1};
static short * host_ldt_entries = NULL;
void ldt_get_host_info(void) static void ldt_get_host_info(void)
{ {
long ret; long ret;
struct ldt_entry * ldt; struct ldt_entry * ldt, *tmp;
int i, size, k, order; int i, size, k, order;
spin_lock(&host_ldt_lock);
if(host_ldt_entries != NULL){
spin_unlock(&host_ldt_lock);
return;
}
host_ldt_entries = dummy_list+1; host_ldt_entries = dummy_list+1;
spin_unlock(&host_ldt_lock);
for(i = LDT_PAGES_MAX-1, order=0; i; i>>=1, order++); for(i = LDT_PAGES_MAX-1, order=0; i; i>>=1, order++);
ldt = (struct ldt_entry *) ldt = (struct ldt_entry *)
__get_free_pages(GFP_KERNEL|__GFP_ZERO, order); __get_free_pages(GFP_KERNEL|__GFP_ZERO, order);
if(ldt == NULL) { if(ldt == NULL) {
printk("ldt_get_host_info: couldn't allocate buffer for host ldt\n"); printk("ldt_get_host_info: couldn't allocate buffer for host "
"ldt\n");
return; return;
} }
...@@ -426,11 +437,13 @@ void ldt_get_host_info(void) ...@@ -426,11 +437,13 @@ void ldt_get_host_info(void)
host_ldt_entries = dummy_list; host_ldt_entries = dummy_list;
else { else {
size = (size + 1) * sizeof(dummy_list[0]); size = (size + 1) * sizeof(dummy_list[0]);
host_ldt_entries = kmalloc(size, GFP_KERNEL); tmp = kmalloc(size, GFP_KERNEL);
if(host_ldt_entries == NULL) { if(tmp == NULL) {
printk("ldt_get_host_info: couldn't allocate host ldt list\n"); printk("ldt_get_host_info: couldn't allocate host ldt "
"list\n");
goto out_free; goto out_free;
} }
host_ldt_entries = tmp;
} }
for(i=0, k=0; i<ret/LDT_ENTRY_SIZE; i++){ for(i=0, k=0; i<ret/LDT_ENTRY_SIZE; i++){
...@@ -480,7 +493,6 @@ long init_new_ldt(struct mmu_context_skas * new_mm, ...@@ -480,7 +493,6 @@ long init_new_ldt(struct mmu_context_skas * new_mm,
* inherited from the host. All ldt-entries found * inherited from the host. All ldt-entries found
* will be reset in the following loop * will be reset in the following loop
*/ */
if(host_ldt_entries == NULL)
ldt_get_host_info(); ldt_get_host_info();
for(num_p=host_ldt_entries; *num_p != -1; num_p++){ for(num_p=host_ldt_entries; *num_p != -1; num_p++){
desc.entry_number = *num_p; desc.entry_number = *num_p;
...@@ -560,6 +572,6 @@ void free_ldt(struct mmu_context_skas * mm) ...@@ -560,6 +572,6 @@ void free_ldt(struct mmu_context_skas * mm)
int sys_modify_ldt(int func, void __user *ptr, unsigned long bytecount) int sys_modify_ldt(int func, void __user *ptr, unsigned long bytecount)
{ {
return(CHOOSE_MODE_PROC(do_modify_ldt_tt, do_modify_ldt_skas, func, return CHOOSE_MODE_PROC(do_modify_ldt_tt, do_modify_ldt_skas, func,
ptr, bytecount)); ptr, bytecount);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment