Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <39955431-2d01-c932-03c7-eecf5265177e@opensynergy.com>
Date:   Fri, 25 Feb 2022 19:21:05 +0100
Content-Language: en-US
To:     Cristian Marussi <cristian.marussi@arm.com>,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Cc:     sudeep.holla@arm.com, james.quinlan@broadcom.com,
        Jonathan.Cameron@Huawei.com, f.fainelli@gmail.com,
        etienne.carriere@linaro.org, vincent.guittot@linaro.org,
        souvik.chakravarty@arm.com, igor.skalkin@opensynergy.com,
        "Michael S. Tsirkin" <mst@redhat.com>,
        virtualization@lists.linux-foundation.org
References: <20220217131234.50328-1-cristian.marussi@arm.com>
 <20220217131234.50328-2-cristian.marussi@arm.com>
From:   Peter Hilber <peter.hilber@opensynergy.com>
Subject: Re: [PATCH v5 1/8] firmware: arm_scmi: Add a virtio channel refcount
In-Reply-To: <20220217131234.50328-2-cristian.marussi@arm.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MlEyczBlWitCZmlqbys2b2MzUmVESDl0ZVV3Wk1sSDhHMkgzWExGTVQ0L0FZ?=
 =?utf-8?B?eDRNdnlGZEN1NlNDaG53emFVUnhQSFJRZis5dXgzK2ppZEtJN0VNdFNYcVZq?=
 =?utf-8?B?cDNidHdTRnBIR3hsYkRPZjhmL0ZTQmdYcUZYaTdqZG16cks3UjhzTmF0Z293?=
 =?utf-8?B?eWFDTzNQYVhxMmNsdE5vR3BPSzI1QnZnR1hKZ0xnTFZWWTQ4RDZaMUdmRUh0?=
 =?utf-8?B?b2NUaVh0eEl3TmppU09icHdKQis0QkM1alB4eURrajVOWEY4TGRVRGMrb0Fp?=
 =?utf-8?B?REhsKzZkZU9NaUdrQkZ6cDJmNjlFNzh1ckc3dnoyeFJObmtPdDJPQ3l2eWlE?=
 =?utf-8?B?ekNvcGJGbXNWS1lzallmSWpWYzNuTlhyelRVR0FKUklRVmlxK3l5eGxGc2o4?=
 =?utf-8?B?MUdML0dYVjlJZzVnMDdMTXl3NkJwUElEcWw1V2g5THpTbjRNVjlVMGEwYmE4?=
 =?utf-8?B?ZDkrNlpTcTRGbU9NNkZLMzRISzcrdmQ2ZVdNT2VRbklWTXozeWpuRXIwc1RV?=
 =?utf-8?B?czZKaUFGVnRmVnVKdCtkeVFpTjVOWHNXeXZsbFVkMHBTNVEzcVZWdTNCbWpM?=
 =?utf-8?B?WnM0REZnUDlnOWFqSll1RHMwTHQzamw2bjB1S0ZKWDdDUzlQeDF5NklkZ0Mz?=
 =?utf-8?B?cFB2aWE0MTU0UjBKNWVJZU9RTFpHV0RuTWV1TlNMSkM1WDMrSUFZaTgrcEg1?=
 =?utf-8?B?UlFudFNlSWhlMm9KSk1OallFaHJUQ3pjd1djV0s5VVdQRllvSmRjbHZwWHJo?=
 =?utf-8?B?WXNEWUcvMW94ZnRRWGlxck8rdHI3MDJ6alhnL1ZodGZXMnlEeTVnS1Jxdjcv?=
 =?utf-8?B?cEM5OWNWVnkra0U1K2VBOGFUV2NsUmxVdlkrWjZlS1VtMFN1TGlQallMOEgv?=
 =?utf-8?B?bjRWZ3pBV1J2VER2Rk85dGNCTi9ZdGREU0N5VWVRalhiQXlwZEVOUUlOKzVi?=
 =?utf-8?B?RUh3NTc3ejBJbUlnQzFKUVZJdGY2ekFHYUsvZHdRMUFueGJqanhHLzgrL216?=
 =?utf-8?B?VFRuRmZGNFIrVk9wRzVUS2ZBM2svSXhNQ3FzMFhxUnM1VE4xVnRsK083TzJt?=
 =?utf-8?B?dUpLSEtZWkxwaHJDSzdJc2w0R0wzMXNCMjlCSkl3Yks4UWR0dFlQUWtnVXNM?=
 =?utf-8?B?dXlVTmxEdWdhcUR5NkdNL0RSU0xBcXpwTEJ1aHc3SWhSUFBLN0pnMFVNZDZY?=
 =?utf-8?B?SnlqY2dLU0NwZHFsTUlVbTVwT3JjTGdVYmtzS3hTc01sbWI3MG1lQjV5c2dP?=
 =?utf-8?B?SGZreTJNRnYvcXM0RGM3SWRqQWNPVkhsNk9EZWhJeDJVRWVBNkxQK1FmbjJi?=
 =?utf-8?B?Tm9MbkV1SllSNy9XSVdTMkk2WlFuR2tMMGVkSU03c0szY1hVTVBOMXBMN3hm?=
 =?utf-8?B?cDYyeFJQNDdMUXJCam1qbjMyT2tLTDBvVUpSQUFKN1AxdmNrVnoydHdza0kr?=
 =?utf-8?B?ZWlkdXdId05IRW16aHdpajBIMkE3cHBXbXpaNTVuSURWWnJnY0RxVEVSVFBX?=
 =?utf-8?B?cjNsOHlrOHU4elZEaW5oNXRudmpDanV2Y3dlVnloaGFEUE5qL050emJ3bDd3?=
 =?utf-8?B?YURodHFPcUlPWXJudmVBaEgvMEFCZWE1OCs4Q2Vwdi9LZm16WjJZWklMV0hl?=
 =?utf-8?B?KzFpSFo2QU5WQlVTOUVIK2I2N3plTGc3RDJSUUFvMnRNb050a0ZMeVE2UXVk?=
 =?utf-8?B?dmRIZS91UmhERzVUU3FteGUyVnZYYVdGeEo5SGMzKzNXamd2VWMzclBzWXN6?=
 =?utf-8?B?bnB2Q1FWMTg3TGNjNVVCdndMdmFidHY3VkUwczdvcVN4bGpBL3EybkZTamNj?=
 =?utf-8?B?ZkpqRXFZdDdpR0lCVUlDODRMWWtRNWo3bWExUnVSbFJnMzcvL05zVFlGTDlE?=
 =?utf-8?B?UEx0ZExXUXhqazNUaE02ak50ak4zMHRWTnJ5dFBBZjlFeU1tRjlJTVhwZ09V?=
 =?utf-8?B?cTdESVlUL1FkWm9sZEdqUkZiTHhwRlFYQ3ZCQzQ5MFFkVnYrR2dONXJuUWVh?=
 =?utf-8?B?VHkrOTBLbHV4L2x6NUFRMnZ6d3JMeUFmODladVQ0cmg3OE41TGc4NTV2Qjcz?=
 =?utf-8?B?Nk5HWThtRGlUa2U4Z2w5YmlHQ3UwZS9qYXRRcTRKOGN5UG10aE1uYitnM2NX?=
 =?utf-8?B?NS9pcjF0Q2VEd0h2clBCQ0toTXBxYkYvNHRXSXRCYm5hUTg3SWpRLzIxOUdU?=
 =?utf-8?B?THNOMUJaYTNJMGNVZC9udDV0UFRKeS9wc0hKcWdSOWk2VW9SUHlYQkkrS0Nn?=
 =?utf-8?Q?Cc8wq/53vsJRPYYDku9MLUONsRbzQJ+Xh/c+7YZUCw=3D?=
