On Mon, Oct 21, 2019 at 03:37:15PM +0200, Jan Kara wrote:
> On Mon 21-10-19 20:18:09, Matthew Bobrowski wrote:
> One nit below.
> > + ext4_es_find_extent_range(inode, &ext4_es_is_delayed,
> > + map->m_lblk, end, &es);
> > +
> > + if (!es.es_len || es.es_lblk > end)
> > + return false;
> > +
> > + if (es.es_lblk > map->m_lblk) {
> > + map->m_len = es.es_lblk - map->m_lblk;
> > + return false;
> > + }
> > +
> > + if (es.es_lblk <= map->m_lblk)
>
> This condition must be always true AFAICT.
That would make sense. I will remove this condition in v6.
> > + offset = map->m_lblk - es.es_lblk;
> > +
> > + map->m_lblk = es.es_lblk + offset;
> > + map->m_len = es.es_len - offset;
> > +
> > + return true;
> > +}
--<M>--
