Nothing is wrong, the CPU you have is based on the bulldozer module architecture (steamroller) and has 2 modules. Each module contains two separate integer cores which share an FPU (although the FPU is in fact two FMAs), front and back ends.
To avoid overtaxing the FPU windows treats these CPUs as hyperthreaded dual cores (4 logical threads) so it will send 4 integer threads to be processed but only two floating point.
This is the best way to use bulldozer based CPUs and is normal, for example the piledriver FX8350 shows up in windows as 4 cores 8 logical and the FX6300 shows up as 3 cores 6 logical
