On Fri, Jul 25, 2025 at 11:10:15AM +0100, Daniel P. Berrangé wrote: > On Fri, Jul 25, 2025 at 11:46:54AM +0200, Hector Cao wrote: > > What do you think of this design: > > 1) Create a file msr.conf inside src/util/modules-load.d/ > > 2) Modify src/util/meson.build to install this file to /etc/modules-load.d/ > > > > Optional: > > - Do you think it is necessary to add the configuration option to enable > > this modules-load file installation ? > > We should probably have a meson_options.txt entry as some distros have it > as a built-in, so it would be wierd to see a file installed to load it.
The other thing to keep in mind is that the file needs to be shipped conditionally, on x86 only. Remind me again, why don't we just use virKModLoad() for this? It's functionality that already exists and can easily be reused, and it would make for an overall cleaner solution IMO. -- Andrea Bolognani / Red Hat / Virtualization
