Re: [PATCH] auxdisplay/cfag12864bfb.c: Replace vm_insert_page

From: Matthew Wilcox
Date: Sat Sep 22 2018 - 08:29:23 EST


On Fri, Sep 21, 2018 at 01:53:16AM +0530, Souptick Joarder wrote:
> There is a plan to remove vm_insert_page permanently
> and replace it with new API vmf_insert_page which will
> return vm_fault_t type. As part of it vm_insert_page
> is removed from this driver.
>
> remap_pfn_range() will be used to map kernel memory to
> user vma.

I think this patch is dangerous. It's not buggy *for this driver*,
but it gives the impression that it's safe to use remap_pfn_range()
for this *kind* of driver.