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

Re: [RFC v2 0/7] add function support to IDL

  • To: Nick Rosbrook <rosbrookn@xxxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Tue, 4 May 2021 16:46:52 +0100
  • Authentication-results: esa4.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, <george.dunlap@xxxxxxxxxx>, "Nick Rosbrook" <rosbrookn@xxxxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Tue, 04 May 2021 15:47:03 +0000
  • Ironport-hdrordr: A9a23:qC5JJa171fIOgaHmDVxmswqjBJskLtp033Aq2lEZdDV+dMuEm8 ey2MkKzBOcslcscVwphNzoAsK9aFzG85od2+MsFJODeCWjh2eyNoFl6uLZrQHIPyHl7OZS2e NBXsFFZOHYNlRxgcbk7ATQKb9J/PC87Kuqie3Cpk0DcShWbchbgjtENg==
  • Ironport-sdr: 4y+ckJVHAPYofB4ReZke1LKLqakm3hKYjN7R5M3/QPb1OdMSh0uMBFTxcS2oaMNJo3t7a0U1F7 R7b+7c6a6Q5XiKEAhG8JEJ6r4KhthBcg9xV4lipLCzAA3WqdxZcn9wWwgLOyDDrXgQeePzsIdA hhitXHqZHBQkUxBHzA/VNgOTfsUhWcskFyMFM/6Av7rD47DqdldkmMzQG8wh/AwzCvoIjned2P iFI4jnduAdzK6MmV5PA9Ij+Z5a5Tel5S2N6VRaPsTwNTiuhcZrFE14oRHYaeeI3FT3EsRmj+MB /Es=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Tue, Mar 02, 2021 at 08:46:12PM -0500, Nick Rosbrook wrote:
> At a Xen Summit design session for the golang bindings (see [1]), we
> agreed that it would be beneficial to expand the libxl IDL with function
> support. In addition to benefiting libxl itself, this would allow other
> language bindings to easily generate function wrappers.
> The first version of this RFC is quite old [1]. I did address comments
> on the original RFC, but also expanded the scope a bit. As a way to
> evaluate function support, I worked on using this addition to the IDL to
> generate device add/remove/destroy functions, and removing the
> corresponding macros in libxl_internal.h. However, I stopped short of
> actually completing a build with this in place, as I thought it made
> sense to get feedback on the idea before working on the next step.

The series looks good to me, beside a few detail.


Anthony PERARD



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