On Wed, Apr 25, 2012 at 04:30:19PM +0800, Peter Teoh wrote: > Fixing the mismatch in signed and unsigned type assignment, which > potentially can lead to integer overflow bug. > > Thanks. > > Reviewed-by: Minchan Kim <minchan@xxxxxxxxxx> > Signed-off-by: Peter Teoh <htmldeveloper@xxxxxxxxx> All indexes in memblock are integers. Changing that particular one to unsigned int doesn't fix anything. I think it just makes things more confusing. If there ever are cases w/ more then 2G memblocks, we're going for 64bit not unsigned. Thanks. -- tejun -- 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>