>> >> I kept the style similar to null_free_sector() where root is calculated >> inside the helper acting on the sector, if we change that for >> null_zero_sector() then I think we need to change for null_free_sector() >> for consistency, unless you strongly object it for some reason. >> > > Nope, just a nit that stood out to me when skimming the patches. Not a > big deal at all. > It is a really good point, I'll send a cleanup on the top of this will CC you. -ck