Project

General

Profile

Actions

Bug #14506

closed

libidspace does not expose id_allocff()

Added by Andy Fiddaman 6 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:

Description

The userland id space library does not currently expose id_allocff() and this would be useful to have. The work on the common code for this was done, but it was omitted from the mapfile and private header.


Related issues

Related to illumos gate - Feature #14197: Implement id_space as a libraryClosedAndy Fiddaman

Actions
Actions #1

Updated by Andy Fiddaman 6 months ago

Actions #2

Updated by Andy Fiddaman 6 months ago

  • Subject changed from libidspace does not expose id_allocff to libidspace does not expose id_allocff()
Actions #3

Updated by Electric Monk 6 months ago

  • Gerrit CR set to 2030
Actions #4

Updated by Andy Fiddaman 5 months ago

  • Status changed from New to In Progress
Actions #5

Updated by Andy Fiddaman 4 months ago

This is being used by the refactored VNC server in #14495 and #14494 for assigning an ID number to each active VNC session.
The debug RFB logs show that IDs are being assigned, and gaps being re-used.

rfb0: connection from ::1
rfb1: connection from ::1
rfb2: connection from ::1
rfb1: client disconnected
rfb1: connection from ::1
rfb3: connection from ::1

Actions #6

Updated by Electric Monk 4 months ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 1aa1f41fe10c6220a7fbef328fac1b72a8355a01

commit  1aa1f41fe10c6220a7fbef328fac1b72a8355a01
Author: Andy Fiddaman <omnios@citrus-it.co.uk>
Date:   2022-04-23T11:02:39.000Z

    14495 bhyve VNC server should support alternate pixel formats
    14494 bhyve VNC server could support multiple connections
    14505 libumem's umem_update thread could be named
    14506 libidspace does not expose id_allocff()
    14640 bhyve VNC authentication fails with openssl3
    Reviewed by: Jason King <jason.brian.king+illumos@gmail.com>
    Reviewed by: Michael Zeller <mike@mikezeller.net>
    Approved by: Robert Mustacchi <rm@fingolfin.org>

Actions

Also available in: Atom PDF