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

Re: [PATCH v1 0/6] tools/ocaml/xenstored: simplify code


  • To: Edwin Torok <edvin.torok@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Christian Lindig <christian.lindig@xxxxxxxxxx>
  • Date: Tue, 18 Aug 2020 09:25:07 +0000
  • Accept-language: en-GB, en-US
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxx>, "dave@xxxxxxxxxx" <dave@xxxxxxxxxx>, "wl@xxxxxxx" <wl@xxxxxxx>
  • Delivery-date: Tue, 18 Aug 2020 09:25:14 +0000
  • Ironport-sdr: BYjaYox372jwQM1C2eyXdlCYXKmUhx5s6YX1pkuDEmzEiSNObLytBIGEV992rvx+wX0MtLKMcb Ul86xYgPnxe8/xkHFzWA6H1gWFwJwhM6tO4t5gRXxQs7iufU1HbzObz1y6FmZNIJMkb3iLOI5Y bXsMUtsVfvDx0siSaM2HqbU/xhNxSzlBPZ+acPT55nc8byYaNbD2etgIjtJwReikzFjt3+KGag cR/+tZ9jHISR6GGaQWJl80rmGenRIzOC+uHo6LRprXhoyjy14Bb3qJs9b5eOIapmRGVYYj4eWg ni8=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHWcof9CyuGFmlmrU+YP676eBh39Kk8RbglgAEV8ACAAEGz3g==
  • Thread-topic: [PATCH v1 0/6] tools/ocaml/xenstored: simplify code

I see little reason to support old OCaml releases and requiring OCaml 4.06 
would be fine with me but I assume that the project might have its own ideas 
about this.

________________________________________
From: Edwin Torok
Sent: 18 August 2020 08:28
To: Christian Lindig; xen-devel@xxxxxxxxxxxxxxxxxxxx
Cc: Ian Jackson; dave@xxxxxxxxxx; wl@xxxxxxx
Subject: Re: [PATCH v1 0/6] tools/ocaml/xenstored: simplify code

On Mon, 2020-08-17 at 14:56 +0200, Christian Lindig wrote:
> This all looks good - I left a small comment on one of the patches
> and I agree that this needs testing. I also wonder about
> compatibility with earlier OCaml releases that we support but I see
> no real obstacles.
>

I've developed the series using OCaml 4.08.1. I think the newest
feature I used was Map.update (OCaml 4.06, nearly 3 years ago).
Looking through https://repology.org/project/ocaml/versions I'm not
sure if we can require more than 4.05 though.
The README in Xen doesn't specify a minimum version, but configure
checks for >=4.02.

I can try to backport my series to OCaml 4.05 (to use Map.find_opt
instead of Map.update) and update the configure check to require 4.05.
It would be possible to backport even further to 4.02 by introducing
additional inefficiencies (Map.mem + Map.find would traverse the map
twice, and Map.find on its own would raise an exception on Not found,
which is more costly than returning None in Map.find_opt), I'd avoid
doing that.

Xen's CI from automation might need some updates to use latest stable
versions:
* Fedora 29 is EOL, should use at least Fedora 31
* Debian Jessie is EOL. Stretch is present, but Buster is missing

Best regards,
--Edwin



 


Rackspace

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