Hi Wanlong Gao,

Thanks for your ACK. But I can't find this patch on upstream ltp repo until now.
Should I re-send the acked patch to you?

Thank you,
Zorro Lang

----- 原始邮件 -----
发件人: "Wanlong Gao" <gaowanl...@cn.fujitsu.com>
收件人: "Zorro Lang" <zl...@redhat.com>
抄送: "Xiaoguang Wang" <wangxg.f...@cn.fujitsu.com>, 
ltp-list@lists.sourceforge.net
发送时间: 星期三, 2014年 10 月 15日 下午 4:12:11
主题: Re: [LTP] [PATCH] ext4-new-features: modify ext4_get_inode_version.sh not 
know $EXT4_DEV definition

On 10/14/2014 07:30 PM, Zorro Lang wrote:
> 在 2014年10月14日 10:55, Xiaoguang Wang 写道:
>> Hi,
>>
>> On 10/13/2014 11:04 PM, Zorro Lang wrote:
>>> ext4_get_inode_version.sh use $EXT4_DEV, but it really doesn't know what is 
>>> $EXT4_DEV.
>>> That cause other cases which use ext4_get_inode_version.sh to fail.
>>>
>>> Signed-off-by: Zorro Lang <zl...@redhat.com>
>>> ---
>>>
>>> Hi,
>>>
>>> When I ran ext4-inode-version, I always hit TFAIL. But I can't reproduce 
>>> the failure manually.
>>> Then I found ext4_inode_version_test.sh use ext4_get_inode_version.sh ->
>>> ext4_get_inode_version.sh use $EXT4_DEV ->
>>> but ext4_get_inode_version.sh doesn't know what is $EXT4_DEV.
>> Yeah, this is a bug :)
>>> About this patch, I export $EXT4_DEV in ext4_inode_version_test.sh, before 
>>> it call ext4_get_inode_version.sh.
>>> But I think add the "export" in front of "EXT4_DEV=$LTP_BIG_DEV" in 
>>> ext4_funcs.sh will be another method.
>> I think adding "export EXT4_DEV=$LTP_BIG_DEV" in ext4_funcs.sh would be 
>> better.
> Hi xiaoguang,
> 
> Do you mean patch likes below is better?
> 
> diff --git a/testcases/kernel/fs/ext4-new-features/ext4_funcs.sh 
> b/testcases/kernel/fs/ext4-new-features/ext4_funcs.sh
> index 0ae534a..b51e942 100755
> --- a/testcases/kernel/fs/ext4-new-features/ext4_funcs.sh
> +++ b/testcases/kernel/fs/ext4-new-features/ext4_funcs.sh
> @@ -45,7 +45,7 @@ ext4_setup()
>          if [ -z "$LTP_BIG_DEV" ];then
>                  tst_brkm TCONF "tests need a big block device(5G-10G)"
>          else
> -               EXT4_DEV=$LTP_BIG_DEV
> +               export EXT4_DEV=$LTP_BIG_DEV

Acked-by: Wanlong Gao <gaowanl...@cn.fujitsu.com>

>          fi
> 
>          tst_tmpdir
> 
> 
> If you want to merge this patch, I will resend it as the better way.
> 
> Thank you,
> Zorro Lang
>>
>> Regards,
>> Xiaoguang Wang
>>
>>> Please review, and tell me which one do you think is better for LTP? Or you 
>>> have a better one;)
>>>
>>> Thank you,
>>> Zorro Lang
>>>
>>>   .../fs/ext4-new-features/ext4-inode-version/ext4_inode_version_test.sh  | 
>>> 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git 
>>> a/testcases/kernel/fs/ext4-new-features/ext4-inode-version/ext4_inode_version_test.sh
>>>  
>>> b/testcases/kernel/fs/ext4-new-features/ext4-inode-version/ext4_inode_version_test.sh
>>> index ac8f021..2549088 100755
>>> --- 
>>> a/testcases/kernel/fs/ext4-new-features/ext4-inode-version/ext4_inode_version_test.sh
>>> +++ 
>>> b/testcases/kernel/fs/ext4-new-features/ext4-inode-version/ext4_inode_version_test.sh
>>> @@ -138,6 +138,8 @@ test_inode_version()
>>>   
>>>   # main
>>>   ext4_setup
>>> +# export EXT4_DEV for ext4_get_inode_version.sh can know it
>>> +export EXT4_DEV=$EXT4_DEV
>>>   
>>>   ext4_test_128_inode_version
>>>   
>>>
> 
> 
> ------------------------------------------------------------------------------
> Comprehensive Server Monitoring with Site24x7.
> Monitor 10 servers for $9/Month.
> Get alerted through email, SMS, voice calls or mobile push notifications.
> Take corrective actions from your mobile device.
> http://p.sf.net/sfu/Zoho
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
> 


------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://p.sf.net/sfu/Zoho
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to