bug: fix kvm device not being properly forwarded
Co-authored-by: Autumn Liu <al63244@my.utexas.edu> Reviewed-on: #9 Co-authored-by: autumn <aut.tea7@gmail.com>
This commit was merged in pull request #9.
This commit is contained in:
@@ -222,13 +222,15 @@ rec {
|
||||
: Bind mounts, keyed by mount point.
|
||||
|
||||
`resources.devices`
|
||||
: Host device nodes to expose, as paths, added to the unit's
|
||||
`DeviceAllow=`. nspawn populates a handful of privileged devices
|
||||
(`/dev/kvm`, `/dev/net/tun`, ...) into the container's own `/dev` once
|
||||
the cgroup allows it, and silently skips ones the host doesn't have -
|
||||
no `resources.fs` bind mount needed. Nodes keep their host ownership,
|
||||
so a consumer inside needs `isolation.privateUsers = "identity"` for
|
||||
the owning gid to mean anything.
|
||||
: Host device nodes to expose, as paths. Bound into the container at the
|
||||
same path and added to the unit's `DeviceAllow=` - the cgroup
|
||||
permission alone doesn't get you a working node (nothing can `mknod` a
|
||||
device inside a user namespace), and the bind alone leaves the cgroup
|
||||
denying `open()`, so both are needed. The bind is unconditional, so
|
||||
the device must actually exist on whatever host runs the container.
|
||||
Nodes keep their host ownership, so a consumer inside needs
|
||||
`isolation.privateUsers = "identity"` for the owning gid to mean
|
||||
anything.
|
||||
|
||||
`isolation.nixDaemon`
|
||||
: Whether the Nix daemon and CLI are available inside the container.
|
||||
@@ -295,6 +297,10 @@ rec {
|
||||
+ lib.optionalString useIdmap ":idmap"
|
||||
) allBindMounts;
|
||||
bindMountHostPaths = map (bm: bm.hostPath) (lib.attrValues allBindMounts);
|
||||
# DeviceAllow= alone only grants cgroup permission - nspawn can't
|
||||
# mknod a device node inside a user namespace, so the node still
|
||||
# needs an explicit bind from the host.
|
||||
deviceFlags = map (d: "--bind=${d}") devices;
|
||||
inherit (deriveIdentity ifPrefix.nspawn name)
|
||||
iface
|
||||
mac
|
||||
@@ -381,7 +387,7 @@ rec {
|
||||
nixosContainer = {
|
||||
inherit autoStart privateUsers;
|
||||
bindMounts = { };
|
||||
extraFlags = bindMountExtraFlags;
|
||||
extraFlags = bindMountExtraFlags ++ deviceFlags;
|
||||
privateNetwork = true;
|
||||
extraVeths = linkVeths;
|
||||
config =
|
||||
|
||||
Reference in New Issue
Block a user