[LTP] [PATCH v2 0/4] sched_{g,s}etattr01: Add missing policies

Petr Vorel pvorel@suse.cz
Tue Jan 25 15:40:39 CET 2022


Hi,

changes v1->v2:
3 new commits

sched_getattr and sched_setattr are 99% identical (2 values are
different). I was thinking to use the same approach from e197796f22
("sethostname: Convert to new API"), but not sure if it's a good
approach.

Do we want to reduce files needed to be updated after new policy is
added? If yes, shouldn't there be just a single directory?
(what name should be using to show 2 syscalls are in sources in this
directory?)

Also docparse expect source file to be just for one binary, thus not
sure whether it should just document both binaries (and thus also
rename source file to show 2 syscalls are for this source?)

Kind regards,
Petr

Petr Vorel (4):
  lapi/sched.h: Add include <sched.h>
  lapi: Move SCHED_DEADLINE definition from tests
  sched_get_priority_min01: Add missing policies
  sched_get_priority_max01: Add missing policies

 include/lapi/sched.h                             | 16 +++++++++++-----
 testcases/kernel/syscalls/kcmp/kcmp03.c          |  1 -
 .../sched_get_priority_max01.c                   | 13 +++++++++----
 .../sched_get_priority_min01.c                   | 13 +++++++++----
 .../syscalls/sched_getattr/sched_getattr01.c     |  3 ---
 .../syscalls/sched_getattr/sched_getattr02.c     |  2 --
 .../syscalls/sched_setattr/sched_setattr01.c     |  2 --
 7 files changed, 29 insertions(+), 21 deletions(-)

-- 
2.34.1



More information about the ltp mailing list