On 2017-01-31 11:46, Thomas Pedersen wrote:
On Tue, Jan 31, 2017 at 11:33 AM, Rajkumar Manoharan wrote:
On 2017-01-31 09:51, Thomas Pedersen wrote:
[...]
+ if (paths_deactivated)
+ sta->mesh->fail_avg = 0;
.. why this indirection? Just reset mesh->fail_avg unconditionally in
this function?
Hmm... As fixed paths are not affected, resetting fail_avg for fixed
path
may give
wrong metric. no?
I don't think setting fail_avg = 0 will affect existing fixed mpath
code. Anyway metrics should not
affect whether a fixed mpath is chosen.
Great. Will remove conditional check. Thanks for your feedback Thomas.
-Rajkumar
