From: Joel Fernandes <joel@joelfernandes.org>
To: Masahiro Yamada <yamada.masahiro@socionext.com>
Cc: "Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
"Steven Rostedt" <rostedt@goodmis.org>,
"Adrian Ratiu" <adrian.ratiu@collabora.com>,
"Alexei Starovoitov" <ast@kernel.org>,
"Andrew Morton" <akpm@linux-foundation.org>,
"atish patra" <atishp04@gmail.com>,
bpf@vger.kernel.org, "Brendan Gregg" <bgregg@netflix.com>,
"Brendan Gregg" <brendan.d.gregg@gmail.com>,
"Daniel Colascione" <dancol@google.com>,
"Daniel Borkmann" <daniel@iogearbox.net>,
"Dan Williams" <dan.j.williams@intel.com>,
"Dietmar Eggemann" <dietmar.eggemann@arm.com>,
duyuchao <yuchao.du@unisoc.com>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Guenter Roeck" <groeck@chromium.org>,
"Jonathan Corbet" <corbet@lwn.net>,
"Karim Yaghmour" <karim.yaghmour@opersys.com>,
"Kees Cook" <keescook@chromium.org>,
"Cc: Android Kernel" <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,
"Manjo Raja Rao" <linux@manojrajarao.com>,
"Masami Hiramatsu" <mhiramat@kernel.org>,
"Michał Gregorczyk" <michalgr@fb.com>,
"Michal Gregorczyk" <michalgr@live.com>,
"Mohammad Husain" <russoue@gmail.com>,
"Olof Johansson" <olof@lixom.net>,
"Qais Yousef" <qais.yousef@arm.com>,
"Randy Dunlap" <rdunlap@infradead.org>,
"Shuah Khan" <shuah@kernel.org>,
"Srinivas Ramana" <sramana@codeaurora.org>,
"Tamir Carmeli" <carmeli.tamir@gmail.com>,
"Yonghong Song" <yhs@fb.com>
Subject: Re: [PATCH] kheaders: Move from proc to sysfs
Date: Fri, 3 May 2019 20:59:20 -0400 [thread overview]
Message-ID: <20190504005920.GA261146@google.com> (raw)
In-Reply-To: <CAK7LNASkR7cauvcLprgrTKNv-iY4yjS278FPGJ-UEYTBrANKYw@mail.gmail.com>
On Sat, May 04, 2019 at 09:21:41AM +0900, Masahiro Yamada wrote:
> On Sat, May 4, 2019 at 9:18 AM Masahiro Yamada
> <yamada.masahiro@socionext.com> wrote:
> >
> > On Sat, May 4, 2019 at 3:27 AM Joel Fernandes (Google)
> > <joel@joelfernandes.org> wrote:
> > >
> > > The kheaders archive consisting of the kernel headers used for compiling
> > > bpf programs is in /proc. However there is concern that moving it here
> > > will make it permanent. Let us move it to /sys/kernel as discussed [1].
> > >
> > > [1] https://lore.kernel.org/patchwork/patch/1067310/#1265969
> > >
> > > Suggested-by: Steven Rostedt <rostedt@goodmis.org>
> > > Signed-off-by: Joel Fernandes (Google) <joel@joelfernandes.org>
> > > ---
> > > This patch applies on top of the previous patch that was applied to the
> > > driver tree:
> > > https://lore.kernel.org/patchwork/patch/1067310/
> > >
> > > kernel/kheaders.c | 40 ++++++++++++++++------------------------
> >
> >
> > Please rename CONFIG_IKHEADERS_PROC.
> >
> > Thanks.
>
>
> Please adjust Kconfig prompt.
> tristate "Enable kernel header artifacts through /proc/kheaders.tar.xz"
>
>
> "depends on PROC_FS" -> "depends on SYSFS"
Sorry to miss these config updates. I will update the patch and resend.
WARNING: multiple messages have this Message-ID (diff)
From: joel at joelfernandes.org (Joel Fernandes)
Subject: [PATCH] kheaders: Move from proc to sysfs
Date: Fri, 3 May 2019 20:59:20 -0400 [thread overview]
Message-ID: <20190504005920.GA261146@google.com> (raw)
In-Reply-To: <CAK7LNASkR7cauvcLprgrTKNv-iY4yjS278FPGJ-UEYTBrANKYw@mail.gmail.com>
On Sat, May 04, 2019 at 09:21:41AM +0900, Masahiro Yamada wrote:
> On Sat, May 4, 2019 at 9:18 AM Masahiro Yamada
> <yamada.masahiro at socionext.com> wrote:
> >
> > On Sat, May 4, 2019 at 3:27 AM Joel Fernandes (Google)
> > <joel at joelfernandes.org> wrote:
> > >
> > > The kheaders archive consisting of the kernel headers used for compiling
> > > bpf programs is in /proc. However there is concern that moving it here
> > > will make it permanent. Let us move it to /sys/kernel as discussed [1].
> > >
> > > [1] https://lore.kernel.org/patchwork/patch/1067310/#1265969
> > >
> > > Suggested-by: Steven Rostedt <rostedt at goodmis.org>
> > > Signed-off-by: Joel Fernandes (Google) <joel at joelfernandes.org>
> > > ---
> > > This patch applies on top of the previous patch that was applied to the
> > > driver tree:
> > > https://lore.kernel.org/patchwork/patch/1067310/
> > >
> > > kernel/kheaders.c | 40 ++++++++++++++++------------------------
> >
> >
> > Please rename CONFIG_IKHEADERS_PROC.
> >
> > Thanks.
>
>
> Please adjust Kconfig prompt.
> tristate "Enable kernel header artifacts through /proc/kheaders.tar.xz"
>
>
> "depends on PROC_FS" -> "depends on SYSFS"
Sorry to miss these config updates. I will update the patch and resend.
WARNING: multiple messages have this Message-ID (diff)
From: joel@joelfernandes.org (Joel Fernandes)
Subject: [PATCH] kheaders: Move from proc to sysfs
Date: Fri, 3 May 2019 20:59:20 -0400 [thread overview]
Message-ID: <20190504005920.GA261146@google.com> (raw)
Message-ID: <20190504005920.o-EPAQamnjhh4_LKGsHYkHgnA68-jkfPd8QVG-3gRUs@z> (raw)
In-Reply-To: <CAK7LNASkR7cauvcLprgrTKNv-iY4yjS278FPGJ-UEYTBrANKYw@mail.gmail.com>
On Sat, May 04, 2019@09:21:41AM +0900, Masahiro Yamada wrote:
> On Sat, May 4, 2019 at 9:18 AM Masahiro Yamada
> <yamada.masahiro@socionext.com> wrote:
> >
> > On Sat, May 4, 2019 at 3:27 AM Joel Fernandes (Google)
> > <joel@joelfernandes.org> wrote:
> > >
> > > The kheaders archive consisting of the kernel headers used for compiling
> > > bpf programs is in /proc. However there is concern that moving it here
> > > will make it permanent. Let us move it to /sys/kernel as discussed [1].
> > >
> > > [1] https://lore.kernel.org/patchwork/patch/1067310/#1265969
> > >
> > > Suggested-by: Steven Rostedt <rostedt at goodmis.org>
> > > Signed-off-by: Joel Fernandes (Google) <joel at joelfernandes.org>
> > > ---
> > > This patch applies on top of the previous patch that was applied to the
> > > driver tree:
> > > https://lore.kernel.org/patchwork/patch/1067310/
> > >
> > > kernel/kheaders.c | 40 ++++++++++++++++------------------------
> >
> >
> > Please rename CONFIG_IKHEADERS_PROC.
> >
> > Thanks.
>
>
> Please adjust Kconfig prompt.
> tristate "Enable kernel header artifacts through /proc/kheaders.tar.xz"
>
>
> "depends on PROC_FS" -> "depends on SYSFS"
Sorry to miss these config updates. I will update the patch and resend.
next prev parent reply other threads:[~2019-05-04 0:59 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-03 18:24 [PATCH] kheaders: Move from proc to sysfs Joel Fernandes (Google)
2019-05-03 18:24 ` Joel Fernandes (Google)
2019-05-03 18:24 ` joel
2019-05-04 0:18 ` Masahiro Yamada
2019-05-04 0:18 ` Masahiro Yamada
2019-05-04 0:18 ` yamada.masahiro
2019-05-04 0:21 ` Masahiro Yamada
2019-05-04 0:21 ` Masahiro Yamada
2019-05-04 0:21 ` yamada.masahiro
2019-05-04 0:59 ` Joel Fernandes [this message]
2019-05-04 0:59 ` Joel Fernandes
2019-05-04 0:59 ` joel
2019-05-04 7:46 ` Alexei Starovoitov
2019-05-04 7:46 ` Alexei Starovoitov
2019-05-04 7:46 ` alexei.starovoitov
2019-05-04 8:11 ` Greg Kroah-Hartman
2019-05-04 8:11 ` Greg Kroah-Hartman
2019-05-04 8:11 ` gregkh
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=20190504005920.GA261146@google.com \
--to=joel@joelfernandes.org \
--cc=adrian.ratiu@collabora.com \
--cc=akpm@linux-foundation.org \
--cc=ast@kernel.org \
--cc=atishp04@gmail.com \
--cc=bgregg@netflix.com \
--cc=bpf@vger.kernel.org \
--cc=brendan.d.gregg@gmail.com \
--cc=carmeli.tamir@gmail.com \
--cc=corbet@lwn.net \
--cc=dan.j.williams@intel.com \
--cc=dancol@google.com \
--cc=daniel@iogearbox.net \
--cc=dietmar.eggemann@arm.com \
--cc=gregkh@linuxfoundation.org \
--cc=groeck@chromium.org \
--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=michalgr@fb.com \
--cc=michalgr@live.com \
--cc=olof@lixom.net \
--cc=qais.yousef@arm.com \
--cc=rdunlap@infradead.org \
--cc=rostedt@goodmis.org \
--cc=russoue@gmail.com \
--cc=shuah@kernel.org \
--cc=sramana@codeaurora.org \
--cc=yamada.masahiro@socionext.com \
--cc=yhs@fb.com \
--cc=yuchao.du@unisoc.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 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.