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

tcg: Do not elide memory barriers for !CF_PARALLEL in system mode #236

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

jrtc27
Copy link
Member

@jrtc27 jrtc27 commented Aug 17, 2023

The virtio devices require proper memory ordering between the vcpus and the iothreads.

Reviewed-by: Philippe Mathieu-Daudé [email protected]
Signed-off-by: Richard Henderson [email protected]

(cherry picked from commit c914d46d0a645e7c633292146f4e38c945d4f847)

The virtio devices require proper memory ordering between
the vcpus and the iothreads.

Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Signed-off-by: Richard Henderson <[email protected]>

(cherry picked from commit c914d46d0a645e7c633292146f4e38c945d4f847)
@arichardson
Copy link
Member

Is this the cause of the timeouts after virtio_random being initialized by cheribsd?

@arichardson
Copy link
Member

FYI I'm planning to update the 6.0 merge PR soon, not sure if that includes this fix.

@jrtc27
Copy link
Member Author

jrtc27 commented Aug 17, 2023

Is this the cause of the timeouts after virtio_random being initialized by cheribsd?

Can't say, but that did make me think of this as a possibility. On my weaker memory model arm64 MacBook it manifested years ago as vtnet queue inconsistencies that caused assertion failures. Don't know how it shows up on x86. Perhaps it doesn't due to sequential consistency. Regardless, it's definitely wrong as it is today.

FYI I'm planning to update the 6.0 merge PR soon, not sure if that includes this fix.

v8.1.0-rc0 is the first tag it shows up in for GitLab, https://patchwork.kernel.org/project/qemu-devel/cover/[email protected]/ is the finalised patch series

@jrtc27 jrtc27 enabled auto-merge (rebase) August 17, 2023 05:20
@jrtc27 jrtc27 merged commit 4fb6c37 into dev Aug 17, 2023
26 checks passed
@jrtc27 jrtc27 deleted the tcg-mb branch August 17, 2023 10:22
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