lkml.org 
[lkml]   [2017]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 07/12 linux-next] udf: store allocation offset in udf_prealloc_extents()
On Fri 06-01-17 21:53:55, Fabian Frederick wrote:
> recalculate offset if needed.
>
> Signed-off-by: Fabian Frederick <fabf@skynet.be>

IMHO not really worth it since numalloc gets updated and forgetting to
update the byte variant is just too easy. Skipped.

Honza

> ---
> fs/udf/inode.c | 21 +++++++++------------
> 1 file changed, 9 insertions(+), 12 deletions(-)
>
> diff --git a/fs/udf/inode.c b/fs/udf/inode.c
> index f563e97..66514ad 100644
> --- a/fs/udf/inode.c
> +++ b/fs/udf/inode.c
> @@ -998,10 +998,10 @@ static void udf_prealloc_extents(struct inode *inode, int c, int lastblock,
> length : UDF_DEFAULT_PREALLOC_BLOCKS) -
> currlength);
> if (numalloc) {
> + loff_t pos = numalloc << inode->i_sb->s_blocksize_bits;
> +
> if (start == (c + 1))
> - laarr[start].extLength +=
> - (numalloc <<
> - inode->i_sb->s_blocksize_bits);
> + laarr[start].extLength += pos;
> else {
> memmove(&laarr[c + 2], &laarr[c + 1],
> sizeof(struct long_ad) * (*endnum - (c + 1)));
> @@ -1011,9 +1011,7 @@ static void udf_prealloc_extents(struct inode *inode, int c, int lastblock,
> laarr[c].extLocation.
> partitionReferenceNum;
> laarr[c + 1].extLength =
> - EXT_NOT_RECORDED_ALLOCATED |
> - (numalloc <<
> - inode->i_sb->s_blocksize_bits);
> + EXT_NOT_RECORDED_ALLOCATED | pos;
> start = c + 1;
> }
>
> @@ -1024,12 +1022,12 @@ static void udf_prealloc_extents(struct inode *inode, int c, int lastblock,
> inode->i_sb->s_blocksize_bits;
>
> if (elen > numalloc) {
> - laarr[i].extLength -=
> - (numalloc <<
> - inode->i_sb->s_blocksize_bits);
> - numalloc = 0;
> + laarr[i].extLength -= pos;
> + numalloc = pos = 0;
> } else {
> numalloc -= elen;
> + pos = numalloc <<
> + inode->i_sb->s_blocksize_bits;
> if (*endnum > (i + 1))
> memmove(&laarr[i],
> &laarr[i + 1],
> @@ -1039,8 +1037,7 @@ static void udf_prealloc_extents(struct inode *inode, int c, int lastblock,
> (*endnum)--;
> }
> }
> - UDF_I(inode)->i_lenExtents +=
> - numalloc << inode->i_sb->s_blocksize_bits;
> + UDF_I(inode)->i_lenExtents += pos;
> }
> }
> }
> --
> 2.7.4
>
>
--
Jan Kara <jack@suse.com>
SUSE Labs, CR

\
 
 \ /
  Last update: 2017-01-10 11:54    [W:0.134 / U:0.540 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site