[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 10/11] xen/arm: smmuv3: Replace linux functions with xen functions.


  • To: Rahul Singh <Rahul.Singh@xxxxxxx>
  • From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • Date: Tue, 12 Jan 2021 11:35:15 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=cwKHPik9KTIlWB7plu+6fTnAo3M/ShlaTW3TKLrdTMQ=; b=OUDzGRMicq1ozE+3uiswIX8IoR4g2oJsJzC7pnSwbDDyNBOPMMJ7kpeT+ERJWDImo4NlkXxtlo1MAw2IA+q5JA+z6OW3N9L12pCr+0//MA8SEtB8GpyMTALtw3qnLqCK6cn49msxLDx2ed+WKwSEJIQH0Pfd0q0gh7FsQ1AmyWY8RicManTJLA3kjlz68JlPgbGWoL4O0t56CVcM0ZycTO+6I+TaAnfnSLybICqdhmPb8sqaH4c12J4N9wKAZHEKjjX41PVc+CbNutvzaMeQQ7BWt4MN4WQSIn4VNx6+80tyRIS8zXlZZ3fyR4yp4sfoNTCMiueE8+Frzsc7A062tw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CxQ/oPgQOvZXJwdqCJ7evDfCFw8iUrVsTpUpfmgDWw9G3KfkokIGe2Wdk0iv4aTkCz7YymV1MrSEYRptlRtM0iX6vfmAr5HPVhwxDbePuG7BBYEMPyv+wPBMsBUm0HJOr81SfE1MZZKOMxYrOjLUd3dvcrY74uANXs31nXeTt/lxsXzCKqkzKd0SeSKQqOqGBln80SR63s9IOaBMt3ZX7JqgD/fguXzKclVr3Aif0FoPfTkXLM92PDaox+vX1SX8gfG4sVhcUgIl3h4H9FpsxoWh4BRL4+D/KTmGBlePBPofYLDCCJTV5hoEc5+uwB5SGcdsovQrpKWhne7RhPWExg==
  • Authentication-results-original: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=arm.com;
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Delivery-date: Tue, 12 Jan 2021 11:35:50 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHW5c3ziqfg08UM3kK3n1F6OraRk6oj4mAA
  • Thread-topic: [PATCH v4 10/11] xen/arm: smmuv3: Replace linux functions with xen functions.

Hi Rahul,

> On 8 Jan 2021, at 14:46, Rahul Singh <Rahul.Singh@xxxxxxx> wrote:
> 
> Replace all Linux device tree handling function with the XEN
> functions.
> 
> Replace all Linux ktime function with the XEN time functions.
> 
> Signed-off-by: Rahul Singh <rahul.singh@xxxxxxx>
Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>

Cheers
Bertrand

> ---
> Changes in v3:
> - This patch is introduce in this version.
> Changes in V4:
> - Move this patch one patch earlier so that there is no need to remove
>   the code.
> ---
> xen/drivers/passthrough/arm/smmu-v3.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/xen/drivers/passthrough/arm/smmu-v3.c 
> b/xen/drivers/passthrough/arm/smmu-v3.c
> index f5f8b4c981..2dfadc6a65 100644
> --- a/xen/drivers/passthrough/arm/smmu-v3.c
> +++ b/xen/drivers/passthrough/arm/smmu-v3.c
> @@ -682,7 +682,7 @@ static void parse_driver_options(struct arm_smmu_device 
> *smmu)
>       int i = 0;
> 
>       do {
> -             if (of_property_read_bool(smmu->dev->of_node,
> +             if (dt_property_read_bool(smmu->dev->of_node,
>                                               arm_smmu_options[i].prop)) {
>                       smmu->options |= arm_smmu_options[i].opt;
>                       dev_notice(smmu->dev, "option %s\n",
> @@ -754,17 +754,17 @@ static void queue_inc_prod(struct arm_smmu_ll_queue *q)
>  */
> static int queue_poll_cons(struct arm_smmu_queue *q, bool sync, bool wfe)
> {
> -     ktime_t timeout;
> +     s_time_t timeout;
>       unsigned int delay = 1, spin_cnt = 0;
> 
>       /* Wait longer if it's a CMD_SYNC */
> -     timeout = ktime_add_us(ktime_get(), sync ?
> +     timeout = NOW() + MICROSECS(sync ?
>                                           ARM_SMMU_CMDQ_SYNC_TIMEOUT_US :
>                                           ARM_SMMU_POLL_TIMEOUT_US);
> 
>       while (queue_sync_cons_in(q),
>             (sync ? !queue_empty(&q->llq) : queue_full(&q->llq))) {
> -             if (ktime_compare(ktime_get(), timeout) > 0)
> +             if ((NOW() > timeout) > 0)
>                       return -ETIMEDOUT;
> 
>               if (wfe) {
> @@ -990,13 +990,13 @@ static void arm_smmu_cmdq_issue_cmd(struct 
> arm_smmu_device *smmu,
>  */
> static int __arm_smmu_sync_poll_msi(struct arm_smmu_device *smmu, u32 
> sync_idx)
> {
> -     ktime_t timeout;
> +     s_time_t timeout;
>       u32 val;
> 
> -     timeout = ktime_add_us(ktime_get(), ARM_SMMU_CMDQ_SYNC_TIMEOUT_US);
> +     timeout = NOW() + MICROSECS(ARM_SMMU_CMDQ_SYNC_TIMEOUT_US);
>       val = smp_cond_load_acquire(&smmu->sync_count,
>                                   (int)(VAL - sync_idx) >= 0 ||
> -                                 !ktime_before(ktime_get(), timeout));
> +                                 !(NOW() < timeout));
> 
>       return (int)(val - sync_idx) < 0 ? -ETIMEDOUT : 0;
> }
> @@ -2649,7 +2649,7 @@ static int arm_smmu_device_dt_probe(struct 
> platform_device *pdev,
>       u32 cells;
>       int ret = -EINVAL;
> 
> -     if (of_property_read_u32(dev->of_node, "#iommu-cells", &cells))
> +     if (!dt_property_read_u32(dev->of_node, "#iommu-cells", &cells))
>               dev_err(dev, "missing #iommu-cells property\n");
>       else if (cells != 1)
>               dev_err(dev, "invalid #iommu-cells value (%d)\n", cells);
> -- 
> 2.17.1
> 




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.