Does the arbiter tree and mask logic overlap in rr_arb_tree.sv ?

[Hu1chen]

I think the lower_idx or upper_idx(#209, 210) is the next port to be executed, why add arbitration tree(#254-)?
is this redundant or there is a problem in my understanding?