Re: [V3 PATCH 1/2] tmpfs: add fallocate support
From: KOSAKI Motohiro
Date: Wed Nov 23 2011 - 21:47:07 EST
>> + while (index < end) {
>> + ret = shmem_getpage(inode, index, &page, SGP_WRITE, NULL);
>
> If the 'page' for index exists before this call, this will return the page without
> allocaton.
>
> Then, the page may not be zero-cleared. I think the page should be zero-cleared.
No. fallocate shouldn't destroy existing data. It only ensure
subsequent file access don't make ENOSPC error.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/