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