Skip to content
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

Knob for vlanhwfilter disable in system settings #252

Closed
fichtner opened this issue Jul 10, 2015 · 6 comments
Closed

Knob for vlanhwfilter disable in system settings #252

fichtner opened this issue Jul 10, 2015 · 6 comments
Assignees
Labels
feature Adding new functionality
Milestone

Comments

@fichtner
Copy link
Member

via: https://forum.opnsense.org/index.php?topic=896.0

@fichtner fichtner added feature Adding new functionality help wanted Contributor missing / timeout labels Jul 10, 2015
@fichtner fichtner added this to the 16.1 milestone Jul 10, 2015
@AdSchellevis
Copy link
Member

Is this what your looking for 2a3a6a1 ?

An option to set all technical parameters for an interface is probably not the best thing to do in the legacy code, maybe more something for the wishlist for future releases? (to prevent too much changes in legacy code and possible transition problems between versions).

@fichtner
Copy link
Member Author

This is exactly the amount of code needed for this to be solved short-term. So, yes, Long-term the interfaces in the new GUI need a better system. :)

As for the patch: yes, thank you, almost. There separate hardware code that sets vlanhwfilter unconditionally on a vlan though:

legacy_interface_flags($if, 'vlanhwtag vlanmtu vlanhwfilter');

@AdSchellevis
Copy link
Member

No problem, will look at that in an hour or so.

@fichtner fichtner removed the help wanted Contributor missing / timeout label Jul 13, 2015
@fichtner
Copy link
Member Author

Awesome, thank you. :D

@AdSchellevis
Copy link
Member

Your welcome.
I guess it makes most sense to disable all hw options when hwfilter is disabled.
Commit cd41d6b contains the fix.

@fichtner
Copy link
Member Author

I tried to backport these changes, but for some reason it doesn't work. It seems to be a larger issue. I can't even unset the tags in ifconfig, at least for the e1000 driver I've tested so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding new functionality
Development

No branches or pull requests

2 participants