[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1720512816.0395122-1-xuanzhuo@linux.alibaba.com>
Date: Tue, 9 Jul 2024 16:13:36 +0800
From: Xuan Zhuo <xuanzhuo@...ux.alibaba.com>
To: Jason Wang <jasowang@...hat.com>
Cc: virtualization@...ts.linux.dev,
linux-kernel@...r.kernel.org,
davem@...emloft.net,
edumazet@...gle.com,
kuba@...nel.org,
pabeni@...hat.com,
netdev@...r.kernel.org,
Venkat Venkatsubra <venkat.x.venkatsubra@...cle.com>,
"Gia-Khanh Nguyen" <gia-khanh.nguyen@...cle.com>,
mst@...hat.com,
jasowang@...hat.com,
eperezma@...hat.com
Subject: Re: [PATCH net-next v3 1/3] virtio: rename virtio_config_enabled to virtio_config_core_enabled
On Tue, 9 Jul 2024 16:02:12 +0800, Jason Wang <jasowang@...hat.com> wrote:
> Following patch will allow the config interrupt to be disabled by a
> specific driver via another boolean. So this patch renames
> virtio_config_enabled and relevant helpers to
> virtio_config_core_enabled.
>
> Cc: Venkat Venkatsubra <venkat.x.venkatsubra@...cle.com>
> Cc: Gia-Khanh Nguyen <gia-khanh.nguyen@...cle.com>
> Signed-off-by: Jason Wang <jasowang@...hat.com>
Reviewed-by: Xuan Zhuo <xuanzhuo@...ux.alibaba.com>
Thanks.
> ---
> drivers/virtio/virtio.c | 22 +++++++++++-----------
> include/linux/virtio.h | 4 ++--
> 2 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c
> index b968b2aa5f4d..73bab89b5326 100644
> --- a/drivers/virtio/virtio.c
> +++ b/drivers/virtio/virtio.c
> @@ -127,7 +127,7 @@ static void __virtio_config_changed(struct virtio_device *dev)
> {
> struct virtio_driver *drv = drv_to_virtio(dev->dev.driver);
>
> - if (!dev->config_enabled)
> + if (!dev->config_core_enabled)
> dev->config_change_pending = true;
> else if (drv && drv->config_changed)
> drv->config_changed(dev);
> @@ -143,17 +143,17 @@ void virtio_config_changed(struct virtio_device *dev)
> }
> EXPORT_SYMBOL_GPL(virtio_config_changed);
>
> -static void virtio_config_disable(struct virtio_device *dev)
> +static void virtio_config_core_disable(struct virtio_device *dev)
> {
> spin_lock_irq(&dev->config_lock);
> - dev->config_enabled = false;
> + dev->config_core_enabled = false;
> spin_unlock_irq(&dev->config_lock);
> }
>
> -static void virtio_config_enable(struct virtio_device *dev)
> +static void virtio_config_core_enable(struct virtio_device *dev)
> {
> spin_lock_irq(&dev->config_lock);
> - dev->config_enabled = true;
> + dev->config_core_enabled = true;
> if (dev->config_change_pending)
> __virtio_config_changed(dev);
> dev->config_change_pending = false;
> @@ -322,7 +322,7 @@ static int virtio_dev_probe(struct device *_d)
> if (drv->scan)
> drv->scan(dev);
>
> - virtio_config_enable(dev);
> + virtio_config_core_enable(dev);
>
> return 0;
>
> @@ -340,7 +340,7 @@ static void virtio_dev_remove(struct device *_d)
> struct virtio_device *dev = dev_to_virtio(_d);
> struct virtio_driver *drv = drv_to_virtio(dev->dev.driver);
>
> - virtio_config_disable(dev);
> + virtio_config_core_disable(dev);
>
> drv->remove(dev);
>
> @@ -455,7 +455,7 @@ int register_virtio_device(struct virtio_device *dev)
> goto out_ida_remove;
>
> spin_lock_init(&dev->config_lock);
> - dev->config_enabled = false;
> + dev->config_core_enabled = false;
> dev->config_change_pending = false;
>
> INIT_LIST_HEAD(&dev->vqs);
> @@ -512,14 +512,14 @@ int virtio_device_freeze(struct virtio_device *dev)
> struct virtio_driver *drv = drv_to_virtio(dev->dev.driver);
> int ret;
>
> - virtio_config_disable(dev);
> + virtio_config_core_disable(dev);
>
> dev->failed = dev->config->get_status(dev) & VIRTIO_CONFIG_S_FAILED;
>
> if (drv && drv->freeze) {
> ret = drv->freeze(dev);
> if (ret) {
> - virtio_config_enable(dev);
> + virtio_config_core_enable(dev);
> return ret;
> }
> }
> @@ -578,7 +578,7 @@ int virtio_device_restore(struct virtio_device *dev)
> if (!(dev->config->get_status(dev) & VIRTIO_CONFIG_S_DRIVER_OK))
> virtio_device_ready(dev);
>
> - virtio_config_enable(dev);
> + virtio_config_core_enable(dev);
>
> return 0;
>
> diff --git a/include/linux/virtio.h b/include/linux/virtio.h
> index 96fea920873b..a6f6df72f01a 100644
> --- a/include/linux/virtio.h
> +++ b/include/linux/virtio.h
> @@ -115,7 +115,7 @@ struct virtio_admin_cmd {
> * struct virtio_device - representation of a device using virtio
> * @index: unique position on the virtio bus
> * @failed: saved value for VIRTIO_CONFIG_S_FAILED bit (for restore)
> - * @config_enabled: configuration change reporting enabled
> + * @config_core_enabled: configuration change reporting enabled by core
> * @config_change_pending: configuration change reported while disabled
> * @config_lock: protects configuration change reporting
> * @vqs_list_lock: protects @vqs.
> @@ -132,7 +132,7 @@ struct virtio_admin_cmd {
> struct virtio_device {
> int index;
> bool failed;
> - bool config_enabled;
> + bool config_core_enabled;
> bool config_change_pending;
> spinlock_t config_lock;
> spinlock_t vqs_list_lock;
> --
> 2.31.1
>
Powered by blists - more mailing lists