Add GUI setting for VLANs PCP
Add support for optionally setting VLANs priority (PCP) through the GUI, saved as part of the xml configuration.
Suggested patches are attached (also include fix for #3737).
#7 Updated by Chris Christensen over 4 years ago
I tested this patch using 2.2.3-Release:
- https://github.com/pfsense/pfsense-tools/compare/pfsense:RELENG_2_2_3...christianchristensen:4133 (PR: https://github.com/pfsense/pfsense-tools/pull/6 )
- https://github.com/pfsense/pfsense/compare/RELENG_2_2_3...christianchristensen:4133 (PR: https://github.com/pfsense/pfsense/pull/1757 )
I used the build instructions from: https://forum.pfsense.org/index.php?topic=83785.0 to generate an image.
This all appears to have worked very nicely - the UI is exposed and the added functionality appears to function. For me was related to Google Fiber QoS ( https://support.google.com/fiber/faq/3333053?hl=en ) that requires 802.1Q VLAN pcp set to 2. (otherwise upload speed is rate-limited)
As for the test, the results speak:
#11 Updated by Chris Buechler almost 4 years ago
This looks to work fine. It's configured correctly, and the appropriate PCP mark is on the traffic.
Chris or anyone else with a real world scenario here, could you please test the latest 2.3 snapshot from https://snapshots.pfsense.org and report back?