X-OriginatorOrg: opensynergy.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a53cac3e-e277-4f88-eeaf-08d9f88b971b
X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB8084.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Feb 2022 18:21:07.2405
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 800fae25-9b1b-4edc-993d-c939c4e84a64
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: tPHZW/iKIsnDg9ecp+FWQdCjEe04vioQmWZHFBZJX1+bbLbZRq7GeaAAiyVYRkmc7xjc4tzpQpswS5S21Y3lig==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB5707
X-TM-AS-ERS: 104.47.12.58-0.0.0.0
X-TMASE-Version: StarCloud-1.3-8.8.1001-26738.002
X-TMASE-Result: 10--5.529700-4.000000
X-TMASE-MatchedRID: gjZGo2H/wj//9O/B1c/Qy65i3jK3KDOoC/ExpXrHizyTsyhw1KPqkKKJ
        8yeXnfHAV620YhTcoiCy30IXTXvfRtIUKEcXjyBBR/j040fRFpK6G/3hYXpyaSt+gViu+UDkZ28
        gEzxS4tJ/jXwcoSRj3UdbuiZ3MdsdKjNqcTucdfhO5y1KmK5bJRSLgSFq3Tnj31GU/N5W5BC6k6
        Ky999wUE5KXWxauwqgP4ADLe03b+fzXojwcywrzM36paW7ZnFoYEpG2D2kcRrXFJ7W3lIp42zTR
        lDGU3Pz/t8GtPJsyDaf1JB9epETFkqyi4srEQAuIQrubkFPQdkgoSivyJYQQEYx760ONDcW3Ff7
        +xvoE8GXhTtgl0lKrTY21qAL0RG06B/jgOwSMZe/x8sBfQ/nQLTrV2IG143XkCszjODX6JLQk3h
        FSUm2ZqdHq3Rt1uM3TGg4ykZAAVhmdyCa5id7ahmCYUYerLHrKkd89GqHvUlULhp+W/a9mZjA2k
        GUhLCkasndy/28hYs7QajFbojNNZcFdomgH0lnOX/V8P8ail1ZDL1gLmoa/PoA9r2LThYYKrauX
        d3MZDUaA0E8UdrP6vAhVQMeX3mJfM0FelI66bKA64UdCEZ9HUVB7onZUGIH
