Skip to content

Comments

fix(core/vm): fix info on HV init#1

Closed
macpijan wants to merge 2 commits intocrosscon:nextfrom
3mdeb:crosscon_integration_next
Closed

fix(core/vm): fix info on HV init#1
macpijan wants to merge 2 commits intocrosscon:nextfrom
3mdeb:crosscon_integration_next

Conversation

@macpijan
Copy link

VA is used twice instead of size property.

danielRep and others added 2 commits May 12, 2025 05:58
Signed-off-by: Daniel Oliveira <drawnpoetry@gmail.com>
VA is used twice instead of size property.

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
@macpijan
Copy link
Author

@DavidMCerdeira It seems to me that you have lost this commit bec5e40 in next branch while force-pushing next branch 50 mins ago. It was fixing this issue: crosscon/CROSSCON-Hypervisor-and-TEE-Isolation-Demos#22

I am no expert here, but it seems some parts of this commit is missing. Can you please clarify whether this fix is applied in other manner?

BTW, I am not sure how good idea it is to force-push next branch if it is being in use already by others, unless absolutely necessary.

@DavidMCerdeira
Copy link

You are right. I was not careful enough.

Here is the restored commit for the fix: d211573

@macpijan
Copy link
Author

Thanks. I am also opening another MR in favor of this one with my fix in the log, if you can take a look: #2

@macpijan macpijan closed this May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants