Re: [PATCH 3/6] udf: update inode's iblock count when allocating new block

2012-10-12 Thread Namjae Jeon
2012/10/12 Jan Kara :
> On Wed 10-10-12 00:09:28, Namjae Jeon wrote:
>> From: Namjae Jeon 
>>
>> 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 
>> Signed-off-by: Ashish Sangwan 
>> ---
>>  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 
> SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 3/6] udf: update inode's iblock count when allocating new block

2012-10-12 Thread Namjae Jeon
2012/10/12 Jan Kara j...@suse.cz:
 On Wed 10-10-12 00:09:28, Namjae Jeon wrote:
 From: Namjae Jeon namjae.j...@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.j...@samsung.com
 Signed-off-by: Ashish Sangwan a.sang...@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 j...@suse.cz
 SUSE Labs, CR
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 3/6] udf: update inode's iblock count when allocating new block

2012-10-11 Thread Jan Kara
On Wed 10-10-12 00:09:28, Namjae Jeon wrote:
> From: Namjae Jeon 
> 
> 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 
> Signed-off-by: Ashish Sangwan 
> ---
>  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 
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 3/6] udf: update inode's iblock count when allocating new block

2012-10-11 Thread Jan Kara
On Wed 10-10-12 00:09:28, Namjae Jeon wrote:
 From: Namjae Jeon namjae.j...@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.j...@samsung.com
 Signed-off-by: Ashish Sangwan a.sang...@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 j...@suse.cz
SUSE Labs, CR
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 3/6] udf: update inode's iblock count when allocating new block

2012-10-09 Thread Namjae Jeon
From: Namjae Jeon 

Currently, du will show wrong block count. This patch fix it.

Signed-off-by: Namjae Jeon 
Signed-off-by: Ashish Sangwan 
---
 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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 3/6] udf: update inode's iblock count when allocating new block

2012-10-09 Thread Namjae Jeon
From: Namjae Jeon namjae.j...@samsung.com

Currently, du will show wrong block count. This patch fix it.

Signed-off-by: Namjae Jeon namjae.j...@samsung.com
Signed-off-by: Ashish Sangwan a.sang...@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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/