[Top] [All Lists]

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

Re: [Xen-devel] [PATCH v7 2/8] libxl: introduce a new structure to represent static shared memory regions

To: Stefano Stabellini <sstabellini@xxxxxxxxxx>
From: Julien Grall <julien.grall@xxxxxxx>
Date: Wed, 12 Sep 2018 11:29:32 +0100
Cc: ian.jackson@xxxxxxxxxxxxx, Stefano Stabellini <stefanos@xxxxxxxxxx>, wei.liu2@xxxxxxxxxx, blackskygg@xxxxxxxxx, xen-devel@xxxxxxxxxxxxx
Delivery-date: Wed, 12 Sep 2018 10:29:38 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>



On 09/12/2018 12:09 AM, Stefano Stabellini wrote:

+
+/* Constants for libxl_static_shm */
+#define LIBXL_SSHM_RANGE_UNKNOWN UINT64_MAX
+#define LIBXL_SSHM_ID_MAXLEN    128


Why do you need to bound the size of the string?


It is passed by the user, it is good to be clear about the max size?

I wasn't able to find other string in libxl to be bound. So I am notsure why this one would be. Maybe Wei knows?


Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

References:
- Re: [Xen-devel] [PATCH v7 2/8] libxl: introduce a new structure to represent static shared memory regions
  - From: Stefano Stabellini

Prev by Date: Re: [Xen-devel] [PATCH] drivers/block/xen-blkback/common.h: use DIV_ROUND_UP instead of reimplementing its function
Next by Date: [Xen-devel] [linux-linus test] 127516: regressions - FAIL
Previous by thread: Re: [Xen-devel] [PATCH v7 2/8] libxl: introduce a new structure to represent static shared memory regions
Next by thread: [Xen-devel] [linux-linus test] 127497: regressions - FAIL
Index(es):
- Date
- Thread

Rackspace

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