Skip to content

gk: add dylib.c.gk_unload_bpf_flow_handler()#454

Merged
AltraMayor merged 5 commits intomasterfrom
del_bpf
Dec 28, 2020
Merged

gk: add dylib.c.gk_unload_bpf_flow_handler()#454
AltraMayor merged 5 commits intomasterfrom
del_bpf

Conversation

@AltraMayor
Copy link
Owner

The Lua function dylib.c.gk_unload_bpf_flow_handler() enables the dynamic configuration block to unload a BPF program in runtime.

This patch prepares the code to have more functions that scan
the flow table to flush flow entries according to given criteria.
The generalized flush_flow_table() can now simplify the code of
gk_synchronize().
This patch is just one step toward a generalized version of
the mechanism that synchronizes flow tables and FIB of
the GK block.
This patch generalizes synchronize_gk_instances(),
so it can be reused.
The Lua function dylib.c.gk_unload_bpf_flow_handler() enables
the dynamic configuration block to unload a BPF program in runtime.
@AltraMayor AltraMayor added the Operational demand This issue would make Gatekeeper safer and/or cheaper to operate label Dec 22, 2020
@AltraMayor AltraMayor added this to the First deployment milestone Dec 22, 2020
@mengxiang0811
Copy link
Collaborator

Patch gk: generalize flush_flow_table() is ready for merge.

@mengxiang0811
Copy link
Collaborator

Patch gk: use flush_flow_table() in gk_synchronize() is ready for merge.

@mengxiang0811
Copy link
Collaborator

Patch gk: drop field num_updated_instances from struct gk_fib is ready for merge.

@mengxiang0811
Copy link
Collaborator

Patch gk: generalize synchronize_gk_instances() is ready for merge.

@mengxiang0811
Copy link
Collaborator

Patch gk: add dylib.c.gk_unload_bpf_flow_handler() is ready for merge.

@AltraMayor AltraMayor merged commit 95a4536 into master Dec 28, 2020
@AltraMayor AltraMayor deleted the del_bpf branch December 28, 2020 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Operational demand This issue would make Gatekeeper safer and/or cheaper to operate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants