Re: [PATCH v3] tracing/selftests: Add ownership modification tests for eventfs

2023-12-22 Thread Steven Rostedt
On Fri, 22 Dec 2023 11:28:31 -0500 Steven Rostedt wrote: > > -- this file has executable permission set unlike the existing > > .tc files in the same directory > > Oh, I forgot to disable that. When developing a new test I make it > standalone as it's easier to test the test, and then cop

Re: [PATCH v3] tracing/selftests: Add ownership modification tests for eventfs

2023-12-22 Thread Steven Rostedt
On Fri, 22 Dec 2023 09:15:42 -0700 Shuah Khan wrote: > On 12/21/23 19:16, Steven Rostedt wrote: > > > > Shuah, > > > > This patch has no dependencies. You can take it through your tree for the > > next merge window if you want. If not, I can take it. > > > Tried to apply this and seeing a co

Re: [PATCH v3] tracing/selftests: Add ownership modification tests for eventfs

2023-12-22 Thread Shuah Khan
On 12/21/23 19:16, Steven Rostedt wrote: Shuah, This patch has no dependencies. You can take it through your tree for the next merge window if you want. If not, I can take it. Tried to apply this and seeing a couple of issues: -- missing SPDX -- this file has executable permission set unlike

Re: [PATCH v3] tracing/selftests: Add ownership modification tests for eventfs

2023-12-21 Thread Google
On Thu, 21 Dec 2023 21:12:29 -0500 Steven Rostedt wrote: > From: "Steven Rostedt (Google)" > > As there were bugs found with the ownership of eventfs dynamic file > creation. Add a test to test it. > > It will remount tracefs with a different gid and check the ownership of > the eventfs direct

Re: [PATCH v3] tracing/selftests: Add ownership modification tests for eventfs

2023-12-21 Thread Steven Rostedt
Shuah, This patch has no dependencies. You can take it through your tree for the next merge window if you want. If not, I can take it. Thanks, -- Steve On Thu, 21 Dec 2023 21:12:29 -0500 Steven Rostedt wrote: > From: "Steven Rostedt (Google)" > > As there were bugs found with the ownersh

[PATCH v3] tracing/selftests: Add ownership modification tests for eventfs

2023-12-21 Thread Steven Rostedt
From: "Steven Rostedt (Google)" As there were bugs found with the ownership of eventfs dynamic file creation. Add a test to test it. It will remount tracefs with a different gid and check the ownership of the eventfs directory, as well as the system and event directories. It will also check the