[libvirt] [PATCH v2] bhyve: Add support for VNC autoport
Roman Bogorodskiy
bogorodskiy at gmail.com
Sun Aug 6 06:04:28 UTC 2017
Ján Tomko wrote:
> On Sun, Jun 25, 2017 at 08:31:24PM +0400, Roman Bogorodskiy wrote:
> >From: Alexander Nusov <alexander.nusov at nfvexpress.com>
> >
> >This patch adds support for automatic VNC port assignment for bhyve guests.
> >---
> >Changes from v1:
> >
> > * Call virPortAllocatorRelease() in virBhyveProcessStop() to release
> > VNC port; that's done unconditionally of using autoport
> > * Call virPortAllocatorSetUsed(.., true) in virBhyveProcessReconnect()
> > to reserve already used VNC ports after daemon restart
> > * Call virPortAllocatorSetUsed(.., true) in bhyveBuildGraphicsArgStr()
> > for domains that don't use autoport so allocator didn't try to use
> > ports allocated by these domains
> > * In dryRun mode (i.e. for domxml-to-native) don't allocate any ports
> > * Add a couple of unit tests
> >
> >Note 1: while adding tests I noticed that port allocator will actually
> >skip already bound ports, so I'm wondering if it makes any sense to use
> >virPortAllocatorSetUsed(.., true)? Right now I cannot come up with any
> >case to trigger this except probably some races when spawning guests
> >simultaneously, but that's hard to reproduce.
> >Note 2: there are still some cases where resources allocated during
> >command preparation are not properly cleaned up; that's not only VNC
> >ports, but also TAP devices. I plan to add proper cleanup routines
> >separately.
> >
> >
> > src/bhyve/bhyve_command.c | 25 +++++++++++--
> > src/bhyve/bhyve_driver.c | 5 +++
> > src/bhyve/bhyve_process.c | 20 +++++++++++
> > src/bhyve/bhyve_utils.h | 3 ++
> > .../bhyvexml2argv-vnc-autoport.args | 12 +++++++
> > .../bhyvexml2argv-vnc-autoport.ldargs | 1 +
> > .../bhyvexml2argv-vnc-autoport.xml | 26 ++++++++++++++
> > tests/bhyvexml2argvtest.c | 7 ++++
> > .../bhyvexml2xmlout-vnc-autoport.xml | 41 ++++++++++++++++++++++
> > tests/bhyvexml2xmltest.c | 1 +
> > 10 files changed, 138 insertions(+), 3 deletions(-)
> > create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-vnc-autoport.args
> > create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-vnc-autoport.ldargs
> > create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-vnc-autoport.xml
> > create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-vnc-autoport.xml
> >
>
> ACK
>
> Jan
Pushed, thanks!
Roman Bogorodskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 455 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170806/d306a8cd/attachment-0001.sig>
More information about the libvir-list
mailing list