From: SeongJae Park <redacted>

On Tue, 22 Jun 2021 11:23:12 -0700 Shakeel Butt [off-list ref] wrote:

On Mon, Jun 21, 2021 at 1:31 AM SeongJae Park [off-list ref] wrote:

quoted

From: SeongJae Park <redacted>

For CPU usage accounting, knowing pid of the monitoring thread could be
helpful.  For example, users could use cpuaccount cgroups with the pid.

This commit therefore exports the pid of currently running monitoring
thread to the user space via 'kdamond_pid' file in the debugfs
directory.

Signed-off-by: SeongJae Park <redacted>
Reviewed-by: Fernand Sieber <redacted>
---

[...]

quoted

+static const struct file_operations kdamond_pid_fops = {
+       .owner = THIS_MODULE,

I don't think you need to set the owner (and for other fops) as these
files are built into modules. Otherwise it looks good.

Good point.  Will remove those from the next spin.


Thanks,
SeongJae Park