From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Kees Cook <keescook@chromium.org>,
Olof Johansson <olof@lixom.net>,
Alexei Starovoitov <alexei.starovoitov@gmail.com>,
Joel Fernandes <joelaf@google.com>,
Joel Fernandes <joel@joelfernandes.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Qais Yousef <qais.yousef@arm.com>,
Dietmar Eggemann <dietmar.eggemann@arm.com>,
Manoj Rao <linux@manojrajarao.com>,
Andrew Morton <akpm@linux-foundation.org>,
Alexei Starovoitov <ast@kernel.org>,
atish patra <atishp04@gmail.com>,
Daniel Colascione <dancol@google.com>,
Dan Williams <dan.j.williams@intel.com>,
Guenter Roeck <groeck@chromium.org>,
Jonathan Corbet <corbet@lwn.net>,
Karim Yaghmour <karim.yaghmour@opersys.com>,
Android Kernel Team <kernel-team@android.com>,
"open list:DOCUMENTATION" <linux-doc@vger.kernel.org>,
"open list:KERNEL SELFTEST FRAMEWORK"
<linux-kselftest@vger.kernel.org>,
linux-trace-devel@vger.kernel.org,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Masami Hiramatsu <mhiramat@kernel.org>,
Randy Dunlap <rdunlap@infradead.org>,
Shuah Khan <shuah@kernel.org>, Yonghong Song <yhs@fb.com>
Subject: Re: [PATCH v5 1/3] Provide in-kernel headers to make extending kernel easier
Date: Tue, 16 Apr 2019 14:49:39 +0200 [thread overview]
Message-ID: <20190416124939.GB6027@kroah.com> (raw)
In-Reply-To: <20190416083306.5646a687@gandalf.local.home>
On Tue, Apr 16, 2019 at 08:33:06AM -0400, Steven Rostedt wrote:
> On Mon, 15 Apr 2019 22:50:10 -0500
> Kees Cook <keescook@chromium.org> wrote:
>
> > On Mon, Apr 15, 2019 at 9:41 AM Steven Rostedt <rostedt@goodmis.org> wrote:
> > > I agree with this assessment. We shouldn't use config.gz as precedence
> > > for this solution. config.gz should have been in debugfs to begin with,
> > > but I don't believe debugfs was around when config.gz was introduced.
> > > (Don't have time to look into the history of the two).
> >
> > I don't agree with this: /proc/config.gz is used by a lot of tools
> > that do sanity-check of running systems. This isn't _debugging_...
> > it's verifying correct kernel builds. It's a fancy version of checking
> > /proc/version.
> >
>
> Then we should perhaps make a new file system call tarballs ;-)
>
> /sys/kernel/tarballs/
>
> and place everything there. That way it removes it from /proc (which is
> the worse place for that) and also makes it something other than debug.
> That's what I did for tracefs.
As horrible as that suggestion is, it does kind of make sense :)
We can't put this in debugfs as that's only for debugging and systems
should never have that mounted for normal operations (users want to
build ebpf programs), and /proc really should be for processes but that
horse is long left the barn.
But, I'm willing to consider putting this either in a system-fs-like
filesystem, or just in sysfs itself, we do have /sys/kernel/ to play
around in if the main objection is that we should not be cluttering up
/proc with stuff like this.
thanks,
greg k-h
next prev parent reply other threads:[~2019-04-16 12:57 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-20 16:31 [PATCH v5 1/3] Provide in-kernel headers to make extending kernel easier Joel Fernandes (Google)
2019-03-20 16:31 ` [PATCH v5 2/3] Add selftests for module build using in-kernel headers Joel Fernandes (Google)
2019-03-20 16:31 ` [PATCH v5 3/3] init/config: Do not select BUILD_BIN2C for IKCONFIG Joel Fernandes (Google)
2019-03-20 18:31 ` [PATCH v5 1/3] Provide in-kernel headers to make extending kernel easier Andrew Morton
2019-03-20 19:42 ` Joel Fernandes
2019-04-08 16:29 ` Olof Johansson
2019-04-08 16:37 ` Daniel Colascione
2019-04-08 16:53 ` Olof Johansson
2019-04-08 20:36 ` Joel Fernandes
2019-04-10 15:07 ` Olof Johansson
2019-04-10 15:50 ` Joel Fernandes
2019-04-10 16:34 ` Olof Johansson
2019-04-10 17:33 ` Joel Fernandes
2019-04-10 17:35 ` Daniel Colascione
2019-04-11 3:15 ` Alexei Starovoitov
2019-04-11 16:30 ` Joel Fernandes
2019-04-14 19:38 ` Olof Johansson
2019-04-15 9:41 ` Enrico Weigelt, metux IT consult
2019-04-15 13:52 ` Joel Fernandes
2019-04-15 14:05 ` Joel Fernandes
2019-04-15 14:41 ` Steven Rostedt
2019-04-16 3:50 ` Kees Cook
2019-04-16 12:33 ` Steven Rostedt
2019-04-16 12:49 ` Greg Kroah-Hartman [this message]
2019-04-16 13:04 ` Joel Fernandes
2019-04-16 13:32 ` Karim Yaghmour
2019-04-16 13:45 ` Steven Rostedt
2019-04-16 14:21 ` Joel Fernandes
2019-04-16 14:22 ` Greg Kroah-Hartman
2019-04-16 14:43 ` Steven Rostedt
2019-04-16 16:42 ` Olof Johansson
2019-04-16 16:46 ` Alexei Starovoitov
2019-04-16 16:57 ` Olof Johansson
2019-04-16 17:22 ` Joel Fernandes
2019-04-16 17:30 ` Alexei Starovoitov
2019-04-16 16:47 ` Olof Johansson
2019-04-10 19:19 ` Arnd Bergmann
2019-04-12 16:16 ` Enrico Weigelt, metux IT consult
2019-04-12 17:25 ` Steven Rostedt
2019-04-08 20:52 ` Karim Yaghmour
2019-04-10 15:15 ` Olof Johansson
2019-04-10 15:44 ` Daniel Colascione
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=20190416124939.GB6027@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=akpm@linux-foundation.org \
--cc=alexei.starovoitov@gmail.com \
--cc=ast@kernel.org \
--cc=atishp04@gmail.com \
--cc=corbet@lwn.net \
--cc=dan.j.williams@intel.com \
--cc=dancol@google.com \
--cc=dietmar.eggemann@arm.com \
--cc=groeck@chromium.org \
--cc=joel@joelfernandes.org \
--cc=joelaf@google.com \
--cc=karim.yaghmour@opersys.com \
--cc=keescook@chromium.org \
--cc=kernel-team@android.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linux-trace-devel@vger.kernel.org \
--cc=linux@manojrajarao.com \
--cc=mhiramat@kernel.org \
--cc=olof@lixom.net \
--cc=qais.yousef@arm.com \
--cc=rdunlap@infradead.org \
--cc=rostedt@goodmis.org \
--cc=shuah@kernel.org \
--cc=yamada.masahiro@socionext.com \
--cc=yhs@fb.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).