lua in zfs

[Warner Losh]

On Thu, Sep 28, 2017 at 9:55 AM, Ian Lepore <ian at freebsd.org> wrote:

> On Thu, 2017-09-28 at 09:40 -0600, Warner Losh wrote:
> > On Thu, Sep 28, 2017 at 9:16 AM, Andriy Gapon <avg at 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.
>

I have a dim memory that this is why the boot loader project used it, but I
may be misremembering.

In any event, so people can follow along at home, there's a freshly rebased
version of the lua-bootloader branch in
https://github.com/bsdimp/freebsd.git I just pushed.

Warner