Yep. above all correct.
*as long as TASK MANAGER for Performance->CPU (with show logical cores selected) shows EIGHT GRAPHS you are fine.
Just FYI:
FX-8350 - this is four "MODULES" which as said have two "cores" per module. Not quite two separate cores so sharing causes issues with performance (the logic being less transistors and cheaper overall but code didn't really utilize them well).
i7-4790K - this has four cores, but is hyperthreaded. In this case a 2nd thread of code sits in a buffer to start being processed during the wait times the 1st thread of code isn't running (waiting for new data)… so it's still just one physical core that toggles between two threads of code
Both of these show up as "eight threads" in Task Manager...