Page MenuHomeFreeBSD

bhyve vtblk: Inform guests of disk resize events.
ClosedPublic

Authored by jhb on May 27 2021, 1:25 PM.
Tags
None
Referenced Files
F86629853: D30506.id.diff
Sun, Jun 23, 7:22 AM
F86584647: D30506.id90003.diff
Sat, Jun 22, 2:55 PM
Unknown Object (File)
Thu, Jun 20, 5:11 AM
Unknown Object (File)
Wed, Jun 19, 2:57 PM
Unknown Object (File)
Wed, Jun 19, 1:44 PM
Unknown Object (File)
Thu, May 30, 3:37 PM
Unknown Object (File)
Apr 29 2024, 3:24 AM
Unknown Object (File)
Apr 5 2024, 4:38 PM
Subscribers

Details

Summary

Register a resize callback with the blockif interface. When the
callback fires, update the size of the disk and notify the guest via a
configuration change interrupt.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 39510
Build 36399: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jun 2 2021, 4:11 PM
grehan added inline comments.
usr.sbin/bhyve/pci_virtio_block.c
446

Does the lock need to be taken here ?

usr.sbin/bhyve/pci_virtio_block.c
446

Does the lock need to be taken here ?

I only did it to protect vbc_capacity, but if the lock isn't held when reading that field then it isn't doing much good. Hmm, I guess if we cared about the race we would need to grab the lock again in pci_vtblk_cfgread()? Maybe the race is such that we don't care? Or if cfg reads are rare, maybe it doesn't hurt to be paranoid and lock in the read?

jhb marked an inline comment as done.Jun 12 2021, 12:58 AM
jhb added inline comments.
usr.sbin/bhyve/pci_virtio_block.c
446

I've gone ahead and removed the lock.