Page MenuHomeFreeBSD

vmm: add VM_DESTROY ioctl call
Needs ReviewPublic

Authored by gusev.vitaliy_gmail.com on Apr 18 2022, 10:04 PM.
Tags
Referenced Files
Unknown Object (File)
Fri, Jan 24, 6:00 PM
Unknown Object (File)
Fri, Jan 10, 1:35 AM
Unknown Object (File)
Fri, Jan 3, 6:05 AM
Unknown Object (File)
Fri, Jan 3, 6:01 AM
Unknown Object (File)
Dec 5 2024, 8:29 PM
Unknown Object (File)
Nov 16 2024, 11:22 PM
Unknown Object (File)
Nov 15 2024, 2:59 AM
Unknown Object (File)
Nov 14 2024, 8:38 AM

Details

Reviewers
None
Group Reviewers
bhyve
Summary

If bhyve compiled with capsicum support, vm_destroy() fails because sysctl calls are not allowed.

This is counterpart of https://reviews.freebsd.org/D31325 can probably can be used as replacement.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Mm, this conflicts with a change I have to make destroy synchronous due to a race with destroy not finishing before bhyve restarts in a loop. I think instead you might want some sort of model where you can set a flag when creating a VM (or perhaps via an ioctl) that causes the VM to be destroyed on last close.

The conflicting change I haven't yet uploaded for review but you can see it here: https://github.com/bsdjhb/freebsd/commit/213d5795a8a1e1835bf936d1c820ffd272e68b11

In D34954#793575, @jhb wrote:

Mm, this conflicts with a change I have to make destroy synchronous due to a race with destroy not finishing before bhyve restarts in a loop. I think instead you might want some sort of model where you can set a flag when creating a VM (or perhaps via an ioctl) that causes the VM to be destroyed on last close.

The conflicting change I haven't yet uploaded for review but you can see it here: https://github.com/bsdjhb/freebsd/commit/213d5795a8a1e1835bf936d1c820ffd272e68b11

I will check it out and update review. Thanks.