2007-11-01 21:36:49

by Jan Blunck

Subject: [PATCH 0/7] struct path related cleanups of d_path() code

Here are some more struct path cleanups. This patch series changes d_path() to
take a struct path argument. The existing users are changed to give struct path
more deeply into the call chain. In some structures I need to replace existing
<dentry,vfsmount> pairs and embed a struct path instead.

Andreas,
since JJ just posted the AppArmor related d_path() patches I don't include
them here too.

Comments?

Jan


>>one-less-parameter-to-__d_path.patch<<
One less parameter to __d_path

>>d_path-kerneldoc_cleanup.diff<<
d_path: kerneldoc cleanup

>>d_path-Use_struct_path_in_struct_avc_audit_data.diff<<
d_path: Use struct path in struct avc_audit_data

>>d_path-Make_proc_get_link_use_a_struct_path_argument.diff<<
d_path: Make proc_get_link() use a struct path argument

>>d_path_Make_get_dcookie_use_a_struct_path_argument.diff<<
d_path: Make get_dcookie() use a struct path argument

>>d_path-use_struct_path.diff<<
d_path: Make d_path() use a struct path

>>nfsd-svc_export_use_struct_path.diff<<
Use struct path in struct svc_export

--

2007-11-05 10:29:00

by Christoph Hellwig

Subject: Re: [PATCH 0/7] struct path related cleanups of d_path() code

On Mon, Oct 29, 2007 at 01:41:20PM +0100, Jan Blunck wrote:
>
> Here are some more struct path cleanups. This patch series changes d_path() to
> take a struct path argument. The existing users are changed to give struct path
> more deeply into the call chain. In some structures I need to replace existing
> <dentry,vfsmount> pairs and embed a struct path instead.

All these patches look good to me, but please reorder patch 6 to apply before
patch 7 in your current series. Also it would be nice if you could also
convert seq_path to take a struct path while you're at it.