From: Etay Meiri <cl1@netvision.net.il>
To: Alexander Viro <viro@math.psu.edu>
Cc: linux-kernel@vger.kernel.org
Subject: Re: exporting open_namei to modules
Date: Sat, 13 Oct 2001 07:58:53 +0200 [thread overview]
Message-ID: <20011013075853.C1069@amber.rog.net> (raw)
In-Reply-To: <20011013011841.B1069@amber.rog.net> <Pine.GSO.4.21.0110121955470.76-100000@weyl.math.psu.edu>
In-Reply-To: <Pine.GSO.4.21.0110121955470.76-100000@weyl.math.psu.edu>; from viro@math.psu.edu on Fri, Oct 12, 2001 at 08:03:33PM -0400
On Fri, Oct 12, 2001 at 08:03:33PM -0400, Alexander Viro wrote:
>
>
> On Sat, 13 Oct 2001, Etay Meiri wrote:
>
> > Hi,
> >
> > Is there a particular reason why open_namei() is
> > not exported to modules?
>
> Is there any reason for exporting it? By default, stuff is _NOT_
> exported. Think for a moment and you'll see why. Exported functions are
> public API. Protection is weaker than for syscalls, but it's there and
> exporting a function makes harder to do changes in core kernel. Unless
> there are damn serious reasons for exporting something, it isn't done.
>
> In particular, open_namei() is a helper function of filp_open(),
> which _is_ exported. What use of open_namei() do you have in mind?
I'm writing a kernel file server. Unlike NFS, clients send me full paths
so I need to translate them to their respective inodes before calling
->open(...).
The reason I didn't use filp_open() in the first place is because it calls
get_empty_filp() and, when I started writing this thing, didn't really understand
and so I thought it was better for me to call open_namei() directly and duplicate
some of the stuff that was going on in filp_open(). Now I see that there is a better
solution.
Thanks Alexander.
>
>
--
************************************************
"When in doubt, use brute force."
Ken Thompson
Etay Meiri
cl1@netvision.net.il
************************************************
prev parent reply other threads:[~2001-10-13 5:55 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-10-12 23:18 exporting open_namei to modules Etay Meiri
2001-10-13 0:03 ` Alexander Viro
2001-10-13 5:58 ` Etay Meiri [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20011013075853.C1069@amber.rog.net \
--to=cl1@netvision.net.il \
--cc=linux-kernel@vger.kernel.org \
--cc=viro@math.psu.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.