[Ocfs2-tools-devel] [PATCH 4/5] Convert disabling of sparse feature to use new code in libocfs2

Jan Kara jack at suse.cz
Sun Oct 18 23:55:55 PDT 2009


On Fri 16-10-09 16:16:54, Tristan wrote:
> Jan Kara wrote:
>> Make code handling update of quota information when disabling
>> sparse feature use new generic code in libocfs2.
>>
>> Signed-off-by: Jan Kara <jack at suse.cz>
>> ---
>>  tunefs.ocfs2/feature_sparse_files.c |  100 ++++++----------------------------
>>  1 files changed, 18 insertions(+), 82 deletions(-)
>>
>> diff --git a/tunefs.ocfs2/feature_sparse_files.c b/tunefs.ocfs2/feature_sparse_files.c
>> index d4dff34..4f8c0db 100644
>> --- a/tunefs.ocfs2/feature_sparse_files.c
>> +++ b/tunefs.ocfs2/feature_sparse_files.c
>> @@ -448,8 +448,6 @@ static errcode_t fill_sparse_files(ocfs2_filesys *fs,
>>  	struct list_head *pos;
>>  	struct sparse_file *file;
>>  	struct tools_progress *prog;
>>   
>
>> -	struct ocfs2_super_block *super = OCFS2_RAW_SB(fs->fs_super);
>>   
> Please don't remove this variable though it will not be used in this patch 
> anymore, your following patch(patch5) however will use this structure 
> again, see:
>
> [PATCH 5/5] Don't account space used for system files when disabling sparse 
> feature
  Thanks for spotting this. Fixed. Since I've got no further comments I'm
reposting the series and hopefully it can be merged.

									Honza
-- 
Jan Kara <jack at suse.cz>
SUSE Labs, CR



More information about the Ocfs2-tools-devel mailing list