[LTP] [PATCH v2] set_tid_address01: refactor with new LTP API
Xinjian Ma (Fujitsu)
maxj.fnst@fujitsu.com
Fri Jan 10 09:53:29 CET 2025
Hi Petr
> > Hi Ma Xinjian, Avinesh,
>
> > ...
> > > + TEST(tst_syscall(__NR_set_tid_address, &newtid));
> > > + if (TST_RET == getpid())
> > > + tst_res(TPASS, "set_tid_address call succeeded: as expected %ld",
> TST_RET);
> > > + else
> > > + tst_res(TFAIL | TTERRNO, "set_tid_address call failed: expected
> > > +%d, but got %ld", getpid(), TST_RET);
>
> > I wanted to replace this with:
>
> > TST_EXP_EQ_LI(tst_syscall(__NR_set_tid_address, &newtid), getpid());
>
> > But that TST_EXP_EQ_LI() not use TTERRNO.
>
> > Maybe use this?
>
> > TST_EXP_PID(tst_syscall(__NR_set_tid_address, &newtid));
> > TST_EXP_EQ_LI(TST_RET, getpid());
>
> OK, this is the correct one liner:
> TST_EXP_VAL(tst_syscall(__NR_set_tid_address, &newtid), getpid());
>
> Any objection to merge with change below?
Looks good.
Best regards
Ma
More information about the ltp
mailing list