Re: [RFC, 7/8] powerpc/xmon: Drop 'valid' from the condition inside 'dump_segments'

On 07/21/2015 03:30 PM, Michael Ellerman wrote:

On Tue, 2015-21-07 at 06:58:45 UTC, Anshuman Khandual wrote:

quoted

quoted

From: "khandual@linux.vnet.ibm.com" <redacted>

Value of 'valid' is zero when 'esid' is zero and it does not matter
when 'esid' is non-zero. 

Yes it does. It tells you whether the entry is valid?

Yeah but it does not change the outcome of the if condition check
here. Non-zero esid will make the condition test pass irrespective
of the value of 'valid'. Yes, valid will be checked inside the code
block to print details, the point was value of valid does not make
any difference to the 'if' condition check in the first place.
Unless I am getting tricked here some how :)

In practice maybe you only see invalid entries that are entirely zero, and so
they get skipped anyway, but that's not guaranteed.