It looks like you have the right components in the wrong places. Try one of these.
Implementation with a P-channel FET:
simulate this circuit – Schematic created using CircuitLab
Keeping the P-channel FET, a logic low input will result in the FET conducting, current through R1, and a logic high at the output.
Implementation with an N-channel FET:
simulate this circuit
Replacing with an N-channel FET, a logic low input will result in the FET not conducting, no current through R1, and a logic high at the output.
Both schematics assume your supply voltage, Vcc, is equivalent to your input voltage. If all you have is a P-channel FET, go with the first schematic. However, N-channel FETs are usually cheaper than a P-channel with similar capability. It also depends when you want a voltage across R1 as well.