[SLOF] [PATCH] Add a global "dir" method

Benjamin Herrenschmidt benh at kernel.crashing.org
Thu Jun 9 22:13:41 AEST 2016


On Thu, 2016-06-09 at 03:27 -0500, Segher Boessenkool wrote:
> 
> Yes, ten gazillion inputs and outputs now ;-)
> 
> You already store that state slightly later, why not do it here already
> and have simpler code, much easier to read?

I tried, I actually had it that way... not that much easier and I'm
paranoid. If I want to come back in 2 years and use find-path for
something else, and forget it clobbers the global is-dir ... I really
really hate side effects :)

> If a word is hard to follow try factoring it.  Stack comments inside
> a word are almost always a bad idea.

I like how the firmworks code is done, they always have a right column
with the stack state ;-)

Cheers,
Ben.



More information about the SLOF mailing list