On 12/7/20 2:52 PM, Vivek Goyal wrote: > On Mon, Dec 07, 2020 at 10:03:24AM -0500, Paul Moore wrote: >> On Mon, Dec 7, 2020 at 9:43 AM Ondrej Mosnacek <[email protected]> wrote: >>> Hi everyone, >>> >>> In [1] we ran into a problem with the current handling of filesystem >>> labeling rules. Basically, it is only possible to specify either >>> genfscon or fs_use_xattr for a given filesystem, but in the case of >>> virtiofs, certain mounts may support security xattrs, while other ones >>> may not. > [ cc virtio-fs list and miklos ] >> Quickly skimming the linked GH issue, it appears that the problem >> really lies in the fact that virtiofs allows one to enable/disable >> xattrs at mount time. What isn't clear to me is why one would need to >> disable xattrs, can you explain that use case? Why does enabling >> xattrs in virtiofs cause problems? > Its not exactly a mount time option. Its a virtiofs file server option. > > xattr support by default is disabled because it has performance > penalty. Users can enable it if they want to. > > So if virtiofsd starts without xattr support and somebody runs a > VM with SELinux enabled, they should still be able to mount virtiofs, > I guess (instead of failing it).
I think the earlier virtio mount docs permitted an immutable SElinux spec for everything in a virtiofs mount , whether or not the the underlying host had xattrs enabled. Should the mount fail or should there be a default SELinux spec for the case there are no xattrs in the host and SELinux is running? There's a question for which good arguments exist on both sides. Case: I installed a package that had 'restorecon' commands in the install script, but that otherwise presupposed no SELinux awareness on the user's part. All the 'I want it to just work' users saw was a failed package installation. The post of a bug to the package maintainers, who then post a bug to the selinux devs. The SELinux support folks all say 'well it worked for us', and all the virtiofs users running on xattr enabled hosts 'see no problem'. Prevent that frustration: it's better to fail at mount time on the guest if selinux is enabled on the guest, xattrs and are not available in the host, with a very visible 'do this to fix it' error message. That way you generate awareness of a 'selinux issue' before it's a mystery. Harry _______________________________________________ Virtio-fs mailing list [email protected] https://www.redhat.com/mailman/listinfo/virtio-fs
