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

RE: [PATCH v3] xen/arm: don't assign domU static-mem to dom0 as reserved-memory


  • To: Stefano Stabellini <sstabellini@xxxxxxxxxx>, "julien@xxxxxxx" <julien@xxxxxxx>
  • From: Penny Zheng <Penny.Zheng@xxxxxxx>
  • Date: Thu, 11 Nov 2021 08:26:26 +0000
  • Accept-language: 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=4wg42JrRQEESIOLWc7L/k/2yYrcDRK/cYBkON798A60=; b=D7HuA0iy6jMgQMqPzunbW0JGIPe4K3P6I3RhCIulkoGj+1/oAdAv15FUw6YnlSDdsu/Ye6irlLvXH6aWQ5R4zAhHt7v5ypZrFA5TyWGl4RKMn8TyKwSwgmkkPJEuY77eGxpSNvPFHCsDsByG6pW68vkDe4hE0OcLgUMKHEFSDjrdBJnX/vqKsLAnTdpSyu1Hha23kRZxNiocJ39ornTL0XFRLiMOFjK9dFhZI2+jU/pGwJfaM6iF2ccl1ckA4OTYQVPkz4GUrdxnPblD820pCiY1hIUJ+3CA5a5+QbkAkWnzHgisB43anpk33q0Olgv3gQYoIwDV0xECTBWlZhLPNg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EY+h1iSSfEvSMMRe2322ze9GyhtHctFLIpEJybP4aSOfNaSk7xbByS9DSD8/sYtPuDTEoiRr/P8rzVCvcNBx7rBqzV3lHoFR30jYpr66b5ZOy/zNXGH2Lg/S8Y2GyL6c+IfvpiF7lM1SbBdCxFivMXg7TSbiiNWLCOaxFYpin+3XocjVAQgmA8jbs8JR0TfkEYgcQi0Apz6jzSA06gnpmuGi9NjCefywj9oFYlExLB2YItaVTYIKC+qd+Arf1eTbUgqujlbrlEzOZQTg2leyGANrmH+PFQhiwqYCT6ep3VeC68NCJ8B8ZGr+awtmrDpMk+z/ywuAAZBG4ypUgBWeWA==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>, "iwj@xxxxxxxxxxxxxx" <iwj@xxxxxxxxxxxxxx>, "Volodymyr_Babchuk@xxxxxxxx" <Volodymyr_Babchuk@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>
  • Delivery-date: Thu, 11 Nov 2021 08:26:58 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHX1nAjHIlrrODS3kOS9MmML45m+av9/FFw
  • Thread-topic: [PATCH v3] xen/arm: don't assign domU static-mem to dom0 as reserved-memory

Hi Stefano

> -----Original Message-----
> From: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> Sent: Thursday, November 11, 2021 4:18 AM
> To: julien@xxxxxxx
> Cc: sstabellini@xxxxxxxxxx; Penny Zheng <Penny.Zheng@xxxxxxx>; Bertrand
> Marquis <Bertrand.Marquis@xxxxxxx>; Wei Chen <Wei.Chen@xxxxxxx>;
> iwj@xxxxxxxxxxxxxx; Volodymyr_Babchuk@xxxxxxxx; xen-
> devel@xxxxxxxxxxxxxxxxxxxx; Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>
> Subject: [PATCH v3] xen/arm: don't assign domU static-mem to dom0 as
> reserved-memory
> 
> From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>
> 
> DomUs static-mem ranges are added to the reserved_mem array for
> accounting, but they shouldn't be assigned to dom0 as the other regular
> reserved-memory ranges in device tree.
> 
> In make_memory_nodes, fix the error by skipping banks with xen_domain set
> to true in the reserved-memory array. Also make sure to use the first valid
> (!xen_domain) start address for the memory node name.
> 
> Fixes: 41c031ff437b ("xen/arm: introduce domain on Static Allocation")
> Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>
> Release-Acked-by: Ian Jackson <iwj@xxxxxxxxxxxxxx>

Thx again for the fixing up.

Reviewed-by: Penny Zheng <penny.zheng@xxxxxxx>
> ---
> Changes in v3:
> - move BUG_ON inside the loop
> 
> Changes in v2:
> - improve commit message
> - improve in-code comment
> - update nr_cells appropriately
> ---
>  xen/arch/arm/domain_build.c | 23 +++++++++++++++++------
>  1 file changed, 17 insertions(+), 6 deletions(-)
> 
> diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c
> index 9e92b640cd..dafbc13962 100644
> --- a/xen/arch/arm/domain_build.c
> +++ b/xen/arch/arm/domain_build.c
> @@ -862,21 +862,25 @@ static int __init make_memory_node(const struct
> domain *d,  {
>      int res, i;
>      int reg_size = addrcells + sizecells;
> -    int nr_cells = reg_size * mem->nr_banks;
> +    int nr_cells = 0;
>      /* Placeholder for memory@ + a 64-bit number + \0 */
>      char buf[24];
>      __be32 reg[NR_MEM_BANKS * 4 /* Worst case addrcells + sizecells */];
>      __be32 *cells;
> 
> -    BUG_ON(nr_cells >= ARRAY_SIZE(reg));
>      if ( mem->nr_banks == 0 )
>          return -ENOENT;
> 
> -    dt_dprintk("Create memory node (reg size %d, nr cells %d)\n",
> -               reg_size, nr_cells);
> +    /* find first memory range not bound to a Xen domain */
> +    for ( i = 0; i < mem->nr_banks && mem->bank[i].xen_domain; i++ )
> +        ;
> +    if ( i == mem->nr_banks )
> +        return 0;
> +
> +    dt_dprintk("Create memory node\n");
> 
>      /* ePAPR 3.4 */
> -    snprintf(buf, sizeof(buf), "memory@%"PRIx64, mem->bank[0].start);
> +    snprintf(buf, sizeof(buf), "memory@%"PRIx64, mem->bank[i].start);
>      res = fdt_begin_node(fdt, buf);
>      if ( res )
>          return res;
> @@ -886,17 +890,24 @@ static int __init make_memory_node(const struct
> domain *d,
>          return res;
> 
>      cells = &reg[0];
> -    for ( i = 0 ; i < mem->nr_banks; i++ )
> +    for ( ; i < mem->nr_banks; i++ )
>      {
>          u64 start = mem->bank[i].start;
>          u64 size = mem->bank[i].size;
> 
> +        if ( mem->bank[i].xen_domain )
> +            continue;
> +
>          dt_dprintk("  Bank %d: %#"PRIx64"->%#"PRIx64"\n",
>                     i, start, start + size);
> 
> +        nr_cells += reg_size;
> +        BUG_ON(nr_cells >= ARRAY_SIZE(reg));
>          dt_child_set_range(&cells, addrcells, sizecells, start, size);
>      }
> 
> +    dt_dprintk("(reg size %d, nr cells %d)\n", reg_size, nr_cells);
> +
>      res = fdt_property(fdt, "reg", reg, nr_cells * sizeof(*reg));
>      if ( res )
>          return res;
> --
> 2.25.1

--
Cheers,
Penny Zheng





 


Rackspace

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