-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Kvm vgic #5163
Kvm vgic #5163
Conversation
1f996b1
to
b34e224
Compare
I had to rename the ICC registers because the old toolchain in syz-old-env didn't recognize them. |
a752d05
to
d121d8f
Compare
PTAL (take your time, not sure it's the best idea to land the series today) |
What's interesting is that the coverage with this series went down by ~500 BBs and gained back only ~300 during the day. |
If it does not restore, potentially it's possible to look at a diff coverage report. |
6281b7b
to
2ef8b8c
Compare
@dvyukov PTAL |
I tested the patches adding device attributes separately, and they were able to gain the same coverage as master. |
The new pseudo-syscall sets up VGICv3 IRQ controller on the host. That still requires guest setup code, which will be submitted separately.
2ef8b8c
to
289d87d
Compare
Implement basic IRQ controller setup for VMs with a single CPU. SYZOS_API_IRQ_SETUP sets up the VGICv3 distributor/redistributor and enables the specified number of SPIs starting from 32. The default IRQ handler is set up to perform a uexit(-2).
According to https://docs.kernel.org/virt/kvm/devices/vcpu.html, VCPUs also have device attributes. Add descriptions for ioctls that manipulate those attributes.
Do not specify the attribute input types for now.
289d87d
to
1bbcea1
Compare
Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md