On Tue, Oct 15, 2019 at 02:11:02PM +0200, David Sterba wrote: > On Mon, Oct 07, 2019 at 06:24:19PM -0400, Dennis Zhou wrote: > > > > + * Bitmap region iterators. Iterates over the bitmap between [@start, @end). > > > > > > Gonna be that guy here, should be '[@start, @end]' > > > > I disagree here. I'm pretty happy with [@start, @end). If btrfs wants to > > carry their own iterators I'm happy to copy and paste them, but as far > > as percpu goes I like [@start, @end). > > It's not clear what the comment was about, if it's the notation of > half-closed interval or request to support closed interval in the > lookup. The orignal code has [,) and that shouldn't be changed when > copying. Or I'm missing something. I think there was just confusion based on the notation where '[' means inclusive and ')' means exclusive. That got cleared up. Thanks, Dennis
