[LTP] [PATCH] syscalls/fgetxattr02: Use loop instead of RAM disk
Yang Xu
xuyang2018.jy@cn.fujitsu.com
Thu Feb 20 11:31:19 CET 2020
Hi
> Hi!
>>> which makes this test fail with ENXIO when we attempt to open the block
>>> device in the test setup.
>>>
>>> LTP depends on heavily on loop device driver already so it makes sense
>>> to switch over to a loop device backed block device instead.
>>>
>>> Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
>>> ---
>>> testcases/kernel/syscalls/fgetxattr/fgetxattr02.c | 7 ++++---
>>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/testcases/kernel/syscalls/fgetxattr/fgetxattr02.c b/testcases/kernel/syscalls/fgetxattr/fgetxattr02.c
>>> index 02e81810a..82fb676be 100644
>>> --- a/testcases/kernel/syscalls/fgetxattr/fgetxattr02.c
>>> +++ b/testcases/kernel/syscalls/fgetxattr/fgetxattr02.c
>>> @@ -210,7 +210,8 @@ static void setup(void)
>>> size_t i = 0;
>>> struct sockaddr_un sun;
>>>
>>> - dev_t dev = makedev(1, 3);
>>> + dev_t chr_dev = makedev(1, 3);
>>> + dev_t blk_dev = makedev(7, 3);
>> Can we use tst_find_free_loopdev to avoid a fixed loop dev like
>> copy_file_range02.c?
>
> I do not think that it matters here, we are not actually touching the
> block device here, we just need to be able to open the block device so
> that we can add an attribute to the file we have created. I does not
> matter if it's used or not.
Sound reasonable, feel free to add
Reviewed-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
>
More information about the ltp
mailing list