On Mon, Jun 21, 2021 at 1:31 AM SeongJae Park <sj38.park@xxxxxxxxx> wrote: > > From: SeongJae Park <sjpark@xxxxxxxxx> > > 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 <sjpark@xxxxxxxxx> > Reviewed-by: Fernand Sieber <sieberf@xxxxxxxxxx> > --- [...] > > +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.