Skip to content
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.

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

Only QAT devices in bus 0000 are handled by QAT device plugin's init container #1999

Open
dcoyle-intel opened this issue Feb 26, 2025 · 0 comments · May be fixed by #2000
Open

Only QAT devices in bus 0000 are handled by QAT device plugin's init container #1999

dcoyle-intel opened this issue Feb 26, 2025 · 0 comments · May be fixed by #2000
Labels
bug Something isn't working qat QAT device plugin related issue

Comments

@dcoyle-intel
Copy link

Describe the bug
Only QAT devices in bus 0000 are handled by QAT device plugin's init container. Therefore for devices in bus 0001 (or any other I guess), services cannot be enabled, VFs cannot be created or VFs cannot be bound to a driver such as vfio-pci

In demo/qat-init.sh, bus 0000: is always prepended to the rest of the device address... see the 3rd line of sysfs_config() below. The same is also present in sriov_enable()

sysfs_config() {
  if [ "$SERVICES_ENABLED_FOUND" = "TRUE" ]; then
    for dev in $DEVS; do
      DEVPATH="/sys/bus/pci/devices/0000:$dev"
      PCI_DEV=$(cat "$DEVPATH"/device 2> /dev/null)
      if [ "$PCI_DEV" != "$QAT_4XXX_DEVICE_PCI_ID" ] && [ "$PCI_DEV" != "$QAT_401XX_DEVICE_PCI_ID" ] && [ "$PCI_DEV" != "$QAT_402XX_DEVICE_PCI_ID" ] && [ "$PCI_DEV" != "$QAT_420XX_DEVICE_PCI_ID" ]; then
        continue
      fi

To Reproduce
Run the QAT device plugin's init container on a server with QAT devices with bus 0001 e.g. 0001:01:00.0

Expected behavior
Services should be configured, VFs created and bound to vfio-pci for QAT devices in bus 0001

@tkatila tkatila added bug Something isn't working qat QAT device plugin related issue labels Feb 26, 2025
@mythi mythi linked a pull request Feb 27, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working qat QAT device plugin related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants