[Ocfs2-devel] 答复: [PATCH] ocfs2: Correct the offset comments of the structure ocfs2_dir_block_trailer.

[Changwei Ge]

Hi Zhonghua and Jospeh,

I'm afraid that it can't be easy to find the offest 0x20 since 
::db_signature only occupy 8 bytes.

Thanks,
Changwei

On 2018/5/9 10:50, Guozhonghua wrote:
> Good Idea, I will send patch v2 for review.
>
> Thanks.
>
> Guozhonghua.
>
> -----邮件原件-----
> 发件人: Joseph Qi [mailto:jiangqi903 at gmail.com]
> 发送时间: 2018年5月9日 9:01
> 收件人: guozhonghua (Cloud); Andrew Morton (akpm at linux-foundation.org)
> 抄送: 'ocfs2-devel at oss.oracle.com' (ocfs2-devel at oss.oracle.com)
> 主题: Re: [Ocfs2-devel] [PATCH] ocfs2: Correct the offset comments of the structure ocfs2_dir_block_trailer.
>
> Umm... We always explicitly comment out the shift of 0x10, 0x20, ..., IMO, we'd better move the comments to the correct place instead of change it to something like 0x28.
>
> Thanks,
> Joseph
>
> On 18/5/8 17:46, Guozhonghua wrote:
>> Correct the offset comments of the structure ocfs2_dir_block_trailer.
>>
>> Signed-off-by: guozhonghua <guozhonghua at h3c.com>
>> ---
>>   fs/ocfs2/ocfs2_fs.h |    4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/fs/ocfs2/ocfs2_fs.h b/fs/ocfs2/ocfs2_fs.h index
>> 5bb4a89..14c60b0 100644
>> --- a/fs/ocfs2/ocfs2_fs.h
>> +++ b/fs/ocfs2/ocfs2_fs.h
>> @@ -808,10 +808,10 @@ struct ocfs2_dir_block_trailer {
>>   /*10*/	__u8		db_signature[8];	/* Signature for verification */
>>   	__le64		db_reserved2;
>>   	__le64		db_free_next;		/* Next block in list (unused) */
>> -/*20*/	__le64		db_blkno;		/* Offset on disk, in blocks */
>> +/*28*/	__le64		db_blkno;		/* Offset on disk, in blocks */
>>   	__le64		db_parent_dinode;	/* dinode which owns me, in
>>   						   blocks */
>> -/*30*/	struct ocfs2_block_check db_check;	/* Error checking */
>> +/*38*/	struct ocfs2_block_check db_check;	/* Error checking */
>>   /*40*/
>>   };
>>   
>> --
>> 1.7.9.5
>>
>> _______________________________________________
>> Ocfs2-devel mailing list
>> Ocfs2-devel at oss.oracle.com
>> https://oss.oracle.com/mailman/listinfo/ocfs2-devel
>>
> _______________________________________________
> Ocfs2-devel mailing list
> Ocfs2-devel at oss.oracle.com
> https://oss.oracle.com/mailman/listinfo/ocfs2-devel