<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 18, 2020 at 4:25 PM Viresh Kumar <<a href="mailto:viresh.kumar@linaro.org" target="_blank">viresh.kumar@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 18-02-20, 09:15, Li Wang wrote:<br>
> On Mon, Feb 17, 2020 at 9:36 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>> wrote:<br>
> <br>
> > ...<br>
> > > > +static struct tst_test test = {<br>
> > > > +       .min_kver = "5.2",<br>
> > > ><br>
> > ><br>
> > > I suggest removing .min_kver check in all of the tests to let they can be<br>
> > > running on many distributions(which backport the features).<br>
> ><br>
> > If we do that we have to explicitely check for ENOSYS errno in each<br>
> > test, quite possibly with a dummy call to the tested syscall in test<br>
> > setup, because once these calls gets libc wrappers we will no longer<br>
> > call the tst_syscall() that checks for it.<br>
> ><br>
> <br>
> +1 add dummy call to the tested syscall in the setup.<br>
> Agree, thanks for point out this.<br>
<br>
Not sure if I understood it very clearly and don't want to spam everyone with an<br>
incorrect patchset still missing this point and so asking here :)<br>
<br>
How exactly would the setup() routine look like? Something like this ?<br>
<br>
        syscall(#NR, ##__VA_ARGS__);<br></blockquote><div> </div><div><span class="gmail_default" style="font-size:small">I suggest to use tst_syscall(...);</span><span class="gmail_default" style="font-size:small">.</span></div><div><br></div><div class="gmail_default" style="font-size:small">A dummy call in setup() just helps check the ENOSYS errno, something like:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">$ cat testcases/cve/cve-2016-7117.c |grep setup -A 10<br>static void setup(void)<br>{<br>        ...<br>        tst_syscall(__NR_recvmmsg, 0, 0, 0, 0, 0);<br>}<br></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>