> > + 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 from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html