Skip to content

draft: support for hexagon VM#88

Draft
Brian Cain (androm3da) wants to merge 3 commits into
hexagon_sysemu_20_dec_2023from
bcain/hvm_00
Draft

draft: support for hexagon VM#88
Brian Cain (androm3da) wants to merge 3 commits into
hexagon_sysemu_20_dec_2023from
bcain/hvm_00

Conversation

@androm3da

@androm3da Brian Cain (androm3da) commented Apr 25, 2024

Copy link
Copy Markdown
Member

No description provided.

androm3da
Brian Cain (androm3da) commented Apr 25, 2024
View reviewed changes
Comment thread docs/system/hexagon/hvm.rst
Comment thread target/hexagon/translate.c Outdated
@androm3da
Brian Cain (androm3da) force-pushed the bcain/hvm_00 branch 7 times, most recently from 3204331 to 38d15d5 Compare May 2, 2024 02:21
@androm3da

Brian Cain (androm3da) commented May 2, 2024

Copy link
Copy Markdown
Member Author

Some progress here..

When booting linux, we get through ~4600 instructions before we hit one of these panics in setup_command_line():

        saved_command_line = memblock_alloc(len + ilen, SMP_CACHE_BYTES);
        if (!saved_command_line)
                panic("%s: Failed to allocate %zu bytes\n", __func__, len + ilen);

        static_command_line = memblock_alloc(len, SMP_CACHE_BYTES);
        if (!static_command_line)
                panic("%s: Failed to allocate %zu bytes\n", __func__, len);

@androm3da

Brian Cain (androm3da) commented May 2, 2024

Copy link
Copy Markdown
Member Author

Some progress here..

When booting linux, we get through ~4600 instructions before we hit one of these panics in setup_command_line():

        saved_command_line = memblock_alloc(len + ilen, SMP_CACHE_BYTES);
        if (!saved_command_line)
                panic("%s: Failed to allocate %zu bytes\n", __func__, len + ilen);

        static_command_line = memblock_alloc(len, SMP_CACHE_BYTES);
        if (!static_command_line)
                panic("%s: Failed to allocate %zu bytes\n", __func__, len);

Now that I think of it, the unimplemented vmnewmap instruction is the most likely cause of the failing memblock_alloc() that leads to the panic

@androm3da
Brian Cain (androm3da) force-pushed the bcain/hvm_00 branch 2 times, most recently from aea8b67 to f6b3f52 Compare May 8, 2024 00:38
@androm3da
Brian Cain (androm3da) force-pushed the hexagon_sysemu_20_dec_2023 branch from cc9e4ca to 4531d97 Compare July 10, 2024 01:00
@androm3da
draft: support for Hexagon VM
704cd90 
Status: boots linux kernel up to setup_command_line() before panic

Hacks:
* vmnewmap call not implemented (does get invoked by linux)
* vmyield call not implemented (does get invoked by linux)
* trap0 not quite right (not _yet_ invoked by linux)
... and more

Signed-off-by: Brian Cain <bcain@quicinc.com>
@androm3da
draft: Implement vmnewmap inst
d83a7e3 
FIXME: while following the spec for vmnewmap and the kernel source, I thought
I understood the expected behavior but based on what I've seen so far I
think it's not yet implemented correctly.

Signed-off-by: Brian Cain <bcain@quicinc.com>
@androm3da
Implement a portion of vmgetinfo
1ed0a26 
Signed-off-by: Brian Cain <bcain@quicinc.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SidManning Sid Manning (SidManning) Awaiting requested review from SidManning
@quic-mathbern Matheus Tavares Bernardino @ Qualcomm (quic-mathbern) Awaiting requested review from quic-mathbern

Assignees

@androm3da Brian Cain (androm3da)

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@androm3da