> +static int __init readahead(char *str) > +{ > + if (!str) > + return -EINVAL; > + vm_max_readahead_kb = memparse(str, &str) / 1024ULL; Just wondering, shouldn't you check whether the str had a valid value [memparse (str, &next); next > str ..] and if it didn't, then use the DEFAULT_VM_MAX_READAHEAD ? Otherwise, incase of a invalid value, the readahead value will become zero. > + default_backing_dev_info.ra_pages = vm_max_readahead_kb > + * 1024 / PAGE_CACHE_SIZE; > + return 0; > +} > + > +early_param("readahead", readahead); > + -Ankit -- 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/ . Don't email: <a href