linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Luis Chamberlain <mcgrof@kernel.org>
Cc: "Song Liu" <song@kernel.org>, "Christoph Hellwig" <hch@lst.de>,
	"Masami Hiramatsu" <mhiramat@kernel.org>,
	"Jarkko Sakkinen" <jarkko@kernel.org>,
	"Guo Ren" <guoren@kernel.org>,
	"Jarkko Sakkinen" <jarkko@profian.com>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
	"Nathaniel McCallum" <nathaniel@profian.com>,
	"Russell King" <linux@armlinux.org.uk>,
	"Catalin Marinas" <catalin.marinas@arm.com>,
	"Will Deacon" <will@kernel.org>,
	"Thomas Bogendoerfer" <tsbogend@alpha.franken.de>,
	"James E.J. Bottomley" <James.Bottomley@hansenpartnership.com>,
	"Helge Deller" <deller@gmx.de>,
	"Michael Ellerman" <mpe@ellerman.id.au>,
	"Benjamin Herrenschmidt" <benh@kernel.crashing.org>,
	"Paul Mackerras" <paulus@samba.org>,
	"Paul Walmsley" <paul.walmsley@sifive.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Albert Ou" <aou@eecs.berkeley.edu>,
	"Heiko Carstens" <hca@linux.ibm.com>,
	"Vasily Gorbik" <gor@linux.ibm.com>,
	"Alexander Gordeev" <agordeev@linux.ibm.com>,
	"Christian Borntraeger" <borntraeger@linux.ibm.com>,
	"Sven Schnelle" <svens@linux.ibm.com>,
	"David S. Miller" <davem@davemloft.net>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Ingo Molnar" <mingo@redhat.com>,
	"Borislav Petkov" <bp@alien8.de>,
	"Dave Hansen" <dave.hansen@linux.intel.com>,
	"the arch/x86 maintainers" <x86@kernel.org>,
	"H. Peter Anvin" <hpa@zytor.com>,
	"Naveen N. Rao" <naveen.n.rao@linux.ibm.com>,
	"Anil S Keshavamurthy" <anil.s.keshavamurthy@intel.com>,
	"Steven Rostedt" <rostedt@goodmis.org>,
	"Kees Cook" <keescook@chromium.org>,
	"Peter Zijlstra (Intel)" <peterz@infradead.org>,
	"Nathan Chancellor" <nathan@kernel.org>,
	"Josh Poimboeuf" <jpoimboe@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	"Marco Elver" <elver@google.com>,
	"Dan Li" <ashimida@linux.alibaba.com>,
	"Sami Tolvanen" <samitolvanen@google.com>,
	"Ard Biesheuvel" <ardb@kernel.org>,
	"Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>,
	"Nick Desaulniers" <ndesaulniers@google.com>,
	"Linus Walleij" <linus.walleij@linaro.org>,
	"Chen Zhongjin" <chenzhongjin@huawei.com>,
	"Nicolas Pitre" <nico@fluxnic.net>,
	"Mark Brown" <broonie@kernel.org>,
	"Luis Machado" <luis.machado@linaro.org>,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	"Joey Gouly" <joey.gouly@arm.com>,
	"Masahiro Yamada" <masahiroy@kernel.org>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"Andrey Konovalov" <andreyknvl@gmail.com>,
	"Kefeng Wang" <wangkefeng.wang@huawei.com>,
	"Atsushi Nemoto" <anemo@mba.ocn.ne.jp>,
	"Guenter Roeck" <linux@roeck-us.net>,
	"Dave Anglin" <dave.anglin@bell.net>,
	"Christophe Leroy" <christophe.leroy@csgroup.eu>,
	"Alexei Starovoitov" <ast@kernel.org>,
	"Nicholas Piggin" <npiggin@gmail.com>,
	"Daniel Axtens" <dja@axtens.net>,
	"Aneesh Kumar K.V" <aneesh.kumar@linux.ibm.com>,
	"Jordan Niethe" <jniethe5@gmail.com>,
	"Anup Patel" <anup@brainfault.org>,
	"Atish Patra" <atishp@atishpatra.org>,
	"Changbin Du" <changbin.du@intel.com>,
	"Heiko Stuebner" <heiko@sntech.de>,
	"Liao Chang" <liaochang1@huawei.com>,
	"Philipp Tomsich" <philipp.tomsich@vrull.eu>,
	"Wu Caize" <zepan@sipeed.com>,
	"Emil Renner Berthing" <kernel@esmil.dk>,
	"Alexander Egorenkov" <egorenar@linux.ibm.com>,
	"Thomas Richter" <tmricht@linux.ibm.com>,
	"Tobias Huschle" <huschle@linux.ibm.com>,
	"Ilya Leoshkevich" <iii@linux.ibm.com>,
	"Tom Lendacky" <thomas.lendacky@amd.com>,
	"Daniel Bristot de Oliveira" <bristot@redhat.com>,
	"Michael Roth" <michael.roth@amd.com>,
	"Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
	"Javier Martinez Canillas" <javierm@redhat.com>,
	"Miroslav Benes" <mbenes@suse.cz>,
	"André Almeida" <andrealmeid@igalia.com>,
	"Tiezhu Yang" <yangtiezhu@loongson.cn>,
	"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
	"Aaron Tomlin" <atomlin@redhat.com>,
	"Linux ARM" <linux-arm-kernel@lists.infradead.org>,
	"open list:BROADCOM NVRAM DRIVER" <linux-mips@vger.kernel.org>,
	"Parisc List" <linux-parisc@vger.kernel.org>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	linux-riscv <linux-riscv@lists.infradead.org>,
	linux-s390 <linux-s390@vger.kernel.org>,
	sparclinux <sparclinux@vger.kernel.org>,
	linux-modules@vger.kernel.org
