-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
openvpn: DCO not enabled on snapshot #22472
Comments
You need to build package enabling dco option in menuconfig. note no such option is available for 23.05 |
Well I figured I could build it from source if I wanted to but it seems odd to ship a binary kmod as an official package but then not ship a user space binary which can use it which is why I opened the issue. |
build it and benchmark it on average mips if possible vs non-dco version. 10x speedup looks mpre like marketing pitch/ Sarcasm not welcome, you literarily request removal of dco kmod. |
I don't have a mips router as my build information in the original issue states. Also I wasn't being sarcastic? That was a genuine statement, without both packages the kmod is useless. I figured that was a mistake since there is PR enabling it but it wasn't enabled in actual builds. Additionally I have benchmarked it on amd64 and it provides quite a large performance increase, maybe it's not as noticeable on mips but I have no way of testing that. |
Build ipk and validate support. Extensive testing needs mips and aarch having significant improvement to make essentially bigger package default. |
If package size is an issue then what about adding a separate dco enabled package that way users could still use it without causing a size increase for storage constrained platforms. |
The burden of proof and PR is on you. |
This is clearly not a productive conversation. I will wait till one of the actual maintainers replies since I'm not looking to submit a PR, this is simply an issue report about a missing feature. I'm not an OpenWRT contributor and based on PR history neither are you and I'm rather tired of your attitude. |
I would also like to have a DCO enabled package for openvpn. I aggree with @asdfjkluiop as it stands now, the kmod is useless and I would add misleading, especially based on its description. |
DCO not enabled on OpenWRT snapshot
Maintainer: @AuthorReflex
Environment: amd64 snapshot build r24162-65380dc41c
Description:
As far as I can tell PR #21547 should've added DCO support but it doesn't seem to be present. I spent a bit of time trying to get DCO working to no avail and then realized when I run
openvpn --version
it does not show the [DCO] feature flag like it should. As far as I can tell the openvpn userspace program is not being build with DCO enabled, at least on amd64 builds.The text was updated successfully, but these errors were encountered: