* [PATCH 3/6] udf: update inode's iblock count when allocating new block
@ 2012-10-09 15:09 Namjae Jeon
2012-10-11 17:54 ` Jan Kara
0 siblings, 1 reply; 3+ messages in thread
From: Namjae Jeon @ 2012-10-09 15:09 UTC (permalink / raw)
To: jack; +Cc: linux-kernel, Namjae Jeon, Namjae Jeon, Ashish Sangwan
From: Namjae Jeon <namjae.jeon@samsung.com>
Currently, du will show wrong block count. This patch fix it.
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Ashish Sangwan <a.sangwan@samsung.com>
---
fs/udf/inode.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/fs/udf/inode.c b/fs/udf/inode.c
index 8266f2e..76a1f57 100644
--- a/fs/udf/inode.c
+++ b/fs/udf/inode.c
@@ -775,6 +775,8 @@ static sector_t inode_getblk(struct inode *inode, sector_t block,
if (isBeyondEOF)
iinfo->i_lenExtents += inode->i_sb->s_blocksize;
}
+ /* update inode's iblocks */
+ inode->i_blocks += (1 << (inode->i_sb->s_blocksize_bits - 9));
/* if the extent the requsted block is located in contains multiple
* blocks, split the extent into at most three extents. blocks prior
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 3/6] udf: update inode's iblock count when allocating new block
2012-10-09 15:09 [PATCH 3/6] udf: update inode's iblock count when allocating new block Namjae Jeon
@ 2012-10-11 17:54 ` Jan Kara
2012-10-12 12:59 ` Namjae Jeon
0 siblings, 1 reply; 3+ messages in thread
From: Jan Kara @ 2012-10-11 17:54 UTC (permalink / raw)
To: Namjae Jeon; +Cc: jack, linux-kernel, Namjae Jeon, Ashish Sangwan
On Wed 10-10-12 00:09:28, Namjae Jeon wrote:
> From: Namjae Jeon <namjae.jeon@samsung.com>
>
> Currently, du will show wrong block count. This patch fix it.
This should be fixed by commit fd4287dbc07763595c1e74edbb6520092290185c.
Have you tested with newish kernel? If recent kernel still has an issue,
can you provide a test case? Thanks!
Honza
>
> Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
> Signed-off-by: Ashish Sangwan <a.sangwan@samsung.com>
> ---
> fs/udf/inode.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/fs/udf/inode.c b/fs/udf/inode.c
> index 8266f2e..76a1f57 100644
> --- a/fs/udf/inode.c
> +++ b/fs/udf/inode.c
> @@ -775,6 +775,8 @@ static sector_t inode_getblk(struct inode *inode, sector_t block,
> if (isBeyondEOF)
> iinfo->i_lenExtents += inode->i_sb->s_blocksize;
> }
> + /* update inode's iblocks */
> + inode->i_blocks += (1 << (inode->i_sb->s_blocksize_bits - 9));
>
> /* if the extent the requsted block is located in contains multiple
> * blocks, split the extent into at most three extents. blocks prior
> --
> 1.7.9.5
>
--
Jan Kara <jack@suse.cz>
SUSE Labs, CR
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 3/6] udf: update inode's iblock count when allocating new block
2012-10-11 17:54 ` Jan Kara
@ 2012-10-12 12:59 ` Namjae Jeon
0 siblings, 0 replies; 3+ messages in thread
From: Namjae Jeon @ 2012-10-12 12:59 UTC (permalink / raw)
To: Jan Kara; +Cc: linux-kernel, Namjae Jeon, Ashish Sangwan
2012/10/12 Jan Kara <jack@suse.cz>:
> On Wed 10-10-12 00:09:28, Namjae Jeon wrote:
>> From: Namjae Jeon <namjae.jeon@samsung.com>
>>
>> Currently, du will show wrong block count. This patch fix it.
> This should be fixed by commit fd4287dbc07763595c1e74edbb6520092290185c.
> Have you tested with newish kernel? If recent kernel still has an issue,
> can you provide a test case? Thanks!
Hi. Jan.
You're right. There is no issue with latest kernel.
Plz ignore this patch.
Thanks a lot!
>
> Honza
>
>>
>> Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
>> Signed-off-by: Ashish Sangwan <a.sangwan@samsung.com>
>> ---
>> fs/udf/inode.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/fs/udf/inode.c b/fs/udf/inode.c
>> index 8266f2e..76a1f57 100644
>> --- a/fs/udf/inode.c
>> +++ b/fs/udf/inode.c
>> @@ -775,6 +775,8 @@ static sector_t inode_getblk(struct inode *inode, sector_t block,
>> if (isBeyondEOF)
>> iinfo->i_lenExtents += inode->i_sb->s_blocksize;
>> }
>> + /* update inode's iblocks */
>> + inode->i_blocks += (1 << (inode->i_sb->s_blocksize_bits - 9));
>>
>> /* if the extent the requsted block is located in contains multiple
>> * blocks, split the extent into at most three extents. blocks prior
>> --
>> 1.7.9.5
>>
> --
> Jan Kara <jack@suse.cz>
> SUSE Labs, CR
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-10-12 12:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-09 15:09 [PATCH 3/6] udf: update inode's iblock count when allocating new block Namjae Jeon
2012-10-11 17:54 ` Jan Kara
2012-10-12 12:59 ` Namjae Jeon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.