Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c with flush_tlb_all()

From: Andrew Morton (akpm@digeo.com)
Date: Thu Feb 20 2003 - 15:20:17 EST


Thomas Schlichter <schlicht@uni-mannheim.de> wrote:
>
> This patch replaces the flush_map() function in the arch/i386/mm/pageattr.c
> file with flush_tlb_all() calls, as the flush_map() function wants to do
> the same, but just forgot the preempt_disable() and preempt_enable() calls.
>

flush_map() is doing a wbinvd:

> -static void flush_kernel_map(void *dummy)
> -{
> - /* Could use CLFLUSH here if the CPU supports it (Hammer,P4) */
> - if (boot_cpu_data.x86_model >= 4)
> - asm volatile("wbinvd":::"memory");

It appears that by replacing flush_map() with flush_tlb_all(), you are no
longer doing this?

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Feb 23 2003 - 22:00:31 EST