Re: [PATCH 2/4] x86: ioremap: fix physical address check

From: H. Peter Anvin
Date: Mon Jun 14 2010 - 14:37:26 EST


On 06/14/2010 04:06 AM, Kenji Kaneshige wrote:
> Index: linux-2.6.34/include/linux/vmalloc.h
> ===================================================================
> --- linux-2.6.34.orig/include/linux/vmalloc.h
> +++ linux-2.6.34/include/linux/vmalloc.h
> @@ -30,7 +30,7 @@ struct vm_struct {
> unsigned long flags;
> struct page **pages;
> unsigned int nr_pages;
> - unsigned long phys_addr;
> + phys_addr_t phys_addr;
> void *caller;
> };

This really doesn't look right at all. If this was required then it
would seem that anything using high addresses would be broken... as such
I can only assume it matters only for lowmem pages...

-hpa
--
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/