Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1735891
| From | Sergey Senozhatsky <sergey.senozhatsky@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers |
| Date | 2017-09-20 18:30 +0200 |
| Message-ID | <urQ2S-5so-15@gated-at.bofh.it> (permalink) |
| Organization | linux.* mail to news gateway |
Hello
RFC
On some arches C function pointers are indirect and point to
a function descriptor, which contains the actual pointer to the code.
This mostly doesn't matter, except for cases when people want to print
out function pointers in symbolic format, because the usual '%pS/%ps'
does not work on those arches as expected. That's the reason why we
have '%pF/%pf', but since it's here because of a subtle ABI detail
specific to some arches (ppc64/ia64/parisc64) it's easy to misuse
'%pF/%pf' and '%pS/%ps' (see [1], for example).
This patch set attempts to move ia64/ppc64/parisc64 C function
pointer ABI details out of printk() to arch code. Function dereference
code now checks if a pointer belongs to a .opd ELF section and dereferences
that pointer only if it does. The kernel and modules have their own .opd
sections that's why I use two different ARCH functions: for kernel and
for module pointer dereference.
I planned to remove dereference_function_descriptor() entirely,
but then I discovered a bunch other uses cases (kgdbts, init/main.c,
extable, etc.), so I decided to keep dereference_function_descriptor()
around because the main point of this patch set is to deprecate %pF/%pf.
But at the same time, I think I can go further and handle both kernel
and module descriptor dereference in dereference_function_descriptor().
We need a module pointer for module .opd check, so that will come at an
extra cost of module lookup (may be there will some other issues along
the way, haven't checked it).
Right now we've got:
- dereference_function_descriptor(addr)
a generic (old) function. it simply attempts to dereference
whatever pointer we give it.
- dereference_kernel_function_descriptor(addr)
dereferences a kernel pointer if it's within the kernel's .opd
section.
- dereference_module_function_descriptor(module, addr)
dereference a module pointer if it's within the module's .opd
section.
*** A BIG NOTE ***
I don't own ia64/ppc64/parisc64 hardware, so the patches are not
tested. Sorry about that!
Another note:
I need to check what is BPF symbol lookup and do we need to
do any dereference there.
v2:
-- convert dereference_function_descriptor() to unsigned long
-- fix kernel descriptor range checks (Helge)
-- fix parisc module descriptor range check (Helge)
-- fix ppc64 module range check
-- add checkpatch patch
Sergey Senozhatsky (7):
switch dereference_function_descriptor() to `unsigned long'
sections: split dereference_function_descriptor()
ia64: Add .opd based function descriptor dereference
powerpc64: Add .opd based function descriptor dereference
parisc64: Add .opd based function descriptor dereference
symbol lookup: use new kernel and module dereference functions
checkpatch: add pF/pf deprecation warning
Documentation/printk-formats.txt | 15 +++++----------
arch/ia64/include/asm/sections.h | 16 ++++++++++++----
arch/ia64/kernel/module.c | 13 +++++++++++++
arch/ia64/kernel/vmlinux.lds.S | 2 ++
arch/parisc/boot/compressed/vmlinux.lds.S | 2 ++
arch/parisc/include/asm/sections.h | 4 +++-
arch/parisc/kernel/module.c | 17 +++++++++++++++++
arch/parisc/kernel/process.c | 15 ++++++++++++---
arch/parisc/kernel/vmlinux.lds.S | 2 ++
arch/parisc/mm/init.c | 4 ++--
arch/powerpc/include/asm/module.h | 3 +++
arch/powerpc/include/asm/sections.h | 17 ++++++++++++++---
arch/powerpc/kernel/module_64.c | 16 ++++++++++++++++
arch/powerpc/kernel/vmlinux.lds.S | 2 ++
drivers/misc/kgdbts.c | 2 +-
include/asm-generic/sections.h | 8 ++++++--
include/linux/moduleloader.h | 4 ++++
init/main.c | 2 +-
kernel/extable.c | 2 +-
kernel/kallsyms.c | 1 +
kernel/module.c | 9 ++++++++-
lib/vsprintf.c | 5 +----
scripts/checkpatch.pl | 6 ++++--
23 files changed, 132 insertions(+), 35 deletions(-)
--
2.14.1
Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread
[RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:30 +0200
[RFC][PATCH v2 2/7] sections: split dereference_function_descriptor() Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:30 +0200
[RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
[RFC][PATCH v2 1/7] switch dereference_function_descriptor() to `unsigned long' Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
[RFC][PATCH v2 5/7] parisc64: Add .opd based function descriptor dereference Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
[RFC][PATCH v2 4/7] powerpc64: Add .opd based function descriptor dereference Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
[RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Joe Perches <joe@perches.com> - 2017-09-20 19:40 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Helge Deller <deller@gmx.de> - 2017-09-20 20:00 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Joe Perches <joe@perches.com> - 2017-09-20 20:30 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-09-21 09:50 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-09-21 02:30 +0200
Re: [RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning Joe Perches <joe@perches.com> - 2017-09-21 04:30 +0200
[RFC][PATCH v2 6/7] symbol lookup: use new kernel and module dereference functions Sergey Senozhatsky <sergey.senozhatsky@gmail.com> - 2017-09-20 18:40 +0200
Re: [RFC][PATCH v2 6/7] symbol lookup: use new kernel and module dereference functions Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-09-21 11:40 +0200
Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Helge Deller <deller@gmx.de> - 2017-09-20 22:20 +0200
Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-09-21 02:40 +0200
Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Santosh Sivaraj <santosh@fossix.org> - 2017-09-22 07:40 +0200
Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-09-22 10:10 +0200
RE: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers "Luck, Tony" <tony.luck@intel.com> - 2017-09-22 18:50 +0200
Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-09-25 09:10 +0200
RE: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers "Luck, Tony" <tony.luck@intel.com> - 2017-09-25 18:40 +0200
Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Helge Deller <deller@gmx.de> - 2017-09-25 19:10 +0200
csiph-web