Adding tests to snphostok

[arvindskumar99]

Added the following tests to snphostok

SME enabled | Checks that SMEE features are enabled on host | Checks MSR 0xC0010010 bit 23 for enablement

SEV INIT STATE | Checks to see if SEV is init | Checks SEV_PLATFORM_STATUS IOCTL

SEV-ES INIT STATE | Check to see if SEV-ES is init | Check SEV_PLATFORM_STATUS IOCTL

SNP enabled in MSR | Check to see if SNP is enabled | Check MSR 0xC0010010 bit 23

System SEV firmware version | Check for SEV FW version | Use SEV_PLATFORM_STATUS IOCTL

SNP INIT | Check if SNP is init | Check SNP_PLATFORM_STATUS IOCTL

RMP INIT | Check if RMP is init | Check SNP_PLATFORM STATUS IOCTL

RMP table addresses | Get the ranges for RMP TABLE ADDRESS | Check MSR 0xC0010132 - 0xC0010133

Compare TCB versions | Make sure current TCB and reported TCB match | Check SNP_PLATFORM_STATUS IOCTLS

[larrydewey]

Just one thought. I don't know that it is a stopper for me, but is worth the discussion.

[larrydewey]

Also, please make sure to rebase your branch :)

[DGonzalezVillal]

Great first PR!

I requested some changes regarding test readability and coding practices.

I also requested an enhancement on one of the tests that I forgot to mention earlier.

[tylerfanelli]

For something such as:

SME enabled: enabled

Are you referring to enabled in the kernel? The "parent" tests for CPU availability.

[arvindskumar99]

For something such as:

SME enabled: enabled

Are you referring to enabled in the kernel? The "parent" tests for CPU availability.

The SME test is checking the MSR for cpu enablement. Do you want me to make that more clear in the result prompt?

[tylerfanelli]

Just trying to understand the difference. So, although it can be supported on a certain CPU, it still must be enabled in the MSR (and all of this is completely separate from having kernel enablement of the same feature?)

Anyway, yes I think SME enabled by MSR would be a clearer message as to what this test does. Instead of printing enabled, you can just have the [ PASS ] show the enablement. Something like:

[ PASS ]   - Secure Memory Encryption (SME)
[ PASS ]       - SME enabled by MSR

[larrydewey]

@arvindskumar99 Looks like you still have some clippy and formatting issues. Please run:

cargo clippy -- -D clippy::all

and

cargo fmt

[larrydewey]

LGTM

[arvindskumar99]

After changes

[DGonzalezVillal]

lgtm