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

DSA and IAA demos fail with "mmap: Operation not permitted" #1779

Closed
tkatila opened this issue Jul 15, 2024 · 6 comments · Fixed by #1795
Closed

DSA and IAA demos fail with "mmap: Operation not permitted" #1779

tkatila opened this issue Jul 15, 2024 · 6 comments · Fixed by #1795
Labels
bug Something isn't working dsa DSA device plugin related issue iaa IAA device plugin related issue

Comments

@tkatila
Copy link
Contributor

tkatila commented Jul 15, 2024

Describe the bug
As Ubuntu hwe kernel updated from 6.5.0-35 to 6.5.0-41, DSA e2e started to fail. The failure seems to be related to a change in the kernel where (due to a security issue) access to the DSA/IAA accelerators is being denied without SYS_RAWIO capability: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=796aec4a5b5850967af0c42d4e84df2d748d570b

To Reproduce

  • Upgrade kernel to Ubuntu HWE >=6.5.0-41 or use new enough vanilla kernel (6.9.3 is known to fail).
  • Run DSA demo example.

Expected behavior
Pod Completes successfully.

System (please complete the following information):

  • OS version: Ubuntu 22.04
  • Kernel version: HWE 6.5.0-41
  • Device plugins version: 0.30.0, not relevant
  • Hardware info: SPR with DSA

Additional context
Adding:

   securityContext:
      capabilities:
        add: ["SYS_RAWIO"]

might fix it temporarily.

@eero-t
Copy link
Contributor

eero-t commented Jul 15, 2024

man capabilities lists that capability allowing following:

CAP_SYS_RAWIO
              •  Perform I/O port operations (iopl(2) and ioperm(2));
              •  access /proc/kcore;
              •  employ the FIBMAP ioctl(2) operation;
              •  open  devices  for  accessing  x86  model-specific  registers
                 (MSRs, see msr(4));
              •  update /proc/sys/vm/mmap_min_addr;
              •  create memory mappings at addresses below the value specified
                 by /proc/sys/vm/mmap_min_addr;
              •  map files in /proc/bus/pci;
              •  open /dev/mem and /dev/kmem;
              •  perform various SCSI device commands;
              •  perform certain operations on hpsa(4) and cciss(4) devices;
              •  perform a range of device-specific operations  on  other  de‐
                 vices.

Which of the listed procfs + sysfs files & dirs are mounted, and need to be mounted, inside DSA/IAA example containers?

@HackToday
Copy link

I added the capability in my pod yaml, it can run with no issue. So the capability is needed, I think.

https://github.com/intel/intel-device-plugins-for-kubernetes/blob/main/cmd/dsa_plugin/README.md#testing-and-demos

@tkatila
Copy link
Contributor Author

tkatila commented Jul 15, 2024

CAP_SYS_RAWIO
              •  perform a range of device-specific operations  on  other  devices.

Which of the listed procfs + sysfs files & dirs are mounted, and need to be mounted, inside DSA/IAA example containers?

I do not believe we need any extra host files or directories. The kernel patch is quite self-explaining: for certain devices check for sys_rawio capability. If no capability, return eperm.

Another question is how should the dsa plugin handle this? If at all.

@eero-t
Copy link
Contributor

eero-t commented Jul 15, 2024

I do not believe we need any extra host files or directories.

I meant is there something that should be masked out of the container...

@tkatila
Copy link
Contributor Author

tkatila commented Jul 15, 2024

I meant is there something that should be masked out of the container...

Probably "everything" that the sys_rawio brings. DSA demo has worked without it before and the kernel side only needs the sys_rawio capability flag.

@mythi
Copy link
Contributor

mythi commented Jul 22, 2024

I do not believe we need any extra host files or directories.

I meant is there something that should be masked out of the container...

runtimes mask out some proc paths by default but it'd be good to cross-check these. In general, adding add: ["SYS_RAWIO"] should be OK, IMO. We add something similar for QAT (IPC_LOCK).

@tkatila tkatila added bug Something isn't working dsa DSA device plugin related issue iaa IAA device plugin related issue labels Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dsa DSA device plugin related issue iaa IAA device plugin related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants