Hi, On Thu, Dec 30, 2010 at 11:26:11AM -0500, Konrad Rzeszutek Wilk wrote: > > -static int increase_reservation(unsigned long nr_pages) > > +static enum bp_state increase_reservation(unsigned long nr_pages) > > { > > + enum bp_state state = BP_DONE; > > + int rc; > > unsigned long pfn, i, flags; > > struct page *page; > > - long rc; > > How come? Is it just a cleanup? I forgot to move it to separate patch. When I was working on protection before CPU exhaust I discovered that HYPERVISOR_memory_op() returns int and rc could be declared as int not as long. Daniel -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>