From owner-freebsd-arch@freebsd.org Thu Sep 28 18:44:08 2017 Return-Path: Delivered-To: freebsd-arch@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id BC204E05DD8 for ; Thu, 28 Sep 2017 18:44:08 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from mail.baldwin.cx (bigwig.baldwin.cx [96.47.65.170]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 97B5B694CC; Thu, 28 Sep 2017 18:44:08 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from ralph.baldwin.cx (c-73-231-226-104.hsd1.ca.comcast.net [73.231.226.104]) by mail.baldwin.cx (Postfix) with ESMTPSA id E72AD10AF07; Thu, 28 Sep 2017 14:44:01 -0400 (EDT) From: John Baldwin To: freebsd-arch@freebsd.org Cc: Warner Losh , Andriy Gapon , Ian Lepore Subject: Re: lua in zfs Date: Thu, 28 Sep 2017 10:52:44 -0700 Message-ID: <13426777.FL7aYimd4u@ralph.baldwin.cx> User-Agent: KMail/4.14.10 (FreeBSD/11.1-STABLE; KDE/4.14.30; amd64; ; ) In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.4.3 (mail.baldwin.cx); Thu, 28 Sep 2017 14:44:02 -0400 (EDT) X-Virus-Scanned: clamav-milter 0.99.2 at mail.baldwin.cx X-Virus-Status: Clean X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Sep 2017 18:44:08 -0000 On Thursday, September 28, 2017 11:07:35 AM Warner Losh wrote: > On Thu, Sep 28, 2017 at 10:50 AM, Andriy Gapon wrote: > > > On 28/09/2017 19:44, Andriy Gapon wrote: > > > On 28/09/2017 18:55, Ian Lepore wrote: > > >> Iirc, the big difference between 5.2.x and 5.3 is that the latter added > > >> support for integers. It seems like that would be a good thing, in the > > >> kernel. > > > > > > I am sure that the ZFS Lua represents numbers as integers and has no > > floating > > > point support at all. > > > > > > Some more info: > > https://github.com/illumos/illumos-gate/blob/master/usr/ > > src/uts/common/fs/zfs/lua/README.zfs > > > Reading this strongly suggests to me that we're going to have two versions > of lua in the tree then: > > 1) This version is a modified version of 5.2.4 > 2) Significant changes have been made to harden this interpreter, which > won't be in 5.3.0 > 3) A number of APIs have been removed, which makes it difficult to use as a > general purpose thing. > > So I think that you should go ahead and import it. The level of work for my > request is very high, and significantly diverges us from upstream. It's > unfortunate that we'll have two versions. But this isn't really pure lua > but a variant maybe better described as lua-zfs. Yes, that was my guess from Andriy's original e-mail. ZFS tends to be a bit self-contained and includes its own version of zlib for example. In general it seems to use its own version of 3rd party utils rather than sharing those with other parts of the system. -- John Baldwin