[lkml]   [2001]   [Sep]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
SubjectRe: Feedback on preemptible kernel patch
Arjan, Jordan, and anyone using preemption and highmem:

Can you _please_ test the following patch? It is my final version of
the highmem patch, and the one I would like to include in the preempt
patch itself.

It should be faster than the patch you have been using before, the locks
in the previous patch were held for the duration that the entire page
remained map. I don't see any reason for that.

I would appreciate if you could patch your kernel and let me know. I am
putting 2.4.10-pre8 patches up at shortly.
2.4.9-ac10 patches are there, too.

If you test, please reply with: kernel version, do you lock/oops with
highmem enabled but no extra patch (you should), do you lock/oops with
highmem enabled and the original highmem patch (you should not), and do
you lock/oops/something with this new highmem patch (I hope not).
Obviously enable CONFIG_PREEMPT and CONFIG_HIGHMEM, and test well.
Please tell me if CONFIG_SMP is enabled (that is another bag of fun...).

Thank you...

--- linux-not-rml/include/asm-i386/highmem.h Tue Sep 11 17:54:32 2001
+++ linux/include/asm-i386/highmem.h Tue Sep 11 18:42:13 2001
@@ -88,6 +88,8 @@
if (page < highmem_start_page)
return page_address(page);

+ ctx_sw_off();
idx = type + KM_TYPE_NR*smp_processor_id();
vaddr = __fix_to_virt(FIX_KMAP_BEGIN + idx);
@@ -97,6 +99,8 @@
set_pte(kmap_pte-idx, mk_pte(page, kmap_prot));

+ ctx_sw_on();
return (void*) vaddr;

@@ -106,6 +110,8 @@
unsigned long vaddr = (unsigned long) kvaddr;
enum fixed_addresses idx = type + KM_TYPE_NR*smp_processor_id();

+ ctx_sw_off();
if (vaddr < FIXADDR_START) // FIXME

@@ -118,6 +124,8 @@
+ ctx_sw_on();

Robert M. Love
rml at
rml at

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:03    [W:0.026 / U:2.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site