X-TMASE-XGENCLOUD: 05bd0edb-ea2a-4d8e-90c0-cdeeeb213cea-0-0-200-0
X-TM-Deliver-Signature: F8AF89DC64E80F49349A396860086AAA
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=opensynergy.com;
        s=TM-DKIM-20210503141657; t=1645813270;
        bh=UaxWS77BNVRPGjMmWHHubNnoNbmOd0rBlKV7WOG8dEo=; l=10325;
        h=Date:To:From;
        b=FTc6KVTkmH7JK8kLOKbQGZi40drjViZDHDyWuk5W6yUZancMR6RLUhmV2JtRy4Sf5
         FX8sDU3326zE5aDxvkxhKmGm/grC1eGvUI2IedCxeBtzko+rL935u8ActB8/YBbSo1
         KTb+Y0JBmPPK604XD/FAmmyYQ6a0gq9kPFKiYzKBBulaxcaKQh5IvcG+2eOiIytv4l
         ouxa5q66+fOSk6xl6HW9keLE61jYSu7EF4MQYeE1eAKhVUNLq0/L0YWnRbr4/G6gDo
         nlUgBVW3tYq4LGIMwAgtTdq3Sg7MgiRPixMuAuXPWL1HahtjregmnY4ThtZxNdzR6o
         g+JKtwkUm+OuQ==
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 17.02.22 14:12, Cristian Marussi wrote:
> Currently SCMI VirtIO channels are marked with a ready flag and related
> lock to track channel lifetime and support proper synchronization at
> shutdown when virtqueues have to be stopped.
> 
> This leads to some extended spinlocked sections with IRQs off on the RX
> path to keep hold of the ready flag and does not scale well especially when
> SCMI VirtIO polling mode will be introduced.
> 
> Add an SCMI VirtIO channel dedicated refcount to track active users on both
> the TX and the RX path and properly enforce synchronization and cleanup at
> shutdown, inhibiting further usage of the channel once freed.
> 
> Cc: "Michael S. Tsirkin" <mst@redhat.com>
> Cc: Igor Skalkin <igor.skalkin@opensynergy.com>
> Cc: Peter Hilber <peter.hilber@opensynergy.com>
> Cc: virtualization@lists.linux-foundation.org
> Signed-off-by: Cristian Marussi <cristian.marussi@arm.com>

