Re: [PATCH 2/2] qemuSecurityChownCallback: Don't initialize storage file subsystem for local file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 26, 2020 at 12:18:03 +0100, Peter Krempa wrote:
> virStorageFileSupportsSecurityDriver ends up initializing the storage
> file backend which after the recent changes to the daemon architecture
> may end up dlopening of the backend modules.
> 
> Since this is required only for remote storage we can optimize the call
> by moving the check whether the backend is supported to the branch which
> deals with remote storage. This will prevent the pointless dlopen for
> local files.
> 
> This is amplified by the fact that we now do most storage driver
> operations in forks so the storage backends may not even be initialized
> in the main daemon. On the other hand, not having gluster linked in the
> main daemon is a big win, so we should keep only the forks having to
> link with it.

Actually, I'll redact this as standard startup of a VM still invokes
the storage file backends, so they will be linked eventually on normal
usage. I've observed this after libvirtd restart.





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux