Re: [PATCH 1/2] perf, x86: Add basic Ivy Bridge support v2
From: Ingo Molnar
Date: Mon Jun 11 2012 - 04:16:37 EST
* Andi Kleen <andi@xxxxxxxxxxxxxx> wrote:
> + char *name;
> + name = "Ivy";
> + name = "Sandy";
> - pr_cont("SandyBridge events, ");
> + pr_cont("%sBridge events, ", name);
Second round review feedback: you'll have to remove this ugly
obfuscation, the kernel is already complex as-is. In performance
insensitive code paths we want straight, obvious code without
tricks.
Please resubmit the patches with this fixed.
Thanks,
Ingo
--
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/