On 10/9/19 12:37 PM, Christoph Hellwig wrote:
On Wed, Oct 09, 2019 at 11:32:54AM +0530, Ritesh Harjani wrote:
We can also get rid of "first_block" argument here.
That would just duplicate filling it out in all callers, so why?
What I meant is "first_block" is same as map->m_lblk.
(unless ext4_map_blocks can change map->m_lblk, which AFAICT, it should
not).
So why pass an extra argument when we are already passing 'map'
structure.
So we can fill iomap->offset using map->m_lblk in ext4_set_iomap()
function.
-ritesh
