lkml.org 
[lkml]   [2008]   [May]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2]: sparc64: Add ftrace support.
David Miller wrote:
> From: Steven Rostedt <srostedt@redhat.com>
> Date: Wed, 14 May 2008 09:32:18 -0400
>
>> If you add the following in arch/sparc64/kernel/Makefile
>>
>> ifdef CONFIG_DYNAMIC_FTRACE
>> obj-y += ftrace.o
>> # Do not trace the ftrace code itself
>> ORIG_CFLAGS := $(KBUILD_CFLAGS)
>> KBUILD_CFLAGS = $(if $(filter-out ftrace, $(basename $(notdir $@))), \
>> $(ORIG_CFLAGS), \
>> $(subst -pg,,$(ORIG_CFLAGS)))
>> endif
>>
>> Then you wont need the "notrace" annotations on the functions.
>
> That frankly looks like crap compared to the simple "notrace"
> annotations in the *.c file.
>
> This Makefile turd is in fact seemingly multiplying in various
> places in the ftrace tree, please reverse this trend.

I wish it wasn't so ugly. The Makefile scripts allow you to add more
CFLAGS to specific files but does not have a way to remove generic
flags. If there was a better way then I would love to do it.

The problem is that adding "notrace" on ever function that needs it,
also becomes quite messy. Especially when you have files where all
functions in the file needs a notrace and there is 40 or so small
functions that this applies to.

To add flags to file foo.c one simply needs to do

CFLAGS_foo.o = -newflags

It would be nice to have something like

CFLAGS_REMOVE_foo.o = -pg

Perhaps that is possible to do. Then the above "Makefile turd" wont be
so crappy.

-- Steve




\
 
 \ /
  Last update: 2008-05-15 00:01    [W:0.045 / U:0.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site