On Wed, Nov 17, 2010 at 05:20:01PM -0700, Mark Shellenbaum wrote:
> Don't fix it that way.
>
> dn->dn_have_spill means you either have a spill or are about to get one
>
> This is more appropriate
>
> if (!(dn->dn_phys->dn_flags & DNODE_FLAG_SPILL_BLKPTR)) {
> ...
> } else {
> ...
> }I saw those two ways of checking this and was wondering what's the difference. Now I know, thank you, Mark:) -- Pawel Jakub Dawidek http://www.wheelsystems.com [email protected] http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am!
pgpP9dqa0Dmp4.pgp
Description: PGP signature
_______________________________________________ zfs-code mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/zfs-code
