Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 17 Aug 2017 10:07 -0700
From:      John Baldwin <jhb@freebsd.org>
To:        Andriy Gapon <avg@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r322601 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys
Message-ID:  <1870652.QF6rqaG5fn@ralph.baldwin.cx>
In-Reply-To: <4786e0df-cbbd-d0fe-2e45-682b9fe24652@FreeBSD.org>
References:  <201708162340.v7GNeWrL098164@repo.freebsd.org> <4786e0df-cbbd-d0fe-2e45-682b9fe24652@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thursday, August 17, 2017 10:42:51 AM Andriy Gapon wrote:
> On 17/08/2017 02:40, John Baldwin wrote:
> > Author: jhb
> > Date: Wed Aug 16 23:40:32 2017
> > New Revision: 322601
> > URL: https://svnweb.freebsd.org/changeset/base/322601
> > 
> > Log:
> >   Mark ZFS ABD inline functions static.
> >   
> >   When built with -fno-inline-functions zfs.ko contains undefined references
> >   to these functions if they are only marked inline.
> >   
> >   Reviewed by:	avg (earlier version)
> 
> I think that you rushed a little bit on this one:
> https://github.com/openzfs/openzfs/pull/444
> 
> But no big deal, it will be easy to merge once the upstream wakes up :-)

Oof, I thought you had previously ok'd to commit this?  (I just hadn't gotten
around to it.)  My apologies for misunderstanding.

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1870652.QF6rqaG5fn>