Add missing commands

[SammyOina]

• New Features

  • Introduced support for multiple SEV modes.
  • Added command-line options for calculating OVMF hashes and managing SEV guest launch measurements.
  • New functions for calculating launch digests and managing SEV hash tables.

• Improvements

  • Enhanced memory management in the OVMF module with new fields and methods.
  • Improved error handling and user feedback in command-line interfaces.

• Bug Fixes

  • Adjusted function signatures for better initialization and configuration of objects.

• Documentation

  • Added string representations for VMM types to improve usability in logging and debugging.

[derpsteb]

Nice, thanks for the contribution! I will take a look once you request a review :).

[SammyOina]

Nice, thanks for the contribution! I will take a look once you request a review :).

@derpsteb this is ready for review

[derpsteb]

Sorry I was away from my laptop for a few weeks. Will pick this up on Monday.

[derpsteb]

hey, went over the bigger part, looks great so far! need to finish the rest tomorrow. only some small comments.

one thing missing is some e2e test for the added functionality. but I think this needs to be done in CI because we need to build ovmf and svsm binaries for that. maybe you can have a look at upstream_test.go - if you have a good idea how to build a similar test for some of the new features it would be much appreciated. (the ovmf.New call in upstream_test.go also needs to be fixed.)

one general thing: could you see that the contents of a file are ordered like this?

• Constants and variables
• Exported functions
• Exported types followed by their new funcs, exported methods, and unexported methods, i.e.
• Unexported functions
• Unexported types and their methods

I will mention this expectation in the readme for future contributions.

thanks again for your work.

[derpsteb]

Suggested change

	func BuildSaveAreaSVSM(eip uint32, sevFeatures uint64, vcpuSig uint64, vmmType vmmtypes.VMMType) (SevEsSaveArea, error) {
	func (VMSA_SVSM) BuildSaveArea(eip uint32, sevFeatures uint64, vcpuSig uint64, vmmType vmmtypes.VMMType) (SevEsSaveArea, error) {

the original BuildSaveArea should also be a method of VMSA. but I can do that in a separate PR.

[SammyOina]

I'll leave that for another PR

[derpsteb]

the original code is referencing a field fcw here. but I can't find it in SevEsSaveArea. Could you make sense of that?

[SammyOina]

I did not see where it was used outside the package but the value is saved on x87Fcw