Kernel module doesn't clean up resources if program crashes or doesn't release handles

[enfiskutensykkel]

No description provided.

[enfiskutensykkel]

Currently mappings are stored on a global linked list. We should instead keep track of open and close of the ioctl file descriptor and create a linked list per process. This would allow us to close all mappings for once process, and avoid checking the current process in lookup.

[patstew]

The other way to go about it is to use mmap to get the memory then the kernel does it for you. You still need an ioctl to get the physical address though,

[enfiskutensykkel]

Hmm, good point. I think the module may be up for a major overhaul, ideally it should support huge pages and allocate contiguous address ranges that it maps for the userspace program. It would simplify the logic in the library, eliminating the hard separation between the code dependent on the SISCI/SmartIO path and also make larger queues and buffers simpler to deal with by avoiding PRP lists. I will give this some thought.