On 30.09.19 08:20, Jeroen Roovers wrote:
On Mon, 9 Sep 2019 15:38:56 +0200
Helge Deller <deller@xxxxxx> wrote:
Sven noticed that calling ioremap() and iounmap() multiple times leads
to a vmap memory leak:
vmap allocation for size 4198400 failed:
use vmalloc=<size> to increase size
It seems we missed calling remove_vm_area() for ioummap().
That patch seems to work (tested on a C8000 for a couple of weeks).
Did you actually stress-tested it?
modprobe/rmmod is not used that often if you have the patch simply
compiled in...
I was surprised it wasn't part of the "parisc architecture updates for
kernel v5.4" pull request.
My hope was that some mm-knowledgeable person would doublecheck.
It seems we could e.g. use vunmap() instead of remove_vm_area() & kfree()...
The patch is not forgotten, it just needs some more thoughts/discussions.
Helge
