[LTP] [PATCH 3/3] syscalls/preadv202: Add new testcase
Xiao Yang
yangx.jy@cn.fujitsu.com
Fri Oct 5 03:17:56 CEST 2018
Hi Cyril,
Thank you to review the whole patch set.
On 2018/10/04 20:19, Cyril Hrubis wrote:
> Hi!
>> Note:
>> From preadv2(2) manpage, preadv2() with invalid flag should return
>> EINVAL, but it actually returned EOPNOTSUPP in current upstream
>> kernel, as below:
>> ---------------------------------------------------------------
>> include/uapi/linux/fs.h:
>> define RWF_SUPPORTED (RWF_HIPRI | RWF_DSYNC | RWF_SYNC | RWF_NOWAIT |\
>> RWF_APPEND)
>> ...
>>
>> include/linux/fs.h:
>> static inline int kiocb_set_rw_flags(struct kiocb *ki, rwf_t flags)
>> {
>> if (unlikely(flags& ~RWF_SUPPORTED)) {
>> return -EOPNOTSUPP;
>> }
>> ...
>> ---------------------------------------------------------------
>>
>> We use EOPNOTSUPP as expected errno for the time being.
> This is probably worth a patch for the man-pages project, will you take
> care of that?
I will try to send a fix patch to the man-pages project as you advised.
Thanks,
Xiao Yang
> Pushed with a minor change, I've changed the EFAULT test to get address
> from tst_get_bad_addr() instead of the hardcoded -1.
>
> Thanks!
>
More information about the ltp
mailing list