Subject: Re: [PATCH] kprobes: Enable tracing for mololithic kernel images
Date: Thu, 9 Jun 2022 05:48:52 +0200	[thread overview]
Message-ID: <20220609034852.GA30873@lst.de> (raw)
In-Reply-To: <YqEF6+YKqCHsWZJW@bombadil.infradead.org>

On Wed, Jun 08, 2022 at 01:26:19PM -0700, Luis Chamberlain wrote:
> No, that was removed because it has only one user.

That is only part of the story.  The other part is that the overall
kernel simply does not have any business allocating exutable memory.
Executable memory is a very special concept for modules or module-like
code like kprobes, and should not be exposed as a general concept.

Especially as executable memory really should not also be writable
for security reasons.  In other words, we should actually never
allocate executable memory, every.  We might seal memory and then
mark it executable after having written to it, which is how modules
and kprobes are implemented on all modern Linux ports anyway.

  reply	other threads:[~2022-06-09  3:49 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-07 23:59 [PATCH] kprobes: Enable tracing for mololithic kernel images Jarkko Sakkinen
2022-06-08  2:35 ` Guo Ren
2022-06-08  5:25   ` Jarkko Sakkinen
2022-06-08 14:21     ` Masami Hiramatsu
2022-06-08 16:12       ` Song Liu
2022-06-08 18:20         ` Song Liu
2022-06-08 20:26           ` Luis Chamberlain
2022-06-09  3:48             ` Christoph Hellwig [this message]
2022-06-09 13:24               ` Luis Chamberlain
2022-06-09 18:41                 ` Edgecombe, Rick P
2022-06-09 22:48                   ` Song Liu
2022-06-14 12:32                   ` jarkko
2022-06-15  6:37                     ` hch
2022-06-15 21:29                       ` jarkko
2022-06-09 13:12         ` Jarkko Sakkinen
2022-06-09 13:23           ` Ard Biesheuvel
2022-06-12 12:18             ` Masami Hiramatsu
2022-06-14 10:54             ` Jarkko Sakkinen
     [not found]         ` <f7ff2a8d-4128-dac9-9768-1d98bfb06a06@csgroup.eu>
2022-06-09 22:23           ` Song Liu
2022-06-09 12:59       ` Jarkko Sakkinen
2022-06-08 16:27 ` Ard Biesheuvel
2022-06-08 18:19   ` Song Liu
2022-06-12 12:30     ` Masami Hiramatsu
2022-06-14 12:30       ` Jarkko Sakkinen
2022-06-09  5:37   ` Jarkko Sakkinen
2022-06-09  7:47 ` Russell King (Oracle)
2022-06-09 11:48   ` Jarkko Sakkinen
2022-06-09 13:44   ` Luis Chamberlain
2022-06-14 12:26     ` Jarkko Sakkinen
     [not found]       ` <144b7e90-5d75-7e2c-4e54-8a73f914dfdf@csgroup.eu>
2022-06-15 21:24         ` Jarkko Sakkinen
     [not found] ` <f2030fb4-4978-068b-6250-5bd5b2746675@csgroup.eu>
2022-06-09 12:57   ` Jarkko Sakkinen

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=20220609034852.GA30873@lst.de \
    --to=hch@lst.de \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=agordeev@linux.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=andrealmeid@igalia.com \
    --cc=andreyknvl@gmail.com \
    --cc=aneesh.kumar@linux.ibm.com \
    --cc=anemo@mba.ocn.ne.jp \
    --cc=anil.s.keshavamurthy@intel.com \
    --cc=anup@brainfault.org \
    --cc=aou@eecs.berkeley.edu \
    --cc=ardb@kernel.org \
    --cc=ashimida@linux.alibaba.com \
    --cc=ast@kernel.org \
    --cc=atishp@atishpatra.org \
    --cc=atomlin@redhat.com \
    --cc=benh@kernel.crashing.org \
    --cc=borntraeger@linux.ibm.com \
    --cc=bp@alien8.de \
    --cc=bristot@redhat.com \
    --cc=broonie@kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=changbin.du@intel.com \
    --cc=chenzhongjin@huawei.com \
    --cc=christophe.leroy@csgroup.eu \
    --cc=dave.anglin@bell.net \
    --cc=dave.hansen@linux.intel.com \
    --cc=davem@davemloft.net \
    --cc=deller@gmx.de \
    --cc=dja@axtens.net \
    --cc=dmitry.torokhov@gmail.com \
    --cc=ebiederm@xmission.com \
    --cc=egorenar@linux.ibm.com \
    --cc=elver@google.com \
    --cc=geert@linux-m68k.org \
    --cc=gor@linux.ibm.com \
    --cc=guoren@kernel.org \
    --cc=hca@linux.ibm.com \
    --cc=heiko@sntech.de \
    --cc=hpa@zytor.com \
    --cc=huschle@linux.ibm.com \
    --cc=iii@linux.ibm.com \
    --cc=jarkko@kernel.org \
    --cc=jarkko@profian.com \
    --cc=javierm@redhat.com \
    --cc=jniethe5@gmail.com \
    --cc=joey.gouly@arm.com \
    --cc=jpoimboe@kernel.org \
    --cc=keescook@chromium.org \
    --cc=kernel@esmil.dk \
    --cc=kirill.shutemov@linux.intel.com \
    --cc=liaochang1@huawei.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=linux-modules@vger.kernel.org \
    --cc=linux-parisc@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=linux@roeck-us.net \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=luis.machado@linaro.org \
    --cc=mark.rutland@arm.com \
    --cc=masahiroy@kernel.org \
    --cc=mbenes@suse.cz \
    --cc=mcgrof@kernel.org \
    --cc=mhiramat@kernel.org \
    --cc=michael.roth@amd.com \
    --cc=mingo@redhat.com \
    --cc=mpe@ellerman.id.au \
    --cc=nathan@kernel.org \
    --cc=nathaniel@profian.com \
    --cc=naveen.n.rao@linux.ibm.com \
    --cc=ndesaulniers@google.com \
    --cc=nico@fluxnic.net \
    --cc=npiggin@gmail.com \
    --cc=palmer@dabbelt.com \
    --cc=paul.walmsley@sifive.com \
    --cc=paulus@samba.org \
    --cc=peterz@infradead.org \
    --cc=philipp.tomsich@vrull.eu \
    --cc=rmk+kernel@armlinux.org.uk \
    --cc=rostedt@goodmis.org \
    --cc=samitolvanen@google.com \
    --cc=song@kernel.org \
    --cc=sparclinux@vger.kernel.org \
    --cc=svens@linux.ibm.com \
    --cc=tglx@linutronix.de \
    --cc=thomas.lendacky@amd.com \
    --cc=tmricht@linux.ibm.com \
    --cc=tsbogend@alpha.franken.de \
    --cc=wangkefeng.wang@huawei.com \
    --cc=will@kernel.org \
    --cc=x86@kernel.org \
    --cc=yangtiezhu@loongson.cn \
    --cc=zepan@sipeed.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).