Bug #1353: Number of queues possible - pfSense - pfSense bugtracker

Custom queries

2.4.5-p1 - Resolved/Closed
2.5.0 - Resolved/Closed
2.5.1 - Resolved/Closed
2.5.2 - Resolved/Closed
2.6.0 - Resolved/Closed
2.7.0 - Resolved/Closed
2.7.1 - All Open Issues
2.7.1 - Resolved/Closed
2.7.2 - Resolved/Closed
2.8.0 - All Open Bugs
2.8.0 - All Open Features
2.8.0 - All Open Issues
2.8.0 - All Open Regressions
2.8.0 - Feedback
2.8.0 - Needs Attention
2.8.0 - New/Confirmed
2.8.0 - Pull Requests
2.8.0 - Regressions affecting 2.7.0
2.8.0 - Resolved/Closed
21.02.2/2.5.1 - Resolved/Closed
21.05 Plus - All Closed Issues
21.05.1 Plus Target - All Closed Issues
21.05.1 Target - All Closed Issues
22.01 Plus - All Closed Issues
22.01 Target - All Closed Issues
22.05 Plus - All Closed Issues
22.05 Target - All Closed Issues
23.01 Plus - All Closed Issues
23.01 Target - All Closed Issues
23.05 Plus - All Closed Issues
23.05 Target - All Closed Issues
23.05.1 Plus - All Closed Issues
23.05.1 Target - All Closed Issues
23.09 Plus - All Closed Issues
23.09 Target - All Closed Issues
23.09.1 Plus - All Closed Issues
23.09.1 Plus - All Open Issues
23.09.1 Target - All Closed Issues
23.09.1 Target - All Open Issues
24.03 Plus - All Closed Issues
24.03 Plus - All Open Issues
24.03 Plus - Feedback Issues
24.03 Plus - Needs Attention/Work
24.03 Plus - New/Confirmed/In Progress Issues
24.03 Plus - Pull Request Review
24.03 Plus - Waiting on Merge
24.03 Target - All Closed Issues
24.03 Target - All Open Issues
All Open Issues assigned to Me
All Open Pull Requests
Any Target - All Open Regressions
Any Target - Feedback Issues
CE-Next - All Closed Issues (Move to specific target)
CE-Next - All Open Issues
CE-Next - Feedback (Likely needs target changed)
New Issues by Category - Future Target
New Issues by Category - No Target
New Issues by Category - No Target+Future
No Target - All Open Issues (Base Only)
No Target - New Issues (Base Only)
No Target - New Issues (Base and Packages)
Release Notes - Plus Target Version (DO NOT EDIT)
Release Notes - Target Version (DO NOT EDIT)

Actions

Copy link

Bug #1353

closed

Number of queues possible

Added by Ermal Luçi about 13 years ago. Updated over 3 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Renato Botelho

Category:

Traffic Shaper (ALTQ)

Target version:

2.5.0

Start date:

03/15/2011

Due date:

% Done:

100%

Estimated time:

Plus Target Version:

Release Notes:

Affected Version:

All

Affected Architecture:

Description

ALTQ algorithm for PRIQ has a forced 15 queue limitation.
While the other algorithms are forced to a 4096 queue limit build time limit.

Triggered by http://forum.pfsense.org/index.php/topic,34435.0.html it would be nice to have the GUI spit out a nice error message when limit reached or remove the limit at all if possible.

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Ermal Luçi almost 13 years ago

Target version changed from 2.0 to 2.1

This is not so critical for 2.0

Actions

Copy link

Updated by Chris Buechler about 12 years ago

Target version deleted (~~2.1~~)

Actions

Copy link

Updated by Viktor Gurov almost 4 years ago

PRIQ also doesn't allow the use of the same priority for more than one queue:

pfctl: q3 does not have a unique priority on interface vtnet0
/tmp/rules.debug:50: errors in queue definition

https://github.com/pfsense/FreeBSD-src/blob/ce9563d5729455de9a9e988d20aa21da79799f47/sys/net/altq/altq_priq.h#L43:

#define    PRIQ_MAXPRI    16    /* upper limit of the number of priorities */

Input validation fix:
https://github.com/pfsense/pfsense/pull/4327

Actions

Copy link

Updated by Jim Pingle almost 4 years ago

Status changed from New to Pull Request Review
Target version set to 2.5.0

Actions

Copy link

Updated by Renato Botelho almost 4 years ago

Status changed from Pull Request Review to Feedback
Assignee set to Renato Botelho
% Done changed from 0 to 100

PR has been merged. Thanks!

Actions

Copy link

Updated by Viktor Gurov almost 4 years ago

error on the latest snapshot:

[11-Jun-2020 09:54:05 Europe/Moscow] PHP Warning:  Invalid argument supplied for foreach() in /etc/inc/shaper.inc on line 1576

Need to check for $parent->queues array first,
Fix:
https://github.com/pfsense/pfsense/pull/4353

Actions

Copy link

Updated by Jim Pingle almost 4 years ago

Status changed from Feedback to Pull Request Review

Actions

Copy link

Updated by Renato Botelho almost 4 years ago

Status changed from Pull Request Review to Feedback

PR has been merged. Thanks!

Actions

Copy link

Updated by Viktor Gurov almost 4 years ago

Check for priority value duplication only for CBQ/FAIRQ/PRIQ queues:
https://github.com/pfsense/pfsense/pull/4387

otherwise it tries to check HFSC for prio duplication

Actions

Copy link

#10

Updated by Jim Pingle almost 4 years ago

Status changed from Feedback to Pull Request Review

Actions

Copy link

#11

Updated by Renato Botelho almost 4 years ago

Status changed from Pull Request Review to Feedback

PR has been merged. Thanks!

Actions

Copy link

#12

Updated by Anonymous over 3 years ago

Status changed from Feedback to Resolved

Actions

Copy link

#13

Updated by Viktor Gurov over 3 years ago

Status changed from Resolved to New

unique priority is mandatory now, which is incorrect
from https://www.freebsd.org/cgi/man.cgi?query=pf.conf&apropos=0&sektion=0&manpath=FreeBSD+12.1-RELEASE+and+Ports&arch=default&format=html:

priq  Priority Queueing.  Queues are flat attached    to the interface,
       thus, queues    cannot have further child queues.  Each    queue has a
       unique priority assigned, ranging from 0 to 15.  Packets in the
       queue with the highest priority are processed first.

same for FAIRQ

but in case of CBQ priority can be the same for different queues

Fix:
https://gitlab.netgate.com/pfSense/pfSense/-/merge_requests/19

Actions

Copy link

#14