Skip to content

QAT Virtual Machine Virtual Function Support #4375

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.

Sign up for GitHub

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

Open
wants to merge 4 commits into
base: current
Choose a base branch
from
Open

QAT Virtual Machine Virtual Function Support #4375

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a62ce1d
Update system_acceleration.py
smith-daniels Jan 11, 2025
3b8f50f
Update show_acceleration.py
smith-daniels Jan 11, 2025
d9492c2
Merge branch 'vyos:current' into current
smith-daniels Mar 1, 2025
7174356
Update system_acceleration.py
smith-daniels Mar 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion src/conf_mode/system_acceleration.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,11 +70,12 @@ def verify(qat):
# PCI id | Chipset
# 19e2 -> C3xx
# 37c8 -> C62x
# 37c9 -> C62xvf
# 0435 -> DH895
# 6f54 -> D15xx
# 18ee -> QAT_200XX
data = re.findall(
'(8086:19e2)|(8086:37c8)|(8086:0435)|(8086:6f54)|(8086:18ee)', output)
'(8086:19e2)|(8086:37c8)|(8086:37c9)|(8086:0435)|(8086:6f54)|(8086:18ee)', output)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From a purely aesthetic point of you, the diff might have looked better if the new value was added after 8086:0435. But I have nothing against the substance.

Copy link
Author

@smith-daniels smith-daniels Mar 4, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought because it was a subset of the physical C62xx physical device that if fit directly after that, but I understand your point from the perspective of reading the code change. I will squash the commits for this as requested.

# If QAT devices found
if not data:
raise ConfigError('No QAT acceleration device found')
Expand Down
4 changes: 2 additions & 2 deletions src/op_mode/show_acceleration.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
def detect_qat_dev():
output, err = popen('lspci -nn', decode='utf-8')
if not err:
data = re.findall('(8086:19e2)|(8086:37c8)|(8086:0435)|(8086:6f54)', output)
data = re.findall('(8086:19e2)|(8086:37c8)|(8086:37c9)|(8086:0435)|(8086:6f54)', output)
# QAT devices found
if data:
return
Expand Down Expand Up @@ -93,7 +93,7 @@ def check_qat_if_conf():
if args.hw:
detect_qat_dev()
# Show availible Intel QAT devices
call('lspci -nn | egrep -e \'8086:37c8|8086:19e2|8086:0435|8086:6f54\'')
call('lspci -nn | egrep -e \'8086:37c8|8086:37c9|8086:19e2|8086:0435|8086:6f54\'')
elif args.flow and args.dev:
check_qat_if_conf()
call('cat '+get_qat_proc_path(args.dev)+"fw_counters")
Expand Down
Loading