New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

migtd: copy GHCI data to stack memory instead of heap #159

Closed

gaojiaqi7 wants to merge 1 commit into intel:main from gaojiaqi7:0220/use_stack_instead_of_heap

Contributor

gaojiaqi7 commented Feb 22, 2024

Closes #87


          migtd: copy GHCI data to stack memory instead of heap

22a60c2

Signed-off-by: Jiaqi Gao <[email protected]>

gaojiaqi7 force-pushed the 0220/use_stack_instead_of_heap branch from 8162892 to 22a60c2 Compare

February 23, 2024 01:27

jyao1 reviewed

View reviewed changes

src/migtd/src/migration/session.rs

@@ @@ -89,7 +90,8 @@ impl MigrationSession { @@
                       #[cfg(not(feature = "vmcall-interrupt"))]
                       tdx::tdvmcall_service(cmd_mem.as_bytes(), rsp_mem.as_mut_bytes(), 0, 0)?;
-                      let private_mem = Self::copy_from_shared_memory(rsp_mem.as_bytes());
+                      let mut private_mem = [0u8; GHCI_PAGE_SIZE];

Contributor

jyao1 Feb 26, 2024

you can use a global data section instead of stack.

gaojiaqi7 closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet