frr icon indicating copy to clipboard operation
frr copied to clipboard

PIM candidate BSR/RP support

Open Jafaral opened this issue 1 year ago • 3 comments

This Brings full BSR and RP candidate support to FRR (rfc 5059)

Replaces #9180 , in coordination with @eqvinox

Jafaral avatar Jul 23 '24 06:07 Jafaral

Ran the test and got this new memory leaks:

  ## showing active allocations in memory group pimd
pimd:     PIM BSR C-RP                  :      2 *         96
pimd:     PIM BSR range                 :      3 *         80
pimd:     PIM BSR C-RP range item       :      4 *         88

My expectation is that we should not be adding new code that leaks

donaldsharp avatar Jul 30 '24 14:07 donaldsharp

Ran the test and got this new memory leaks:

  ## showing active allocations in memory group pimd
pimd:     PIM BSR C-RP                  :      2 *         96
pimd:     PIM BSR range                 :      3 *         80
pimd:     PIM BSR C-RP range item       :      4 *         88

My expectation is that we should not be adding new code that leaks

will fix.

Jafaral avatar Aug 01 '24 04:08 Jafaral

Working on addressing comments from @nabahr and @mobash-rasool

Jafaral avatar Aug 26 '24 21:08 Jafaral

I addressed all comments on this PR.

Jafaral avatar Aug 29 '24 16:08 Jafaral