Re: [PATCH] fs: fcntl_setlease defies lease_init assumptions

From: Martin J. Bligh
Date: Mon May 08 2006 - 23:48:53 EST


Christoph Lameter wrote:
On Mon, 8 May 2006, Pekka Enberg wrote:


I think it sounds like it's worth it, but I'm not going to really push it.

Sounds good to me. Andrew?


virt_to_page is not cheap on NUMA.

On IA64 virt_to_page is:

#define virt_to_page(kaddr) pfn_to_page(__pa(kaddr) >> PAGE_SHIFT)

# define pfn_to_page(pfn) (vmem_map + (pfn))

vmem_map is not a linear map but a virtual mapping that may require several faults to get the information.

Can't you use sparsemem instead? It solves the same problem without the
magic faulting, doesn't it?

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