Petr Vorel

Results 108 comments of Petr Vorel
trafficstars

@duncs maybe having docker based CI would reflect the reality better. See examples: https://github.com/linux-test-project/ltp/actions https://github.com/iputils/iputils/actions (both migrated from travis to github actions)

Thanks for info, I'll have a look. I guess ntirpc goal to be replacement for libtirpc, right? But is it also API compatible (is it drop-in replacement without requiring the...

Thanks for a detailed table. @metan-ucw @mdoucha know more than me. AFAIK we noticed at least incorrect errno `EISDIR` (obviously there is also `ENODATA`) and hang on `TST_FD_EPOLL` (I did...

Yeah, thanks! https://github.com/linux-test-project/ltp/tree/master/testcases/kernel/syscalls/sync_file_range/sync_file_range02.c

Indeed 3 testcases from 2018 (https://github.com/linux-test-project/ltp/tree/master/testcases/kernel/syscalls/tgkill/). I wonder if anything is missing.

We merge improvement (hopefully) in 66517b89141fc455ed807f3b95e5260dcf9fb90f, but there are still some TODO from Jan Kara (https://lore.kernel.org/ltp/20240528095030.valplwc5t3m3betn@quack3/): * Use at least direct IO to verify the data is stored on disk...

At least verification for storing data on the disc was added in 21be0b196e7a79b18a5094d0083307f77654c660. @metan-ucw Do we want to have this open for looking into abort fs or we are done?...

You are using old LTP. Although mq_open01.c was not modified since (20230929), generally it's recommended to use the latest release or current master branch. It's hard knowing without the details...

You put a lot of info in PR description but no info in git commit message. Could you please extend it? Also, it'd be worth to mention in the commit...

Thanks for having a look, no rush with the PR. Sure, I'll modify it. Also code contains bug, which needs to be fixed: https://github.com/iputils/iputils/pull/561/commits/988caddd5b5d27bbcb88fc0877fea015e1bb8f62. I'll have look into it next...