> > + if (!bdi->dirty_exceeded) > > + ratelimit = current->nr_dirtied_pause; > > + else > > + ratelimit = min(ratelimit, 32 >> (PAGE_SHIFT - 10)); > > Usage of ratelimit before init? > > Maybe: > > ratelimit = current->nr_dirtied_pause; > if (bdi->dirty_exceeded) > ratelimit = min(ratelimit, 32 >> (PAGE_SHIFT - 10)); Good catch, thanks! That's indeed the original form. I changed it to make the code more aligned... Thanks, Fengguang -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>