Skip to content

[Bug] UART has txfifo bug #3046

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

Open
cperciva opened this issue Jul 4, 2022 · 1 comment
Open

[Bug] UART has txfifo bug #3046

cperciva opened this issue Jul 4, 2022 · 1 comment
Labels
Priority: Low Indicates that an issue or pull request should be resolved behind issues or pull requests labelled ` Status: Parked Indicates that an issues or pull request will be revisited later Type: Bug Indicates an unexpected problem or unintended behavior

Comments

@cperciva
Copy link
Contributor

cperciva commented Jul 4, 2022

Describe the bug

Just tripped over this with FreeBSD: I need to set hw.broken_txfifo=1 to make the UART work once we enter userland in a Firecracker VM. This is a workaround I added to FreeBSD about a decade ago to address a QEMU/Xen bug which has long since been fixed there -- IIRC after the txfifo filled up, we wouldn't get an interrupt telling us when it had drained.

I haven't looked into the Firecracker UART code to see what's going on there; I'm opening this mainly as a placeholder so I don't forget to take a look at it later.

To Reproduce

Boot FreeBSD in Firecracker.

Expected behaviour

The UART should work. Instead, once FreeBSD hits userland it writes 16 characters and then gets stuck.

Environment

Running aljimenezb's pvh-v2 branch. FreeBSD HEAD with Firecracker hacks. m5.metal EC2 instance.

@cperciva cperciva added the Type: Bug Indicates an unexpected problem or unintended behavior label Jul 4, 2022
@mattschlebusch mattschlebusch added the Priority: Medium Indicates than an issue or pull request should be resolved ahead of issues or pull requests labelled label May 9, 2023
@pb8o
Copy link
Contributor

pb8o commented Sep 18, 2023

It may need this PR in rust-vmm first: rust-vmm/vm-superio#96

@pb8o pb8o added Priority: Low Indicates that an issue or pull request should be resolved behind issues or pull requests labelled ` and removed Priority: Medium Indicates than an issue or pull request should be resolved ahead of issues or pull requests labelled labels Sep 18, 2023
@ShadowCurse ShadowCurse added the Status: Parked Indicates that an issues or pull request will be revisited later label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low Indicates that an issue or pull request should be resolved behind issues or pull requests labelled ` Status: Parked Indicates that an issues or pull request will be revisited later Type: Bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

4 participants