Project

General

Profile

Actions

Feature #4133

closed

Add GUI setting for VLANs PCP

Added by Clement Barnier almost 7 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Interfaces
Target version:
Start date:
12/20/2014
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

Description

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).


Files

pf_pcp_git.diff (12.2 KB) pf_pcp_git.diff PHP interface patch (pfSense GIT) Clement Barnier, 12/20/2014 08:34 AM
pf_pcp_source.diff (12.5 KB) pf_pcp_source.diff Source patch Clement Barnier, 12/20/2014 08:34 AM
pf_pcp_tools.diff (1.24 KB) pf_pcp_tools.diff pfPorts patch (php55-pfSense-module) Clement Barnier, 12/20/2014 08:34 AM
pf_pcp_git.diff (12.2 KB) pf_pcp_git.diff Corrected PHP interface patch (missing brackets) Clement Barnier, 01/03/2015 05:33 AM
Actions #1

Updated by Renato Botelho almost 7 years ago

  • Target version changed from 2.2 to 2.2.1

Target to 2.2.1 for future review, 2.2 is now in RC state

Actions #2

Updated by Clement Barnier almost 7 years ago

I just realized that the blob in the git diff regarding configuration compatibility has a few missing brackets. I guess I decided to change it at the last minute.
Tested on 2.2-RC without problem.

Actions #3

Updated by Jim Thompson almost 7 years ago

  • Assignee set to Renato Botelho
Actions #4

Updated by Chris Buechler over 6 years ago

  • Target version changed from 2.2.1 to 2.2.2
Actions #5

Updated by Chris Buechler over 6 years ago

  • Target version changed from 2.2.2 to 2.2.3
Actions #6

Updated by Chris Buechler over 6 years ago

  • Target version changed from 2.2.3 to 2.3
Actions #7

Updated by Chris Christensen over 6 years ago

I tested this patch using 2.2.3-Release:

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:

Actions #8

Updated by Chris Buechler over 6 years ago

Thanks Chris! We'll get this integrated for 2.3.

Actions #9

Updated by Renato Botelho almost 6 years ago

  • Assignee changed from Renato Botelho to Luiz Souza

Luiz, can you please check kernel patch available at https://github.com/pfsense/pfsense-tools/pull/6?

Actions #10

Updated by Luiz Souza almost 6 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

All the parts are committed, should work out-of-box in 2.3.

Actions #11

Updated by Chris Buechler almost 6 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?

Actions #12

Updated by Chris Buechler almost 6 years ago

  • Status changed from Feedback to Resolved

Chris confirmed via email that this worked fine (aside from hitting #5453 and having other issues from that)

Actions

Also available in: Atom PDF