On 5/31/22 12:04 AM, Christoph Hellwig wrote: > On Thu, May 26, 2022 at 10:38:38AM -0700, Stefan Roesch wrote: >> This changes the function signature of the function xfs_ilock_iocb(): >> - the parameter iocb is replaced with ip, passing in an xfs_inode >> - the parameter iocb_flags is added to be able to pass in the iocb flags >> >> This allows to call the function from xfs_file_buffered_writes. > > xfs_file_buffered_write? But even that already has the iocb, so I'm > not sure why we need that change to start with. Yes xfs_file_buffered_write(). The problem is that xfs_iolock_iocb uses: iocb->ki_filp->f_inode, but xfs_file_buffered_write: iocb->ki_ki_filp->f_mapping->host This requires to pass in the xfs_inode *.