Re: [PATCH v2] scripts/recordmcount.{c,pl}: support -ffunction-sections .text.* section names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 12, 2021 at 02:17:49PM -0800, Manoj Gupta wrote:
> From: Joe Lawrence <joe.lawrence@xxxxxxxxxx>
> 
> commit 9c8e2f6d3d361439cc6744a094f1c15681b55269 upstream.
> 
> When building with -ffunction-sections, the compiler will place each
> function into its own ELF section, prefixed with ".text".  For example,
> a simple test module with functions test_module_do_work() and
> test_module_wq_func():
> 
>   % objdump --section-headers test_module.o | awk '/\.text/{print $2}'
>   .text
>   .text.test_module_do_work
>   .text.test_module_wq_func
>   .init.text
>   .exit.text
> 
> Adjust the recordmcount scripts to look for ".text" as a section name
> prefix.  This will ensure that those functions will be included in the
> __mcount_loc relocations:
> 
>   % objdump --reloc --section __mcount_loc test_module.o
>   OFFSET           TYPE              VALUE
>   0000000000000000 R_X86_64_64       .text.test_module_do_work
>   0000000000000008 R_X86_64_64       .text.test_module_wq_func
>   0000000000000010 R_X86_64_64       .init.text
> 
> Link: http://lkml.kernel.org/r/1542745158-25392-2-git-send-email-joe.lawrence@xxxxxxxxxx
> 
> Signed-off-by: Joe Lawrence <joe.lawrence@xxxxxxxxxx>
> Signed-off-by: Steven Rostedt (VMware) <rostedt@xxxxxxxxxxx>
> 
> [Manoj: Resolve conflict on 4.4.y/4.9.y because of missing 42c269c88dc1]
> Signed-off-by: Manoj Gupta <manojgupta@xxxxxxxxxx>
> ---
> 
> Changes v1 -> v2:
>   Change "nc" to "Manoj"

Now queued up, thanks.

greg k-h



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux