Re: [PATCH] kernel: Fix ftrace.c compiler warning when calling ftrace_nop_replace()

From: Eduardo Silva
Date: Sun May 22 2011 - 22:27:40 EST

On Sun, May 22, 2011 at 9:36 PM, H. Peter Anvin <hpa@xxxxxxxxx> wrote:
> On 05/22/2011 06:19 PM, Eduardo Silva wrote:
>> the function ftrace_nop_replace() returns a 'static const unsigned char *'
>> value, so when the caller perform a direct assignment to a
>> 'static unsigned char *', the compiler raise the following warning message:
>>  ftrace.c:308:6: warning: assignment discards qualifiers from pointer target type
>>  ftrace.c:318:6: warning: assignment discards qualifiers from pointer target type
>> Adding the proper casts the message goes away.
>> Signed-off-by: Eduardo Silva <edsiper@xxxxxxxxx>
> This quiets a warning of something that potentially looks like a real
> bug.  Perhaps "new" should be const?

The value is returned in the following function:

static const unsigned char *ftrace_nop_replace(void)
return ideal_nops[NOP_ATOMIC5];

ideal_nops seems to be constant and architecture dependent, so make
sense to return it as a constant pointer... i think that we need more
comments about it.


>        -hpa
> --
> H. Peter Anvin, Intel Open Source Technology Center
> I work for Intel.  I don't speak on their behalf.

Eduardo Silva
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at