On 6/11/14, 3:38 PM, Joerg Goltermann wrote: > > > On 11.06.2014 14:56, Daniil Lunev via illumos-developer wrote: >> Updated patch to the LZ4 metadata compression >> http://cr.illumos.org/~webrev/DKOIman/LZ4_MD/ >> ZDB output is attached. Not so big gain on compression rate, though, it >> is better. Also, it was shown in initial lz4 commit that lz4 is faster. >> That's why changing metadata compression to the lz4 aglorithm is a >> reasonable idea. > > I think it's better to cache the result of > "spa_feature_is_active(os->os_spa, SPA_FEATURE_LZ4_COMPRESS);" > and use this in dmu_write_policy()
Not necessary, feature_get_refcount already caches the value. Cheers, -- Saso _______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
