On Mon, May 19, 2025 at 04:28:34PM +0530, Ekansh Gupta wrote: > > > On 5/19/2025 4:22 PM, Dmitry Baryshkov wrote: > > On Tue, May 13, 2025 at 09:58:25AM +0530, Ekansh Gupta wrote: > >> User request for remote heap allocation is supported using ioctl > >> interface but support for unmap is missing. This could result in > >> memory leak issues. Add unmap user request support for remote heap. > > Can this memory be in use by the remote proc? > Remote heap allocation request is only intended for audioPD. Other PDs > running on DSP are not intended to use this request.
'Intended'. That's fine. I asked a different question: _can_ it be in use? What happens if userspace by mistake tries to unmap memory too early? Or if it happens intentionally, at some specific time during work. > > > >> Fixes: 532ad70c6d449 ("misc: fastrpc: Add mmap request assigning for > >> static PD pool") > >> Cc: sta...@kernel.org > >> Signed-off-by: Ekansh Gupta <ekansh.gu...@oss.qualcomm.com> > >> --- > >> drivers/misc/fastrpc.c | 62 ++++++++++++++++++++++++++++++++++-------- > >> 1 file changed, 51 insertions(+), 11 deletions(-) > >> > -- With best wishes Dmitry