On 08/26/2015 09:38 AM, Timur Tabi wrote:
> On Wed, Aug 19, 2015 at 5:07 PM, Al Stone <al.stone@xxxxxxxxxx> wrote:
>> + pr_err("Variable length MADT subtable %d is wrong size: %d\n",
>> + entry->type, entry->length);
>> + return 1;
>> + }
>> + }
>> + } else {
>> + if (entry->length != len) {
>> + pr_err("MADT subtable %d is wrong size: %d\n",
>> + len, entry->type);
>
> Can we make these a little more descriptive?
>
> pr_err("Variable length MADT subtable type %d is wrong size: %d,
> should be %d\n",
> entry->type, entry->length, len);
>
> pr_err("MADT subtable type %d is wrong size: %d, should be %d\n",
> entry->type, entry->length, len);
>
Sure. It's always a fine line between verbose and not enough info.
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3@xxxxxxxxxx
-----------------------------------
--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|