All of lore.kernel.org
 help / color / mirror / Atom feed
From: gregkh@linuxfoundation.org (Greg KH)
To: linux-security-module@vger.kernel.org
Subject: [PATCH] security: Export few symbols referred by other modules
Date: Wed, 6 Dec 2017 16:32:31 +0100	[thread overview]
Message-ID: <20171206153231.GC29623@kroah.com> (raw)
In-Reply-To: <0322ddc2-f1a2-93f5-d34d-d57f6be734a3@codeaurora.org>

On Wed, Dec 06, 2017 at 08:45:01PM +0530, Hareesh Gundu wrote:
> On 12/5/2017 6:51 PM, Tetsuo Handa wrote:
> > James Morris wrote:
> > > On Tue, 5 Dec 2017, Hareesh Gundu wrote:
> > > 
> > > > Export mmap_min_addr and security_mmap_addr() to allow
> > > > kernel modules to use them.
> > > > 
> > > > Signed-off-by: Hareesh Gundu <hareeshg@codeaurora.org>
> Can other folks comment whether this should be EXPORT_SYMBOL or
> EXPORT_SYMBOL_GPL ?

What do you think?

> > > I'm not sure whether this should be EXPORT_SYMBOL or EXPORT_SYMBOL_GPL, as
> > > this hook was added in 2009, well after EXPORT_SYMBOL_GPL came into being.
> > > 
> > > Most of the LSM hooks are marked EXPORT_SYMBOL because they were part of
> > > an existing interface when EXPORT_SYMBOL_GPL was introduced, IIRC.
> > > 
> > > What do folks think?
> > > 
> ?That's right, This change is for outside kernel tree modules.

Then it should not be exported at all, sorry.

Please work to get your code merged into the kernel tree and then it can
be exported properly for it.  Odds are, you don't really need these
symbols, as {hint}, no one else does...

sorry,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-security-module" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2017-12-06 15:32 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-05 10:58 [PATCH] security: Export few symbols referred by other modules Hareesh Gundu
2017-12-05 12:04 ` James Morris
2017-12-05 13:21   ` Tetsuo Handa
2017-12-06 15:15     ` Hareesh Gundu
2017-12-06 15:32       ` Greg KH [this message]
2017-12-07  0:10         ` James Morris
2017-12-05 13:34   ` Greg KH

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=20171206153231.GC29623@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=linux-security-module@vger.kernel.org \
    /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.