`getitab` alloc memory for `itab` by `persistentalloc`, and that couldnt be free about goloader HOT 6 CLOSED

https://github.com/golang/go/blob/3e7ec131667c31448365f47643cf9b58d08ffd26/src/runtime/heapdump.go#L118

another point worth paying attention to, runtime/debug.WriteHeapDump will store *_type to typecache

from goloader.

This memory cannot be released.
remove itab from itabtable to avoid invalid address errors if it is not allocated to the same block of memory during load the same obj file again.

from goloader.

Also it's worth noting that this code path is only hit for relatively infrequent dynamic interface assignments (not static ones, where an itab would have been emitted by the compiler and so would be in mmapped memory instead)

from goloader.

https://github.com/golang/go/blob/3e7ec131667c31448365f47643cf9b58d08ffd26/src/runtime/heapdump.go#L118

another point worth paying attention to, runtime/debug.WriteHeapDump will store *_type to typecache

@fumeboy
This seems to be only used during call heapdump.
*_type is stored in typecache, but it only use pointer for comparison. If the type does not exist, adding it.
it seems does not cause problem, but it retains useless data.
I tested dump heap when loaded the module, and dump heap after unload module, the process exits as normal

from goloader.

Yeah, types are only dumped which are currently in the itabtable, so old types that have been cached are never indirected (they should eventually get bumped off the bucket of 4 if enough hash collisions happen).

from goloader.

https://github.com/golang/go/blob/a031f4ef83edc132d5f49382bfef491161de2476/src/reflect/type.go#L1558

reflect's lookupCache will also keep pointer to _type, and it is safe to delete.

from goloader.