Messages in this thread | | | Date | Wed, 26 Jul 2006 19:59:31 -0400 | From | Dave Jones <> | Subject | Re: [PATCH] vm/agp: remove private page protection map |
| |
On Thu, Jul 27, 2006 at 12:24:27AM +0100, Dave Airlie wrote: > > > >I'm happy with the intent of your vm_get_page_prot() patch (and would > >like to extend it to other places after, minimizing references to the > >protection_map[]). But there's a few aspects which distress me - the > >u8 type nowhere else in mm, the requirement that caller mask the arg, > >agp_convert_mmap_flags still using its own conversion from PROT_ to VM_ > >while there's an inline in mm.h (though why someone thought to optimize > >and so obscure that version puzzles me!). Would you be happy to insert > >your Sign-off in the replacement below? > > No worries, I think davej can drop my one from his tree as well and take > this..
Done, and pushed out to agpgart.git
Thanks,
Dave
-- http://www.codemonkey.org.uk - 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/
| |