Date: Thu, 28 Sep 2017 09:55:09 -0600 From: Ian Lepore <ian@freebsd.org> To: Warner Losh <imp@bsdimp.com>, Andriy Gapon <avg@freebsd.org> Cc: "freebsd-arch@freebsd.org" <freebsd-arch@freebsd.org> Subject: Re: lua in zfs Message-ID: <1506614109.31939.20.camel@freebsd.org> In-Reply-To: <CANCZdfp7=_SyD4SovOFuA1hq8cQHOy%2BGQ8EbtnDZ5LnFNQuE8Q@mail.gmail.com> References: <ae7a34af-4c91-daae-e917-b98ac7fb69d9@FreeBSD.org> <CANCZdfqocwEArqntzUdwBufxa%2BNANHs6VdKJrRFwGHT8HpaFyw@mail.gmail.com> <fd8098c7-37dd-e66a-e528-2668203381a5@FreeBSD.org> <CANCZdfp7=_SyD4SovOFuA1hq8cQHOy%2BGQ8EbtnDZ5LnFNQuE8Q@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 2017-09-28 at 09:40 -0600, Warner Losh wrote: > On Thu, Sep 28, 2017 at 9:16 AM, Andriy Gapon <avg@freebsd.org> > wrote: > > > > > On 28/09/2017 16:13, Warner Losh wrote: > > > > > > Which version? Where in the tree? Can this be disabled? > > It's lua 5.2.4 at the moment. Its source code will reside in > > sys/cddl/contrib/opensolaris/uts/common/fs/zfs/lua. At run-time it > > will > > be in > > zfs.ko or in kernel if it's built with ZFS option. For time being, > > it's > > not > > possible to turn it off individually, but if you don't include ZFS > > or any > > CDDL > > licensed code at all, then you won't have Lua too. But I think > > that it > > should > > be possible to add a new knob specifically to compile ZFS without > > Lua, if > > anybody wants it. > > OK. Any chance you could put it in sys/contrib/lua instead? The same > place > that the boot loader lua project has it? The lua boot loader project > is > using 5.3.0 it seems, is there any reason you couldn't use that > instead? > > I've been working on this in the background and was hoping to get it > in > before 12. > > Warner 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. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1506614109.31939.20.camel>