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.