On Mon, Jun 04, 2018 at 01:46:09AM -0700, Christoph Hellwig wrote: > > +EXPORT_SYMBOL(path_open); > > EXPORT_SYMBOL_GPL, please. No. If interface makes sense, export it. If it doens't, don't. Don't mix "it's a shit API, but we need it for some in-kernel module" with "out-of-tree code should be GPL, especially if it uses this". For non-trivial work I will, teeth gritting, accept that kind of stuff. For anything as trivial as this - fuck, no. In this particular case, it *is* a dubious API - AFAICS, ovl_open_realfile() could just pass vfsmount/dentry from the right layer to vfs_open(). We might or might not need path_open() for the duration of the series (I hadn't looked into the PITA it would be to reorder), but it really looks like it could disappear by the end of it, along with the temporary export.
