Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-20 Thread Sergey Senozhatsky
On (09/20/17 11:51), Michael Ellerman wrote: [..] > > unlike ppc_function_entry(), printk() can get called on any symbol, > > not just function pointers. > > > > for example, > > > > cat /proc/kallsyms | grep shrinker_rwsem > > 81a4b1e0 d shrinker_rwsem > > Yep, good point. So your patch

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-20 Thread Sergey Senozhatsky
On (09/20/17 11:51), Michael Ellerman wrote: [..] > > unlike ppc_function_entry(), printk() can get called on any symbol, > > not just function pointers. > > > > for example, > > > > cat /proc/kallsyms | grep shrinker_rwsem > > 81a4b1e0 d shrinker_rwsem > > Yep, good point. So your patch

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-19 Thread Michael Ellerman
Sergey Senozhatsky writes: > On (09/19/17 20:22), Michael Ellerman wrote: >> > On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: >> >> We are moving towards separate kernel and module function descriptor >> >> dereference callbacks. This patch enables it for

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-19 Thread Michael Ellerman
Sergey Senozhatsky writes: > On (09/19/17 20:22), Michael Ellerman wrote: >> > On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: >> >> We are moving towards separate kernel and module function descriptor >> >> dereference callbacks. This patch enables it for powerpc64. >> >> >> >> For pointers

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-19 Thread Sergey Senozhatsky
On (09/19/17 20:22), Michael Ellerman wrote: > > On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: > >> We are moving towards separate kernel and module function descriptor > >> dereference callbacks. This patch enables it for powerpc64. > >> > >> For pointers that belong to the kernel > >> -

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-19 Thread Sergey Senozhatsky
On (09/19/17 20:22), Michael Ellerman wrote: > > On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: > >> We are moving towards separate kernel and module function descriptor > >> dereference callbacks. This patch enables it for powerpc64. > >> > >> For pointers that belong to the kernel > >> -

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-19 Thread Michael Ellerman
"Naveen N. Rao" writes: > On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: >> We are moving towards separate kernel and module function descriptor >> dereference callbacks. This patch enables it for powerpc64. >> >> For pointers that belong to the kernel >> -

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-19 Thread Michael Ellerman
"Naveen N. Rao" writes: > On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: >> We are moving towards separate kernel and module function descriptor >> dereference callbacks. This patch enables it for powerpc64. >> >> For pointers that belong to the kernel >> - Added __start_opd and __end_opd

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-16 Thread Sergey Senozhatsky
On (09/16/17 15:13), Naveen N. Rao wrote: [..] > Would it be simpler to just use kernel_text_address() and dereference > everything else? See commit 83e840c770f2c5 ("powerpc64/elfv1: Only > dereference function descriptor for non-text symbols") for a related > patch. I had this idea, see

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-16 Thread Sergey Senozhatsky
On (09/16/17 15:13), Naveen N. Rao wrote: [..] > Would it be simpler to just use kernel_text_address() and dereference > everything else? See commit 83e840c770f2c5 ("powerpc64/elfv1: Only > dereference function descriptor for non-text symbols") for a related > patch. I had this idea, see

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-16 Thread Naveen N. Rao
On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: > We are moving towards separate kernel and module function descriptor > dereference callbacks. This patch enables it for powerpc64. > > For pointers that belong to the kernel > - Added __start_opd and __end_opd pointers, to track the kernel >

Re: [PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-16 Thread Naveen N. Rao
On 2017/09/16 12:53PM, Sergey Senozhatsky wrote: > We are moving towards separate kernel and module function descriptor > dereference callbacks. This patch enables it for powerpc64. > > For pointers that belong to the kernel > - Added __start_opd and __end_opd pointers, to track the kernel >

[PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-15 Thread Sergey Senozhatsky
We are moving towards separate kernel and module function descriptor dereference callbacks. This patch enables it for powerpc64. For pointers that belong to the kernel - Added __start_opd and __end_opd pointers, to track the kernel .opd section address range; - Added

[PATCH 3/5] powerpc64: Add .opd based function descriptor dereference

2017-09-15 Thread Sergey Senozhatsky
We are moving towards separate kernel and module function descriptor dereference callbacks. This patch enables it for powerpc64. For pointers that belong to the kernel - Added __start_opd and __end_opd pointers, to track the kernel .opd section address range; - Added