Re: [PATCH 1/8] vfs - change d_manage() to take a struct path

From: Eric W. Biederman
Date: Tue Oct 11 2016 - 12:07:19 EST


Ian Kent <raven@xxxxxxxxxx> writes:

> For the autofs module to be able to reliably check if a dentry is a
> mountpoint in a multiple namespace environment the ->d_manage() dentry
> operation will need to take a path argument instead of a dentry.

Taking a quick look overall I see no issues with this series. Overall
it seems straight forward.

On the nit side I expect saying const struct path * in the functions
that now take a struct path would be useful.

I suspect it would also be useful to say
const struct path *path;
path = &file->f_path;

In the one part of the code where you do that. Instead of copying the
path out of the struct file.

Overall I expect that will keep down bugs at no reduction in usability.
Just a statement that the struct path won't change when it is passed
to various functions.

Eric