On Thu, Jul 3, 2025 at 5:26 PM Michael S. Tsirkin <mst@redhat.com> wrote:
>
> Fix a couple of comments to match reality.
> Initialize config_driver_disabled to be consistent with
> other fields (note: the structure is already zero initialized,
> so this is not a bugfix as such).
>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Jason Wang <jasowang@redhat.com>
Thanks
> ---
> drivers/virtio/virtio.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c
> index 95d5d7993e5b..c441c8cc71ef 100644
> --- a/drivers/virtio/virtio.c
> +++ b/drivers/virtio/virtio.c
> @@ -147,7 +147,7 @@ EXPORT_SYMBOL_GPL(virtio_config_changed);
>
> /**
> * virtio_config_driver_disable - disable config change reporting by drivers
> - * @dev: the device to reset
> + * @dev: the device to disable
> *
> * This is only allowed to be called by a driver and disabling can't
> * be nested.
> @@ -162,7 +162,7 @@ EXPORT_SYMBOL_GPL(virtio_config_driver_disable);
>
> /**
> * virtio_config_driver_enable - enable config change reporting by drivers
> - * @dev: the device to reset
> + * @dev: the device to enable
> *
> * This is only allowed to be called by a driver and enabling can't
> * be nested.
> @@ -530,6 +530,7 @@ int register_virtio_device(struct virtio_device *dev)
> goto out_ida_remove;
>
> spin_lock_init(&dev->config_lock);
> + dev->config_driver_disabled = false;
> dev->config_core_enabled = false;
> dev->config_change_pending = false;
>
> --
> MST
>