On 11-08-15, 20:11, Dan Carpenter wrote:
On Tue, Aug 11, 2015 at 08:29:38PM +0530, Viresh Kumar wrote:
This is weird to me, because we are going backwards. What happens if we goto free_table without adding anything?
It will WARN() today.
Then the current code is buggy.
Urg, it wouldn't WARN in this case. Sorry, didn't read it correctly earlier.
I suspect it's fine, but if it's a bug then this code still has problems.
I don't think we have a bug here, we never added anything and so don't need to free it.
What about if we only increment count when _opp_add_static_v2() succeeds
That's not what we want.
If the first call to _opp_add_static_v2() fails we call of_free_opp_table() and you say that triggers a WARN().
No it doesn't.
So, coming back to the point you made about freeing table on !count, because there were no nodes present in the DT opp table, we have never tried to add any OPPs. And so there is no need to call of_free_opp_table() in that case.
Do you still think the current code is wrong ?