On Mon, 15 Jul 2013, Chen Gang wrote: > On 07/12/2013 09:49 PM, Christoph Lameter wrote: > > On Fri, 12 Jul 2013, Chen Gang wrote: > > > >> Since alloc_loc_track() will alloc additional space, and already knows > >> about 'max', so need be sure of 'max' must be larger than 't->count'. > > > > alloc_loc_track is only called if t->count > max from add_location: > > > > For add_location(), if "t->count > t->max", it calls alloc_loc_track() > with "max == 2 * t->max". > > In this case we need be sure that "t->count < 2 * t->max". We are sure about that since t->count is always incremented by one and then checked against t->max. The location database is build up from a single hardware thread without any concurrency. So we do not really need this patch. -- 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/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>