On Tue, Nov 26, 2019 at 01:20:51PM +0100, Johannes Thumshirn wrote: > The 'start' variable indicates the start of a filemap and is set to the > iocb's position, which we have already cached as 'pos', upon function > entry. > > 'pos' is used as a cursor indicating the current position and updated > later in iomap_dio_rw(), but not before the last use of 'start'. > > Remove 'start' as it's synonym for 'pos' before we're entering the loop > calling iomapp_apply(). > > Signed-off-by: Johannes Thumshirn <jthumshirn@xxxxxxx> Looks good, Reviewed-by: Christoph Hellwig <hch@xxxxxx>