Skip to content

Commit

Permalink
northd: Add a noop handler for northd SB mac binding.
Browse files Browse the repository at this point in the history
Signed-off-by: Numan Siddique <[email protected]>
  • Loading branch information
numansiddique authored and hzhou8 committed Nov 14, 2023
1 parent c6f336e commit 790e36e
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion northd/inc-proc-northd.c
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@ void inc_proc_northd_init(struct ovsdb_idl_loop *nb,
engine_add_input(&en_northd, &en_sb_mirror, NULL);
engine_add_input(&en_northd, &en_sb_meter, NULL);
engine_add_input(&en_northd, &en_sb_datapath_binding, NULL);
engine_add_input(&en_northd, &en_sb_mac_binding, NULL);
engine_add_input(&en_northd, &en_sb_dns, NULL);
engine_add_input(&en_northd, &en_sb_ha_chassis_group, NULL);
engine_add_input(&en_northd, &en_sb_ip_multicast, NULL);
Expand All @@ -196,6 +195,14 @@ void inc_proc_northd_init(struct ovsdb_idl_loop *nb,
engine_add_input(&en_northd, &en_global_config,
northd_global_config_handler);

/* northd engine node uses the sb mac binding table to
* cleanup mac binding entries for deleted logical ports
* and datapaths. Any update to to SB mac binding doesn't
* change the northd engine node state or data. Hence
* it is ok to add a noop_handler here. */
engine_add_input(&en_northd, &en_sb_mac_binding,
engine_noop_handler);

engine_add_input(&en_northd, &en_sb_port_binding,
northd_sb_port_binding_handler);
engine_add_input(&en_northd, &en_nb_logical_switch,
Expand Down

0 comments on commit 790e36e

Please sign in to comment.