diff options
author | anlan_cs <vic.lan@pica8.com> | 2023-05-30 10:45:45 +0200 |
---|---|---|
committer | anlan_cs <vic.lan@pica8.com> | 2023-05-30 11:29:38 +0200 |
commit | b016b552a6169e0e2f6c3a8d10429f9fe63b4add (patch) | |
tree | effa29f553a6fdf59a391b1dea68c47672424b02 /lib | |
parent | Merge pull request #13602 from opensourcerouting/fix/reuse_subgroup_process_a... (diff) | |
download | frr-b016b552a6169e0e2f6c3a8d10429f9fe63b4add.tar.xz frr-b016b552a6169e0e2f6c3a8d10429f9fe63b4add.zip |
pimd: Fix missing pimreg interface
`pimregX` of specific vrf can be deleted from kernel after this vrf
is deleted. However, then `pimdregX` will never come back to
kernel after adding it ( the same vrf ) back. That is to say, it exists
only in daemon, but not in kernel.
The root cause is this `pimregX` is not really deleted for its special
usage, and `pim_if_create_pimreg()` wants reusing it.
I have tried 4 solutions:
1. Remove the `configured` flag of `pimregX`, allow its deletion.
A few timers still use `pimregX` after its deletion. So, not adopted.
2. Remove `pimregX` by vrf hook in `pim_instance_terminate()`.
It has no vrf id there. So, not adopted.
3. Reuse `pimregX` in `pim_if_create_pimreg()`.
If `pim->regiface->info` is true, then reuse old `pimregX` and only call
`pim_if_add_vif()` to install it into kernel. But at that time, it maybe
is in default VRF. The `pim_zebra_interface_set_master()` doesn't work
at that time because it shouldn't wait there for its changing into
correct VRF. So, not adopted.
4. Not reuse it.
If `pim->regiface->info` is true, there must have been pim instance with
VRF deleted and created before. Actually delele old one in
`pim_if_create_pimreg()`, then recreate new one.
Finally, this PR adopted the fourth solution.
Fixes #13454
Signed-off-by: anlan_cs <vic.lan@pica8.com>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions