Skip to content

Conversation

@PolynomialDivision
Copy link
Contributor

Adding a delete function at runtime allows the babeld ipc to add and remove interfaces with filters. Further, it also allows more dynamic routing selection.

Adding a delete function at runtime allows the babeld ipc to add and
remove interfaces with filters. Further, it also allows more dynamic
routing selection.

Signed-off-by: Nick Hainke <vincent@systemli.org>
@jech
Copy link
Owner

jech commented Oct 30, 2022

I'm extremely confused. This PR appears to add two functions that are not used anywhere. Is part of the patch missing?

@PolynomialDivision
Copy link
Contributor Author

It's used by the IPC I added in openwrt:
https://github.com/openwrt/routing/pull/911/files

@jech
Copy link
Owner

jech commented Oct 30, 2022

I see. It feels weird to add dead code to babeld, so I'm hesitating between adding the funcitonality to babeld's native IPC, or to ask you to maintain it in your fork. Do you have any opinions?

@PolynomialDivision
Copy link
Contributor Author

I can maintain it as patch. I just wanted to get some feedback from you if I can do such things. :)

@jech
Copy link
Owner

jech commented Oct 31, 2022

The code looks good to me.

Perhaps you could explain why you need this functionality, so we can think together whether there's a better solution to your problem?

@PolynomialDivision
Copy link
Contributor Author

I'm adding and removing dynamically interfaces. Sometimes they even have the same name. I'm also adding filters on them. When I remove now an interface, I want to also to remove the filters. Maybe a clean removing of an interface and its filters would be some good solution for me.

@jech
Copy link
Owner

jech commented Nov 1, 2022 via email

@PolynomialDivision
Copy link
Contributor Author

Thanks! :)

@PolynomialDivision
Copy link
Contributor Author

Any news? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants