On 05/17/2018 09:20 AM, Greg KH wrote:
>> + file_data = kzalloc(sizeof(*file_data), GFP_KERNEL);
>> + if (!file_data)
>> + return -ENOMEM;
>> +
>> + pr_debug("%s - called\n", __func__);
> Please do not add "tracing" functions like this. The kernel has a
> wonderful built-in function tracing functionality, don't try to write
> your own. These lines should just be removed.
but pr_debug() works great with DYNAMIC_DEBUG.
Someone does not need to go all the way to tracing to get decent
debug capability.
--
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
