Date: Thu, 28 Sep 2017 09:59:27 -0600 From: Warner Losh <imp@bsdimp.com> To: Ian Lepore <ian@freebsd.org> Cc: Andriy Gapon <avg@freebsd.org>, "freebsd-arch@freebsd.org" <freebsd-arch@freebsd.org> Subject: Re: lua in zfs Message-ID: <CANCZdfog71wGH5rRTJBZ5LHAcRjXOx2gSJXdMXdgRWhJ1yahPA@mail.gmail.com> In-Reply-To: <1506614109.31939.20.camel@freebsd.org> 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> <1506614109.31939.20.camel@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Sep 28, 2017 at 9:55 AM, Ian Lepore <ian@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@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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfog71wGH5rRTJBZ5LHAcRjXOx2gSJXdMXdgRWhJ1yahPA>