Re: [PATCH 2/2][RESEND] tracing: Shrink the size of struct ftrace_event_field
From: Li Zefan
Date: Fri Jul 26 2013 - 23:14:39 EST
>> @@ -111,6 +111,11 @@ static int __trace_define_field(struct list_head *head, const char *type,
>> field->size = size;
>> field->is_signed = is_signed;
> I think we should just change is_signed to bool. At least the parameter.
> Or we can make the assignment: field->is_signed = !!is_signed; and nuke
> the check below.
Changing field->is_signed to bool won't shrink the size of the struct.
I prefer: field->is_signed = !!is_signed.
>> + WARN_ON(offset >= (1 << 12));
>> + WARN_ON(size >= (1 << 12));
>> + WARN_ON(is_signed >= (1 << 1));
>> + WARN_ON(field->filter_type >= (1 << 4));
> Note, the test for field->filter_type is wrong.
> We should make a helper macro:
> #define VERIFY_SIZE(type) WARN_ON(type > field->type)
> and then have:
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/