Re: [LTP] [PATCH v4 4/4] lapi/stat.h: Remove deprecated STATX_ALL macro

["Yang Xu (Fujitsu)" <xuyang2018.jy@fujitsu.com>]

on 2023/04/27 5:56, Eric Biggers wrote:
> On Thu, Apr 06, 2023 at 01:40:22PM +0800, Yang Xu wrote:
>> diff --git a/testcases/kernel/syscalls/statx/statx07.c b/testcases/kernel/syscalls/statx/statx07.c
>> index b13c11f72..c798c7a10 100644
>> --- a/testcases/kernel/syscalls/statx/statx07.c
>> +++ b/testcases/kernel/syscalls/statx/statx07.c
>> @@ -62,15 +62,15 @@ static int get_mode(char *file_name, int flag_type, char *flag_name)
>>   {
>>   	struct statx buf;
>>   
>> -	TEST(statx(AT_FDCWD, file_name, flag_type, STATX_ALL, &buf));
>> +	TEST(statx(AT_FDCWD, file_name, flag_type, STATX_BASIC_STATS | STATX_BTIME, &buf));
>>   
>>   	if (TST_RET == -1) {
>>   		tst_brk(TFAIL | TST_ERR,
>> -			"statx(AT_FDCWD, %s, %s, STATX_ALL, &buf)",
>> +			"statx(AT_FDCWD, %s, %s, STATX_BASIC_STATS | STATX_BTIME, &buf)",
>>   			file_name, flag_name);
>>   	}
>>   
>> -	tst_res(TINFO, "statx(AT_FDCWD, %s, %s, STATX_ALL, &buf) = %o",
>> +	tst_res(TINFO, "statx(AT_FDCWD, %s, %s, STATX_BASIC_STATS | STATX_BTIME, &buf) = %o",
>>   		file_name, flag_name, buf.stx_mode);
>>   
>>   	return buf.stx_mode;
> 
> Looks like this place just wants STATX_BASIC_STATS.

Yes, I will only use STATX_BASIC_STATS for statx07.c

Best Regards
Yang Xu
> 
> - Eric

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp