[OpenZFS Developer] a ZFS SA bug and my patch

Ned Bass bass6 at llnl.gov
Wed Dec 11 01:17:08 UTC 2013


On Fri, Nov 22, 2013 at 10:07:06AM -0800, Ned Bass wrote:
> Hi James,
> 
> On Fri, Nov 22, 2013 at 07:27:19AM -0800, James Pan wrote:
> > Hi Ned,
> > Thanks very much for your comments.
> > Actually 'done' is not necessary and can be completely removed.
> > The value of *will_spill is initialized only when buftype is SA_BONUS, so to
> > remove buftype == SA_BONUS from the check, we need to set it to False for
> > SA_SPILL buftype as well.
> > 
> > I've revised the patch to reflect these changes, could you help review it again
> > and get it checked in to the main branch if it is OK?
> > Thanks a lot.
> 
> It looks good to me, except for a couple of formatting issues noted
> below.  We'll get it tested in the Linux port and post results.

This patch checked out well in testing, and it's been merged in the Linux
port:

https://github.com/zfsonlinux/zfs/commit/472e7c6

I recommend that it be considered for inclusion upstream.

Ned


More information about the zfs-devel mailing list