Re: [dm-devel] [PATCH 02/45] filemap: consistently use ->f_mapping over ->i_mapping

2020-11-30 Thread Johannes Thumshirn
Looks good, Reviewed-by: Johannes Thumshirn -- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel

[dm-devel] [PATCH 02/45] filemap: consistently use ->f_mapping over ->i_mapping

2020-11-28 Thread Christoph Hellwig
Use file->f_mapping in all remaining places that have a struct file available to properly handle the case where inode->i_mapping != file_inode(file)->i_mapping. Signed-off-by: Christoph Hellwig Reviewed-by: Jan Kara Reviewed-by: Hannes Reinecke Reviewed-by: Matthew Wilcox (Oracle) --- mm/file

Re: [dm-devel] [PATCH 02/45] filemap: consistently use ->f_mapping over ->i_mapping

2020-11-25 Thread Jan Kara
On Tue 24-11-20 14:27:08, Christoph Hellwig wrote: > Use file->f_mapping in all remaining places that have a struct file > available to properly handle the case where inode->i_mapping != > file_inode(file)->i_mapping. > > Signed-off-by: Christoph Hellwig Looks good to me. You can add: Reviewed-

Re: [dm-devel] [PATCH 02/45] filemap: consistently use ->f_mapping over ->i_mapping

2020-11-24 Thread Hannes Reinecke
On 11/24/20 2:27 PM, Christoph Hellwig wrote: Use file->f_mapping in all remaining places that have a struct file available to properly handle the case where inode->i_mapping != file_inode(file)->i_mapping. Signed-off-by: Christoph Hellwig --- mm/filemap.c | 13 ++--- 1 file changed,

Re: [dm-devel] [PATCH 02/45] filemap: consistently use ->f_mapping over ->i_mapping

2020-11-24 Thread Matthew Wilcox
On Tue, Nov 24, 2020 at 02:27:08PM +0100, Christoph Hellwig wrote: > Use file->f_mapping in all remaining places that have a struct file > available to properly handle the case where inode->i_mapping != > file_inode(file)->i_mapping. > > Signed-off-by: Christoph Hellwig Reviewed-by: Matthew Wilc

[dm-devel] [PATCH 02/45] filemap: consistently use ->f_mapping over ->i_mapping

2020-11-24 Thread Christoph Hellwig
Use file->f_mapping in all remaining places that have a struct file available to properly handle the case where inode->i_mapping != file_inode(file)->i_mapping. Signed-off-by: Christoph Hellwig --- mm/filemap.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/mm/