Gustavo Moreira

It sounds good. I saw other tickets where you have been working in cache improvements. I also noticed that, and that was the reason I needed to wipe the cache...

That makes sense. As `__dentry_path` wasn't returning the `dentry path` but the `dentry name`. Instead of building the path for then split it, it would be better to simply do:...

All done. Also, fixed some minor stuff and changed the warning log lines to LOGLEVEL_V. Otherwise, those lines will be always shown in the output and that's not the intention.

Please check the changes in [9b0b254](https://github.com/volatilityfoundation/volatility3/pull/597/commits/9b0b2547d5a69b5ba78416fa3c858a41489e4dde). If an InvalidAddressException is raised reading the addresses, it returns a "?" value and a debug message is printed, but the plugin continues. Please...

Also, in [5f70c70](https://github.com/volatilityfoundation/volatility3/pull/597/commits/5f70c7031c107645be93a71d392177c123c60307) I added more improvements in the way the socket family states are managed.

Thanks for the review guys. I will finish and submit a new batch with the requested changes probably next week.

Thanks Andrew, no worries. It will be really nice to see this code integrated. I need to take a look at Volatility 3, not sure how much effort it means...

@ikelos @digitalisx Cool, here is the last batch of fixes and improvements. Following your suggestions, I disaggregated pid, fd, address and port, and socket memory address to dedicated new columns....

Hey @ikelos/@atcuno this is still awaiting a review :pray:

> I don't know enough about the structures to know the difference between a `tgid` and a `pid`. I'm happy to defer to @atcuno decision on this front? Hey! sorry...