Re: [PATCH v2] x86: UV BAU fix for non-consecutive nasids

[Ingo Molnar]

* Cliff Wickman <cpw@xxxxxxx> wrote:

> From: Cliff Wickman <cpw@xxxxxxx>
> 
> (patch v2 has improved changelog and correction of a broken line)

but i did not point out just a single broken line - there's literally a dozen 
in your patch, still!

*All* of these measures i mentioned in my previous mail:

  - eliminate many repetitive strings
  - reduce indentation by the introduction of helper inlines
  - sensible shortening of variable/field/definition names

... applies to this particular case and will solve the problem.

This whole file is still utterly ugly and i'm not surprised at all it's showing 
bugs. Functions are still *way* too large and the code is hard to read.

If you don't know how to simplify a particular function then please ask but i 
see half a dozen things that could be done to clean up any of these big 
functions, just at a first quick glance. Those patterns of code ugliness repeat 
all over the whole file!

I dont mind if you do it as delta patches after this fix (so that the fix stays 
mergable) but we need to see progress in this code first, the pain in this file 
has been going on for *way* too long.

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/