Feature #12714: Show ``Inactive`` for Hardware Crypto output instead of empty field on System Information dashboard widget when nothing can be accelerated - 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 Target - All Closed Issues
24.03 Plus - All Closed Issues
24.03 Target - All Closed Issues
24.11 Plus - All Closed Issues
24.11 Plus - All Open Issues
24.11 Plus - Feedback Issues
24.11 Plus - Needs Attention/Work
24.11 Plus - New/Confirmed/In Progress Issues
24.11 Target - All Closed Issues
24.11 Target - All Open Issues
25.01 Plus - All Closed Issues
25.01 Plus - All Open Issues
25.01 Plus - Feedback Issues
25.01 Plus - Needs Attention/Work
25.01 Plus - New/Confirmed/In Progress Issues
25.01 Plus - Pull Request Review
25.01 Plus - Waiting on Merge
25.01 Target - All Closed Issues
25.01 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

Feature #12714

closed

Show ``Inactive`` for Hardware Crypto output instead of empty field on System Information dashboard widget when nothing can be accelerated

Added by → luckman212 almost 3 years ago. Updated over 2 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Jim Pingle

Category:

Dashboard

Target version:

2.7.0

Start date:

Due date:

% Done:

Estimated time:

Plus Target Version:

22.05

Release Notes:

Default

Description

on 22.01.r.20220120.1938 I'm seeing an empty string on the Dashboard Widget under System Information -> Hardware crypto:

I looked at the code, e.g.
https://github.com/pfsense/pfsense/blob/f73ace96e837ca2f0957a5fafe5794e033231c2e/src/usr/local/www/includes/functions.inc.php#L306

I guess it might not show anything there unless the crypto alg is being actively used?

But what is the intent of this display- to show capabilities or what's actually in use? (I thought capabilities...)

Here's a display from a 5100 running 2.4.5:

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Jim Pingle almost 3 years ago

Status changed from New to Not a Bug
Plus Target Version deleted (~~22.01~~)

It's working fine here on the latest RC on 6100 and several other platforms.

From the output in the widget above you don't have AES-NI nor QAT loaded (both say "inactive") so it's accurately saying nothing will be accelerated.

In the lower screenshot you have AES-NI active, so it's showing what it would accelerate.

If I have one or the other module loaded, it shows the expected list of ciphers. If I disable all of the acceleration modules and reboot, the field is blank, which is expected and correct.

Actions

Copy link

Updated by → luckman212 almost 3 years ago

Okay thanks Jim

FYI I have my crypto set to "QAT" on the 6100... which also seems to be the default setting.

So it's correct that the widget displays what's ACTIVE on the system, not what it SUPPORTS? Was this always true? Intuitively I would expect it to show what features are supported.

Actions

Copy link

Updated by Jim Pingle almost 3 years ago

It does show both supported (See the "Yes") and active/inactive. The Yes/No is whether or not the hardware supports the function, the active/inactive is whether or not the module is loaded. It's all there in the widget.

Actions

Copy link

Updated by → luckman212 almost 3 years ago

Ok, I would suggest a small wording change then

"Hardware crypto" → "Hardware crypto (active)"

And the crypto_accel_get_algs() function should return "Inactive" instead of an empty string in case $algs_str is empty....

Just would be clearer imo. If you're ok with it I can make a PR for this.

Actions

Copy link

Updated by → luckman212 almost 3 years ago

Jim Pingle it was a 1-line PR so I just went ahead...

https://github.com/pfsense/pfsense/pull/4553

Actions

Copy link

Updated by → luckman212 almost 3 years ago

Can this be "un-closed" so the PR can be considered

Actions

Copy link

Updated by Jim Pingle almost 3 years ago

Tracker changed from Bug to Feature
Subject changed from Dashboard - System Information > Hardware Crypto returns empty string on 22.01 rc to Show ``Inactive`` for Hardware Crypto output instead of empty field on System Information dashboard widget when nothing can be accelerated
Status changed from Not a Bug to Pull Request Review
Target version set to CE-Next
Plus Target Version set to Plus-Next