Messages in this thread | | | Date | Wed, 28 Jun 2023 00:01:01 +0500 | Subject | Re: [PATCH v2] selftests: prctl: Add new prctl test for PR_SET_NAME | From | Muhammad Usama Anjum <> |
| |
On 6/10/23 2:43 AM, Shuah Khan wrote: > On 6/7/23 09:36, Osama Muhammad wrote: >> This patch will add the new test, which covers the prctl call >> PR_SET_NAME command. The test tries to give a name using the PR_SET_NAME >> call and then confirm it that it changed correctly by using PR_GET_NAME. >> It also tries to rename it with empty name.In the test PR_GET_NAME is >> tested by passing null pointer to it and check its behaviour. >> >> Signed-off-by: Osama Muhammad <osmtendev@gmail.com> >> >> --- >> changes since v1: >> -Used TASK_COMM_LEN instead of using numerical value 16. > > Please add linux/sched.h here as an include to pull this. > It is good to add an explicit include as opposed taking > a chance on it being included from another include. TASK_COMM_LEN is defined in include/linux/sched.h. It is only to be used by the kernel.
If we look at include/uapi/linux/sched.h, TASK_COMM_LEN isn't defined. So this test looks good to me.
> > thanks, > -- Shuah
-- BR, Muhammad Usama Anjum
| |