[Ocfs2-devel] [PATCH] ocfs2: clean up two unused functions in suballoc.c

[Changwei Ge]

Hi Gang,

On 2018/2/26 13:40, Gang He wrote:
> Hi Changwei,
> 
> The code change looks OK, but maybe these code is still useful for temporary debugging?
True, they could be used for temporary debugging.
If other developers also take them as a potential debug methods for future use, 
I'd like to give up this patch.

Thanks,
Changwei

> 
> 
> Thanks
> Gang
> 
> 
>>>>
>> The two functions are no longer used.
>>
>> Signed-off-by: Changwei Ge <ge.changwei at h3c.com>
>> ---
>>   fs/ocfs2/suballoc.c | 49 -------------------------------------------------
>>   1 file changed, 49 deletions(-)
>>
>> diff --git a/fs/ocfs2/suballoc.c b/fs/ocfs2/suballoc.c
>> index 9f0b95a..328be8b 100644
>> --- a/fs/ocfs2/suballoc.c
>> +++ b/fs/ocfs2/suballoc.c
>> @@ -79,8 +79,6 @@ static u64 ocfs2_group_from_res(struct
>> ocfs2_suballoc_result *res)
>>   	return ocfs2_which_suballoc_group(res->sr_blkno, res->sr_bit_offset);
>>   }
>>   
>> -static inline void ocfs2_debug_bg(struct ocfs2_group_desc *bg);
>> -static inline void ocfs2_debug_suballoc_inode(struct ocfs2_dinode *fe);
>>   static inline u16 ocfs2_find_victim_chain(struct ocfs2_chain_list *cl);
>>   static int ocfs2_block_group_fill(handle_t *handle,
>>   				  struct inode *alloc_inode,
>> @@ -2624,53 +2622,6 @@ int ocfs2_release_clusters(handle_t *handle,
>>   				    _ocfs2_clear_bit);
>>   }
>>   
>> -static inline void ocfs2_debug_bg(struct ocfs2_group_desc *bg)
>> -{
>> -	printk("Block Group:\n");
>> -	printk("bg_signature:       %s\n", bg->bg_signature);
>> -	printk("bg_size:            %u\n", bg->bg_size);
>> -	printk("bg_bits:            %u\n", bg->bg_bits);
>> -	printk("bg_free_bits_count: %u\n", bg->bg_free_bits_count);
>> -	printk("bg_chain:           %u\n", bg->bg_chain);
>> -	printk("bg_generation:      %u\n", le32_to_cpu(bg->bg_generation));
>> -	printk("bg_next_group:      %llu\n",
>> -	       (unsigned long long)bg->bg_next_group);
>> -	printk("bg_parent_dinode:   %llu\n",
>> -	       (unsigned long long)bg->bg_parent_dinode);
>> -	printk("bg_blkno:           %llu\n",
>> -	       (unsigned long long)bg->bg_blkno);
>> -}
>> -
>> -static inline void ocfs2_debug_suballoc_inode(struct ocfs2_dinode *fe)
>> -{
>> -	int i;
>> -
>> -	printk("Suballoc Inode %llu:\n", (unsigned long long)fe->i_blkno);
>> -	printk("i_signature:                  %s\n", fe->i_signature);
>> -	printk("i_size:                       %llu\n",
>> -	       (unsigned long long)fe->i_size);
>> -	printk("i_clusters:                   %u\n", fe->i_clusters);
>> -	printk("i_generation:                 %u\n",
>> -	       le32_to_cpu(fe->i_generation));
>> -	printk("id1.bitmap1.i_used:           %u\n",
>> -	       le32_to_cpu(fe->id1.bitmap1.i_used));
>> -	printk("id1.bitmap1.i_total:          %u\n",
>> -	       le32_to_cpu(fe->id1.bitmap1.i_total));
>> -	printk("id2.i_chain.cl_cpg:           %u\n", fe->id2.i_chain.cl_cpg);
>> -	printk("id2.i_chain.cl_bpc:           %u\n", fe->id2.i_chain.cl_bpc);
>> -	printk("id2.i_chain.cl_count:         %u\n", fe->id2.i_chain.cl_count);
>> -	printk("id2.i_chain.cl_next_free_rec: %u\n",
>> -	       fe->id2.i_chain.cl_next_free_rec);
>> -	for(i = 0; i < fe->id2.i_chain.cl_next_free_rec; i++) {
>> -		printk("fe->id2.i_chain.cl_recs[%d].c_free:  %u\n", i,
>> -		       fe->id2.i_chain.cl_recs[i].c_free);
>> -		printk("fe->id2.i_chain.cl_recs[%d].c_total: %u\n", i,
>> -		       fe->id2.i_chain.cl_recs[i].c_total);
>> -		printk("fe->id2.i_chain.cl_recs[%d].c_blkno: %llu\n", i,
>> -		       (unsigned long long)fe->id2.i_chain.cl_recs[i].c_blkno);
>> -	}
>> -}
>> -
>>   /*
>>    * For a given allocation, determine which allocators will need to be
>>    * accessed, and lock them, reserving the appropriate number of bits.
>> -- 
>> 2.7.4
>>
>>
>> _______________________________________________
>> Ocfs2-devel mailing list
>> Ocfs2-devel at oss.oracle.com
>> https://oss.oracle.com/mailman/listinfo/ocfs2-devel
> 
>