Skip to content
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.

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

Configurable Machine for MIPS, i386, X86_64 #5

Merged
merged 4 commits into from
Feb 3, 2021
Merged

Configurable Machine for MIPS, i386, X86_64 #5

merged 4 commits into from
Feb 3, 2021

Conversation

AndrewFasano
Copy link
Contributor

Port of configurable-machine features from PANDA, updated to hopefully work with QEMU 5.1.

Note that although this PR contains a configurable mcahine for the i386 and x86_64-softmmu targets, those don't currently build in avatar-qemu. It's probably a simple missing #ifdef target_xxx guard, but I'm not too familiar with the codebase:

  LINK    i386-softmmu/qemu-system-i386
hw/avatar/interrupts.o: In function `avatar_armv7m_nvic_forward_write':
/home/fasano/git/avatar-qemu/hw/avatar/interrupts.c:58: undefined reference to `get_current_pc'
/home/fasano/git/avatar-qemu/hw/avatar/interrupts.c:63: undefined reference to `rmem_tx_queue_ref'
/home/fasano/git/avatar-qemu/hw/avatar/interrupts.c:63: undefined reference to `qemu_avatar_mq_send'
/home/fasano/git/avatar-qemu/hw/avatar/interrupts.c:64: undefined reference to `rmem_rx_queue_ref'
/home/fasano/git/avatar-qemu/hw/avatar/interrupts.c:64: undefined reference to `qemu_avatar_mq_receive'

Someone who knows how to use avatar-qemu should probably test the MIPS configurable machine before merging, I only made sure it compiled. Looks like a bunch of the QEMU APIs related to CPU creation changed (here?).

@AndrewFasano AndrewFasano changed the title Revert "Missing MIPS functions: avatartwo/avatar#67" Configurable Machine for MIPS, i386, X86_64 Jan 25, 2021
@rawsample rawsample merged commit ee8fcc7 into avatartwo:dev/qemu-5.1 Feb 3, 2021
@AndrewFasano AndrewFasano deleted the dev/qemu-5.1-configurable branch February 17, 2021 23:36
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.

2 participants