Create td-shim-interface crate for publishig

[OuyangHang33]

Fix: #654

[jyao1]

@NK-fengsf, @studychao, would you please take a look?

[NK-fengsf]

@OuyangHang33 , would you mind move TdShimLoader from td_shim_tools crate to td-shim-interface crate?

[OuyangHang33]

@NK-fengsf I'll check the feasibility today, and will let you know if move the td-shim-loader crate.

[OuyangHang33]

Hi @NK-fengsf ,
Because TdShimLoader defined the specific file operation and it's the consumer of metadata, it's not suitable to move loader.rs fromtd-shim-tool to the object of consumption td-shim-interface.

Could you give the reason why you need TdShimLoader, and how do you want to use it.

[NK-fengsf]

Hi @NK-fengsf , Because TdShimLoader defined the specific file operation and it's the consumer of metadata, it's not suitable to move loader.rs fromtd-shim-tool to the object of consumption td-shim-interface.

Could you give the reason why you need TdShimLoader, and how do you want to use it.

Dragonball VMM parse tdshim image and get Vec using TdShimLoader::parse(filename:&String)。
impl TdShimLoader {
pub fn parse(filename: &String) -> Option<(TdxMetadataDescriptor, Vec)> {}
}

I can make a copy of loader.rs into Dragonball sandbox , if it is not suitable to move loader.rs fromtd-shim-tool to td-shim-interface.

[jyao1]

Dragonball VMM parse tdshim image and get Vec using TdShimLoader::parse(filename:&String)。
impl TdShimLoader {
pub fn parse(filename: &String) -> Option<(TdxMetadataDescriptor, Vec)> {}
}

If my understanding is correct, you just need to have a parse() function.
Can we change the API to accept a binary string and output the metadata descriptor?

The reason is that we are NOT able to handle file operation in non-std env.

[NK-fengsf]

g and output the me

yes, I just need the parse function. It's okay for me to change the API to accept a binary string and output the metadata descriptor.

[OuyangHang33]

Hi @jyao1 @NK-fengsf , I create the PR to change the API and split the read_file operation: #660.
If the new parse function has no problems, I can move it to the td-shim-interface.

[NK-fengsf]

@OuyangHang33 , would you mind move TdShimLoader from td_shim_tools crate to td-shim-interface crate?

Hi @OuyangHang33 , I have no other needs apart from this. Thank you very much.

[OuyangHang33]

Hi @NK-fengsf , I have moved TdShimLoader from td_shim_tools crate to td-shim-interface crate.

[OuyangHang33]

Hi @NK-fengsf , could you test the new crate? if it can meet your requirement, we will publish it after merged.
Thank you!

[NK-fengsf]

Hi @NK-fengsf , could you test the new crate? if it can meet your requirement, we will publish it after merged. Thank you!

@OuyangHang33 Okay， I will finish the test before tomorrow evening。

[NK-fengsf]

Hi @NK-fengsf , could you test the new crate? if it can meet your requirement, we will publish it after merged. Thank you!

I have tested the td-shim-interface crate， it works well in dragonball-sandbox. @OuyangHang33 , CC @jyao1