Re: [PATCH] vfio/pci: make nointxmask and disable_idle_d3 module params read-only

[Alex Williamson <alex@shazbot.org>]

On Wed, 10 Jun 2026 13:47:34 +0800
Guixin Liu <kanie@linux.alibaba.com> wrote:

> The nointxmask and disable_idle_d3 module parameters are registered with
> S_IWUSR, exposing a writable node under /sys/module/vfio_pci/parameters/.
> However, writing to them at runtime has no effect on driver behaviour:
> 
>  - Both values are copied into vfio-pci-core's own static variables via
>    vfio_pci_core_set_params(), which is only called once at module init
>    time.  A later write to the vfio_pci front-end variable is never
>    propagated to the core.
> 
>  - disable_idle_d3 additionally gates paired runtime PM get/put calls
>    across the probe/remove (register/unregister) and open/close
>    (enable/disable) life cycles.  Changing it at runtime would tear
>    these pairs apart and leak or underflow the runtime PM usage counter.
> 
>  - nointxmask is latched into per-device state (vdev->nointx /
>    vdev->pci_2_3) at device open time, so a later change can only affect
>    newly opened devices and yields inconsistent behaviour.
> 
> These parameters are effectively load-time constants.  Drop the write
> permission so they match disable_vga, which is already read-only, and to
> stop misleading users into thinking they are runtime tunable.
> 
> Signed-off-by: Guixin Liu <kanie@linux.alibaba.com>
> ---
>  drivers/vfio/pci/vfio_pci.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Thanks for the patch, I think this actually highlights a broader
problem.  Making disable_idle_d3 a read-only module parameter in
vfio-pci doesn't change that vfio-pci can be unload and reloaded with
different parameter values, therefore the gap is not closed.  We need
to use per device setting, latched at device probe, which actually
also returns some value to these parameters being runtime modified.
This feature was inadvertently lost in the vfio-pci-core split.

I've copied you on the broader fix:

https://lore.kernel.org/kvm/20260611182528.4004073-1-alex.williamson@nvidia.com/T/#t

Thanks,
Alex