Re: [PATCH x86/mm] x86-64, NUMA: Fix distance table handling

From: Ingo Molnar
Date: Wed Mar 02 2011 - 05:36:44 EST



* Tejun Heo <tj@xxxxxxxxxx> wrote:

> FWIW, I'm not really decided about 80 vs. whatever column issue.

It only really matters when the underlying code structure is clearly inefficient:
too many indentations, etc.

but printks or function calls that go beyond 80 cols a bit do not deserve to be
line-broken.

> Having a common limit definitely helps a lot but it seems almost
> impossible to agree on one - is it 90, 95, 100 or 120? Given that, it
> almost seems just sticking to 80 might be the only doable solution.

The problem is that many sensible code structures break with a limit of 80.

So i'd suggest being permissive when the code is fine (printks, function calls at
the first or second level of indentation, etc.) and being conservative when the
underlying code is not fine.

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/