Acked-by: Peter Hilber <peter.hilber@opensynergy.com>

> ---
> v4 --> v5
> - removed unneeded virtqueue re-enable when fail to acquire channel in
>   complete_cb
> v2 --> v3
> - Break virtio device at shutdown while cleaning up SCMI channel
> ---
>  drivers/firmware/arm_scmi/virtio.c | 143 +++++++++++++++++++----------
>  1 file changed, 92 insertions(+), 51 deletions(-)
> 
> diff --git a/drivers/firmware/arm_scmi/virtio.c b/drivers/firmware/arm_scmi/virtio.c
> index fd0f6f91fc0b..ed00b072e981 100644
> --- a/drivers/firmware/arm_scmi/virtio.c
> +++ b/drivers/firmware/arm_scmi/virtio.c
> @@ -17,7 +17,9 @@
>   * virtqueue. Access to each virtqueue is protected by spinlocks.
>   */
>  
> +#include <linux/completion.h>
>  #include <linux/errno.h>
> +#include <linux/refcount.h>
>  #include <linux/slab.h>
>  #include <linux/virtio.h>
>  #include <linux/virtio_config.h>
> @@ -27,6 +29,7 @@
>  
>  #include "common.h"
>  
> +#define VIRTIO_MAX_RX_TIMEOUT_MS	60000
>  #define VIRTIO_SCMI_MAX_MSG_SIZE 128 /* Value may be increased. */
>  #define VIRTIO_SCMI_MAX_PDU_SIZE \
>  	(VIRTIO_SCMI_MAX_MSG_SIZE + SCMI_MSG_MAX_PROT_OVERHEAD)
> @@ -39,23 +42,21 @@
>   * @cinfo: SCMI Tx or Rx channel
>   * @free_list: List of unused scmi_vio_msg, maintained for Tx channels only
>   * @is_rx: Whether channel is an Rx channel
> - * @ready: Whether transport user is ready to hear about channel
>   * @max_msg: Maximum number of pending messages for this channel.
> - * @lock: Protects access to all members except ready.
> - * @ready_lock: Protects access to ready. If required, it must be taken before
> - *              lock.
> + * @lock: Protects access to all members except users.
> + * @shutdown_done: A reference to a completion used when freeing this channel.
> + * @users: A reference count to currently active users of this channel.
>   */
>  struct scmi_vio_channel {
>  	struct virtqueue *vqueue;
>  	struct scmi_chan_info *cinfo;
>  	struct list_head free_list;
>  	bool is_rx;
> -	bool ready;
>  	unsigned int max_msg;
> -	/* lock to protect access to all members except ready. */
> +	/* lock to protect access to all members except users. */
>  	spinlock_t lock;
> -	/* lock to rotects access to ready flag. */
> -	spinlock_t ready_lock;
> +	struct completion *shutdown_done;
> +	refcount_t users;
>  };
>  
>  /**
> @@ -76,6 +77,63 @@ struct scmi_vio_msg {
>  /* Only one SCMI VirtIO device can possibly exist */
>  static struct virtio_device *scmi_vdev;
>  
> +static void scmi_vio_channel_ready(struct scmi_vio_channel *vioch,
> +				   struct scmi_chan_info *cinfo)
> +{
> +	unsigned long flags;
> +
> +	spin_lock_irqsave(&vioch->lock, flags);
> +	cinfo->transport_info = vioch;
> +	/* Indirectly setting channel not available any more */
> +	vioch->cinfo = cinfo;
> +	spin_unlock_irqrestore(&vioch->lock, flags);
> +
> +	refcount_set(&vioch->users, 1);
> +}
> +
> +static inline bool scmi_vio_channel_acquire(struct scmi_vio_channel *vioch)
> +{
> +	return refcount_inc_not_zero(&vioch->users);
> +}
> +
> +static inline void scmi_vio_channel_release(struct scmi_vio_channel *vioch)
> +{
> +	if (refcount_dec_and_test(&vioch->users)) {
> +		unsigned long flags;
> +
> +		spin_lock_irqsave(&vioch->lock, flags);
> +		if (vioch->shutdown_done) {
> +			vioch->cinfo = NULL;
> +			complete(vioch->shutdown_done);
> +		}
> +		spin_unlock_irqrestore(&vioch->lock, flags);
> +	}
> +}
> +
> +static void scmi_vio_channel_cleanup_sync(struct scmi_vio_channel *vioch)
> +{
> +	unsigned long flags;
> +	DECLARE_COMPLETION_ONSTACK(vioch_shutdown_done);
> +
> +	/*
> +	 * Prepare to wait for the last release if not already released
> +	 * or in progress.
> +	 */
> +	spin_lock_irqsave(&vioch->lock, flags);
> +	if (!vioch->cinfo || vioch->shutdown_done) {
> +		spin_unlock_irqrestore(&vioch->lock, flags);
> +		return;
> +	}
> +	vioch->shutdown_done = &vioch_shutdown_done;
> +	virtio_break_device(vioch->vqueue->vdev);
> +	spin_unlock_irqrestore(&vioch->lock, flags);
> +
> +	scmi_vio_channel_release(vioch);
> +
> +	/* Let any possibly concurrent RX path release the channel */
> +	wait_for_completion(vioch->shutdown_done);
> +}
> +
>  static bool scmi_vio_have_vq_rx(struct virtio_device *vdev)
>  {
>  	return virtio_has_feature(vdev, VIRTIO_SCMI_F_P2A_CHANNELS);
> @@ -119,7 +177,7 @@ static void scmi_finalize_message(struct scmi_vio_channel *vioch,
>  
>  static void scmi_vio_complete_cb(struct virtqueue *vqueue)
>  {
> -	unsigned long ready_flags;
> +	unsigned long flags;
>  	unsigned int length;
>  	struct scmi_vio_channel *vioch;
>  	struct scmi_vio_msg *msg;
> @@ -130,27 +188,24 @@ static void scmi_vio_complete_cb(struct virtqueue *vqueue)
>  	vioch = &((struct scmi_vio_channel *)vqueue->vdev->priv)[vqueue->index];
>  
>  	for (;;) {
> -		spin_lock_irqsave(&vioch->ready_lock, ready_flags);
> -
> -		if (!vioch->ready) {
> -			if (!cb_enabled)
> -				(void)virtqueue_enable_cb(vqueue);
> -			goto unlock_ready_out;
> -		}
> +		if (!scmi_vio_channel_acquire(vioch))
> +			return;
>  
> -		/* IRQs already disabled here no need to irqsave */
> -		spin_lock(&vioch->lock);
> +		spin_lock_irqsave(&vioch->lock, flags);
>  		if (cb_enabled) {
>  			virtqueue_disable_cb(vqueue);
>  			cb_enabled = false;
>  		}
>  		msg = virtqueue_get_buf(vqueue, &length);
>  		if (!msg) {
> -			if (virtqueue_enable_cb(vqueue))
> -				goto unlock_out;
> +			if (virtqueue_enable_cb(vqueue)) {
> +				spin_unlock_irqrestore(&vioch->lock, flags);
> +				scmi_vio_channel_release(vioch);
> +				return;
> +			}
>  			cb_enabled = true;
>  		}
> -		spin_unlock(&vioch->lock);
> +		spin_unlock_irqrestore(&vioch->lock, flags);
>  
>  		if (msg) {
>  			msg->rx_len = length;
> @@ -161,19 +216,14 @@ static void scmi_vio_complete_cb(struct virtqueue *vqueue)
>  		}
>  
>  		/*
> -		 * Release ready_lock and re-enable IRQs between loop iterations
> -		 * to allow virtio_chan_free() to possibly kick in and set the
> -		 * flag vioch->ready to false even in between processing of
> -		 * messages, so as to force outstanding messages to be ignored
> -		 * when system is shutting down.
> +		 * Release vio channel between loop iterations to allow
> +		 * virtio_chan_free() to eventually fully release it when
> +		 * shutting down; in such a case, any outstanding message will
> +		 * be ignored since this loop will bail out at the next
> +		 * iteration.
>  		 */
> -		spin_unlock_irqrestore(&vioch->ready_lock, ready_flags);
> +		scmi_vio_channel_release(vioch);
>  	}
> -
> -unlock_out:
> -	spin_unlock(&vioch->lock);
> -unlock_ready_out:
> -	spin_unlock_irqrestore(&vioch->ready_lock, ready_flags);
>  }
>  
>  static const char *const scmi_vio_vqueue_names[] = { "tx", "rx" };
> @@ -273,35 +323,20 @@ static int virtio_chan_setup(struct scmi_chan_info *cinfo, struct device *dev,
>  		}
>  	}
>  
> -	spin_lock_irqsave(&vioch->lock, flags);
> -	cinfo->transport_info = vioch;
> -	/* Indirectly setting channel not available any more */
> -	vioch->cinfo = cinfo;
> -	spin_unlock_irqrestore(&vioch->lock, flags);
> -
> -	spin_lock_irqsave(&vioch->ready_lock, flags);
> -	vioch->ready = true;
> -	spin_unlock_irqrestore(&vioch->ready_lock, flags);
> +	scmi_vio_channel_ready(vioch, cinfo);
>  
>  	return 0;
>  }
>  
>  static int virtio_chan_free(int id, void *p, void *data)
>  {
> -	unsigned long flags;
>  	struct scmi_chan_info *cinfo = p;
>  	struct scmi_vio_channel *vioch = cinfo->transport_info;
>  
> -	spin_lock_irqsave(&vioch->ready_lock, flags);
> -	vioch->ready = false;
> -	spin_unlock_irqrestore(&vioch->ready_lock, flags);
> +	scmi_vio_channel_cleanup_sync(vioch);
>  
>  	scmi_free_channel(cinfo, data, id);
>  
> -	spin_lock_irqsave(&vioch->lock, flags);
> -	vioch->cinfo = NULL;
> -	spin_unlock_irqrestore(&vioch->lock, flags);
> -
>  	return 0;
>  }
>  
> @@ -316,10 +351,14 @@ static int virtio_send_message(struct scmi_chan_info *cinfo,
>  	int rc;
>  	struct scmi_vio_msg *msg;
>  
> +	if (!scmi_vio_channel_acquire(vioch))
> +		return -EINVAL;
> +
>  	spin_lock_irqsave(&vioch->lock, flags);
>  
>  	if (list_empty(&vioch->free_list)) {
>  		spin_unlock_irqrestore(&vioch->lock, flags);
> +		scmi_vio_channel_release(vioch);
>  		return -EBUSY;
>  	}
>  
> @@ -342,6 +381,8 @@ static int virtio_send_message(struct scmi_chan_info *cinfo,
>  
>  	spin_unlock_irqrestore(&vioch->lock, flags);
>  
> +	scmi_vio_channel_release(vioch);
> +
>  	return rc;
>  }
>  
> @@ -416,7 +457,6 @@ static int scmi_vio_probe(struct virtio_device *vdev)
>  		unsigned int sz;
>  
>  		spin_lock_init(&channels[i].lock);
> -		spin_lock_init(&channels[i].ready_lock);
>  		INIT_LIST_HEAD(&channels[i].free_list);
>  		channels[i].vqueue = vqs[i];
>  
> @@ -503,7 +543,8 @@ const struct scmi_desc scmi_virtio_desc = {
>  	.transport_init = virtio_scmi_init,
>  	.transport_exit = virtio_scmi_exit,
>  	.ops = &scmi_virtio_ops,
> -	.max_rx_timeout_ms = 60000, /* for non-realtime virtio devices */
> +	/* for non-realtime virtio devices */
> +	.max_rx_timeout_ms = VIRTIO_MAX_RX_TIMEOUT_MS,
>  	.max_msg = 0, /* overridden by virtio_get_max_msg() */
>  	.max_msg_size = VIRTIO_SCMI_MAX_MSG_SIZE,
>  };