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

CPUID and NFD support: AVX_VNNI #844

Closed
ionutnechita opened this issue Jul 7, 2022 · 4 comments · Fixed by #851
Closed

CPUID and NFD support: AVX_VNNI #844

ionutnechita opened this issue Jul 7, 2022 · 4 comments · Fixed by #851
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@ionutnechita
Copy link

Hi NFD Team,

I have ADL CPU, but i not have AVX_VNNI in label list.
I have v0.11.0 image and this issue is observed.

{
  "beta.kubernetes.io/arch": "amd64",
  "beta.kubernetes.io/os": "linux",
  "feature.node.kubernetes.io/cpu-cpuid.ADX": "true",
  "feature.node.kubernetes.io/cpu-cpuid.AESNI": "true",
  "feature.node.kubernetes.io/cpu-cpuid.AVX": "true",
  "feature.node.kubernetes.io/cpu-cpuid.AVX2": "true",
  "feature.node.kubernetes.io/cpu-cpuid.CETIBT": "true",
  "feature.node.kubernetes.io/cpu-cpuid.CETSS": "true",
  "feature.node.kubernetes.io/cpu-cpuid.CMPXCHG8": "true",
  "feature.node.kubernetes.io/cpu-cpuid.FMA3": "true",
  "feature.node.kubernetes.io/cpu-cpuid.FXSR": "true",
  "feature.node.kubernetes.io/cpu-cpuid.FXSROPT": "true",
  "feature.node.kubernetes.io/cpu-cpuid.IBPB": "true",
  "feature.node.kubernetes.io/cpu-cpuid.LAHF": "true",
  "feature.node.kubernetes.io/cpu-cpuid.MOVBE": "true",
  "feature.node.kubernetes.io/cpu-cpuid.MOVDIR64B": "true",
  "feature.node.kubernetes.io/cpu-cpuid.MOVDIRI": "true",
  "feature.node.kubernetes.io/cpu-cpuid.OSXSAVE": "true",
  "feature.node.kubernetes.io/cpu-cpuid.SCE": "true",
  "feature.node.kubernetes.io/cpu-cpuid.SERIALIZE": "true",
  "feature.node.kubernetes.io/cpu-cpuid.SHA": "true",
  "feature.node.kubernetes.io/cpu-cpuid.STIBP": "true",
  "feature.node.kubernetes.io/cpu-cpuid.VMX": "true",
  "feature.node.kubernetes.io/cpu-cpuid.WAITPKG": "true",
  "feature.node.kubernetes.io/cpu-cpuid.X87": "true",
  "feature.node.kubernetes.io/cpu-cpuid.XSAVE": "true",
  "feature.node.kubernetes.io/cpu-hardware_multithreading": "true",
  "feature.node.kubernetes.io/cpu-model.family": "6",
  "feature.node.kubernetes.io/cpu-model.id": "154",
  "feature.node.kubernetes.io/cpu-model.vendor_id": "Intel",
  "feature.node.kubernetes.io/cpu-pstate.scaling_governor": "powersave",
  "feature.node.kubernetes.io/cpu-pstate.status": "active",
  "feature.node.kubernetes.io/cpu-pstate.turbo": "true",
  "feature.node.kubernetes.io/cpu-rdt.RDTL2CA": "true",
  "feature.node.kubernetes.io/kernel-config.NO_HZ": "true",
  "feature.node.kubernetes.io/kernel-config.NO_HZ_IDLE": "true",
  "feature.node.kubernetes.io/kernel-version.full": "5.15.0-40-generic",
  "feature.node.kubernetes.io/kernel-version.major": "5",
  "feature.node.kubernetes.io/kernel-version.minor": "15",
  "feature.node.kubernetes.io/kernel-version.revision": "0",
  "feature.node.kubernetes.io/pci-0300_8086.present": "true",
  "feature.node.kubernetes.io/pci-0300_8086.sriov.capable": "true",
  "feature.node.kubernetes.io/storage-nonrotationaldisk": "true",
  "feature.node.kubernetes.io/system-os_release.ID": "ubuntu",
  "feature.node.kubernetes.io/system-os_release.VERSION_ID": "22.04",
  "feature.node.kubernetes.io/system-os_release.VERSION_ID.major": "22",
  "feature.node.kubernetes.io/system-os_release.VERSION_ID.minor": "04",
  "gpu.intel.com/cards": "card0",
  "kubernetes.io/arch": "amd64",
  "kubernetes.io/hostname": "10.10.10.10",
  "kubernetes.io/os": "linux",
  "node-role.kubernetes.io/worker": "true"
}

cpuid -1 | grep AVX-VNNI
      AVX-VNNI: AVX VNNI neural network instrs = true

This CPU feature is not implemented in NFD?

BR,
Ionut Nechita
ionutnechita

@ionutnechita ionutnechita added the kind/feature Categorizes issue or PR as related to a new feature. label Jul 7, 2022
@marquiz
Copy link
Contributor

marquiz commented Jul 8, 2022

Hmm yes, we don't support it yet. TBH I wasn't even aware of AVX-VNNI (non AVX512-variant) 😇

On cpuid we rely on https://github.com/klauspost/cpuid so could you submit a feature request there (or even submit a PR implementing the feature 😊)? When we get that in the cpuid package we can pull it in NFD

@ionutnechita
Copy link
Author

Ok. Understood for this.
Will create new ticket for report this.

A good day.

@ionutnechita
Copy link
Author

This issue is solved in "klauspost/cpuid#106"

https://github.com/klauspost/cpuid/releases/tag/v2.1.0

It's possible to update NFD version with this fix?

@mythi
Copy link
Contributor

mythi commented Jul 19, 2022

This issue is solved in "klauspost/cpuid#106"

https://github.com/klauspost/cpuid/releases/tag/v2.1.0

It's possible to update NFD version with this fix?

See #851

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants