• Daniel Vetter's avatar
    /dev/mem: Only set filp->f_mapping · 0fb1b1ed
    Daniel Vetter authored
    When we care about pagecache maintenance, we need to make sure that
    both f_mapping and i_mapping point at the right mapping.
    
    But for iomem mappings we only care about the virtual/pte side of
    things, so f_mapping is enough. Also setting inode->i_mapping was
    confusing me as a driver maintainer, since in e.g. drivers/gpu we
    don't do that. Per Dan this seems to be copypasta from places which do
    care about pagecache consistency, but not needed. Hence remove it for
    slightly less confusion.
    Signed-off-by: default avatarDaniel Vetter <daniel.vetter@intel.com>
    Cc: Jason Gunthorpe <jgg@ziepe.ca>
    Cc: Kees Cook <keescook@chromium.org>
    Cc: Dan Williams <dan.j.williams@intel.com>
    Cc: Andrew Morton <akpm@linux-foundation.org>
    Cc: John Hubbard <jhubbard@nvidia.com>
    Cc: Jérôme Glisse <jglisse@redhat.com>
    Cc: Jan Kara <jack@suse.cz>
    Cc: Dan Williams <dan.j.williams@intel.com>
    Cc: linux-mm@kvack.org
    Cc: linux-arm-kernel@lists.infradead.org
    Cc: linux-samsung-soc@vger.kernel.org
    Cc: linux-media@vger.kernel.org
    Reviewed-by: default avatarDan Williams <dan.j.williams@intel.com>
    Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
    Link: https://patchwork.freedesktop.org/patch/msgid/20201127164131.2244124-10-daniel.vetter@ffwll.ch
    0fb1b1ed
mem.c 23.5 KB