On Tue, Jun 05, 2018 at 11:16:16AM +0300, Dan Carpenter wrote: > The return statement is indented too far. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Thanks! Acked-by: Naoya Horiguchi <n-horiguchi@xxxxxxxxxxxxx> > > diff --git a/mm/memory-failure.c b/mm/memory-failure.c > index de0bc897d6e7..72cde4b0939e 100644 > --- a/mm/memory-failure.c > +++ b/mm/memory-failure.c > @@ -1147,7 +1147,7 @@ static unsigned long dax_mapping_size(struct address_space *mapping, > if (page->mapping != mapping) { > xa_unlock_irq(&mapping->i_pages); > i_mmap_unlock_read(mapping); > - return 0; > + return 0; > } > vma_interval_tree_foreach(vma, &mapping->i_mmap, pgoff, pgoff) { > unsigned long address = vma_address(page, vma); >