Page MenuHomeFreeBSD

virtio_gpu: Avoid panic if a 0x0 mode is provided
Needs ReviewPublic

Authored by val_packett.cool on Jun 25 2024, 5:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 7, 3:22 AM
Unknown Object (File)
Tue, Feb 3, 9:09 AM
Unknown Object (File)
Sat, Jan 31, 9:10 PM
Unknown Object (File)
Jan 12 2026, 6:54 PM
Unknown Object (File)
Dec 31 2025, 2:14 PM
Unknown Object (File)
Dec 22 2025, 7:29 AM
Unknown Object (File)
Dec 18 2025, 6:05 AM
Unknown Object (File)
Dec 10 2025, 11:09 AM
Subscribers

Details

Reviewers
andrew
Group Reviewers
Contributor Reviews (src)
Summary

On an amd64 QEMU host, using "-device virtio-gpu-gl" results in a 0x0
mode being provided, which would panic the kernel due to a zero-sized
allocation attempt. Suggest using virtio-vga(-gl) which does not have
that issue.

Sponsored by: Ihor Antonov <ihor@antonovs.family>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58342
Build 55230: arc lint + arc unit

Event Timeline

emaste added inline comments.
sys/dev/virtio/gpu/virtio_gpu.c
507

It's not necessarily 0x0 (e.g. 640x0 would also trigger this). We could either do "skipping %ux%u" and include the width and height, or e.g. "skipping display mode with width/height 0"

Or just skip it without a message