Re: [PATCH] madvise() against 2.3.52-3

Hi,

On Tue, 14 Mar 2000 18:13:19 -0500 (EST), Chuck Lever [off-list ref]
said:

+	lock_kernel();	/* is this really necessary? */
+
+	flush_cache_range(vma->vm_mm, start, end);
+	zap_page_range(vma->vm_mm, start, end - start);
+	flush_tlb_range(vma->vm_mm, start, end);
+
+	unlock_kernel();

I'd have thought we'd be safe without it --- the zap_page_range()
already takes the page table lock,  The flush_tlb_range should be safe
on SMP without the kernel lock now, shouldn't it?  We certainly take a
tlb lock on Intel to guard this --- I'm not sure if it's 100% safe on
other architectures.

--Stephen
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux.eu.org/Linux-MM/