On Mon, Jan 15, 2018 at 10:16:54AM -0700, Edmund Nadolski wrote: > > > On 01/15/2018 05:31 AM, Nikolay Borisov wrote: > > delayed_iput_count wa supposed to be used to implement, well, delayed > > iput. The idea is that we keep accumulating the number of iputs we do > > until eventually the inode is deleted. Turns out we never really > > switched the delayed_iput_count from 0 to 1, hence all conditional > > code relying on the value of that member being different than 0 was > > never executed. This, as it turns out, didn't cause any problem due > > to the simple fact that the generic inode's i_count member was always > > used to count the number of iputs. So let's just remove the unused > > member and all unused code. This patch essentially provides no > > functional changes. > > > > Signed-off-by: Nikolay Borisov <nbori...@suse.com> > > Since the 8089fe62c6 changelog mentions the need for a count, it might > be nice to include a brief code comment about the i_count effect.
Agreed. > Reviewed-by: Edmund Nadolski <enadol...@suse.com> Reviewed-by: David Sterba <dste...@suse.com> -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html