From nobody Thu Jan 29 15:39:45 2026 X-Original-To: dev-commits-src-branches@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4f23H61HxZz6QrML for ; Thu, 29 Jan 2026 15:39:46 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R13" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4f23H60CM9z3xkZ for ; Thu, 29 Jan 2026 15:39:46 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1769701186; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=UgC28bYDb4NP8gV0i+pyih8qmqaZonQksxsjgqOofLg=; b=anRJKLxbQGxtBvknKOHtFqfBAtzVlkp7ObftSk/hfzsFLlrTJT1Mx79CRub/FX7w0DT2Sr +nNcUux4sCaOwF2dQM0Ay+wdwLbR+qGvhCO2SimqurRYAmwApMeQp7reiPCcJKP6g8nxnz Xm7o/vM4/FtZb2Da9h6lvvgj4O/P1Kds0LlPZN9nuT8+zsmaWKy4okJ7NlwPhm6LY1Crlv 5y+0TIH7mjm4Y/rQWoGaX/pv7RdaEDxFGXRc1lt4pWyMdNuwDXBz7/LgLSu8/ryGuHqUXG OBMGjTkvRGYEK2zBefJscZCAoS0JvOE93cC120w0Yno/I+BaIQEIdzkLwtr6XA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1769701186; a=rsa-sha256; cv=none; b=iwACQfyZqr5aAcqtE4M+N8ZcnMK4ArzfZAGSc2o0CtYfcenfKD/KZ7lAxwKbQfhd2U/5S/ 6AOeC9FiYieyp41HGYkP4zBHOf4BGHYZicH7gOeLQP+y+PhktH5v4uvUdJh+HChod3HRYo lI2t90si681hvxVlxfkjnHaSTJLHD777Fiw0wh/I7mAAXa2mg5m6ebuHjzBUrx2BPWhpbX PAMlvsk1v5ZIc5hK1Fcb/HyoSAVaQn7+eLkwFlJHLnqMTG5W23CbAgVRXON6XzJ/b7RNYo R5MHjOyTdwh9rz05vd1W94ksbGEdR6sAYevzjPLddd/pIPlt1je8B15Y39yd+w== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1769701186; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=UgC28bYDb4NP8gV0i+pyih8qmqaZonQksxsjgqOofLg=; b=OGV7YTz2RuKn7il6VUCBAui2aLJTVJNab+kJjF1aIyHwQiZhnK4L9SAhSVRgaZbqWsnpwi 0bYz0xo58wDDSyOzcz5b3FQdSPUVaU7T89BFhI6JL5RtT9ad5/NSaL4fHklOc2ut7Z9858 S1lvIBTPuMinpkH4qkjCo+PEAqbDPk3NLGyLFxRqb9rO9PRCexn/I3Ztta9pTjq3dEow15 LBvEQsU9gfPpbQ4JQbz6A5/K2aZ2uu4ZtWHh6Q4ktUVCL1UVTRiZu18LDsAY9Q71oppy4Z Ha4it/6klxFsHVJ+WVGy2v0tpuf4fEHwvkGSsGURKByaIMcfmEMggINL8d/78g== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4f23H56JK5zl6y for ; Thu, 29 Jan 2026 15:39:45 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 37e76 by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Thu, 29 Jan 2026 15:39:45 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: John Baldwin Subject: git: 3b622bdf474b - stable/14 - ctladm: Use nvlist instead of home-rolled name-value lists List-Id: Commits to the stable branches of the FreeBSD src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-branches List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-branches@freebsd.org Sender: owner-dev-commits-src-branches@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: jhb X-Git-Repository: src X-Git-Refname: refs/heads/stable/14 X-Git-Reftype: branch X-Git-Commit: 3b622bdf474bc2100be99f29fa1c69d96ec13c38 Auto-Submitted: auto-generated Date: Thu, 29 Jan 2026 15:39:45 +0000 Message-Id: <697b7f41.37e76.6a48be5a@gitrepo.freebsd.org> The branch stable/14 has been updated by jhb: URL: https://cgit.FreeBSD.org/src/commit/?id=3b622bdf474bc2100be99f29fa1c69d96ec13c38 commit 3b622bdf474bc2100be99f29fa1c69d96ec13c38 Author: John Baldwin AuthorDate: 2025-02-05 19:54:47 +0000 Commit: John Baldwin CommitDate: 2026-01-29 15:14:10 +0000 ctladm: Use nvlist instead of home-rolled name-value lists Reviewed by: asomers Sponsored by: Chelsio Communications Differential Revision: https://reviews.freebsd.org/D48774 (cherry picked from commit 4dfc2553c3732b142fd6122f223971419e359799) --- usr.sbin/ctladm/ctladm.c | 125 ++++++++++++++++++++--------------------------- 1 file changed, 54 insertions(+), 71 deletions(-) diff --git a/usr.sbin/ctladm/ctladm.c b/usr.sbin/ctladm/ctladm.c index 5b73e22d6c0d..d20456ceae45 100644 --- a/usr.sbin/ctladm/ctladm.c +++ b/usr.sbin/ctladm/ctladm.c @@ -3222,14 +3222,17 @@ bailout: return (retval); } -/* - * Name/value pair used for per-LUN attributes. - */ -struct cctl_lun_nv { - char *name; - char *value; - STAILQ_ENTRY(cctl_lun_nv) links; -}; +/* Helper function to output values from an nvlist of strings. */ +static void +print_nvlist(const nvlist_t *nvl, const char *fmt) +{ + const char *name; + void *cookie; + + cookie = NULL; + while ((name = nvlist_next(nvl, NULL, &cookie)) != NULL) + printf(fmt, name, nvlist_get_string(nvl, name)); +} /* * Backend LUN information. @@ -3241,7 +3244,7 @@ struct cctl_lun { uint32_t blocksize; char *serial_number; char *device_id; - STAILQ_HEAD(,cctl_lun_nv) attr_list; + nvlist_t *attr_list; STAILQ_ENTRY(cctl_lun) links; }; @@ -3283,7 +3286,7 @@ cctl_start_element(void *user_data, const char *name, const char **attr) devlist->num_luns++; devlist->cur_lun = cur_lun; - STAILQ_INIT(&cur_lun->attr_list); + cur_lun->attr_list = nvlist_create(NV_FLAG_NO_UNIQUE); STAILQ_INSERT_TAIL(&devlist->lun_list, cur_lun, links); for (i = 0; attr[i] != NULL; i += 2) { @@ -3303,6 +3306,7 @@ cctl_end_element(void *user_data, const char *name) struct cctl_devlist_data *devlist; struct cctl_lun *cur_lun; char *str; + int error; devlist = (struct cctl_devlist_data *)user_data; cur_lun = devlist->cur_lun; @@ -3349,21 +3353,12 @@ cctl_end_element(void *user_data, const char *name) } else if (strcmp(name, "ctllunlist") == 0) { /* Nothing. */ } else { - struct cctl_lun_nv *nv; - - nv = calloc(1, sizeof(*nv)); - if (nv == NULL) - err(1, "%s: can't allocate %zd bytes for nv pair", - __func__, sizeof(*nv)); - - nv->name = strdup(name); - if (nv->name == NULL) - err(1, "%s: can't allocated %zd bytes for string", - __func__, strlen(name)); - - nv->value = str; + nvlist_move_string(cur_lun->attr_list, name, str); + error = nvlist_error(cur_lun->attr_list); + if (error != 0) + errc(1, error, "%s: can't add lun attribute nv pair", + __func__); str = NULL; - STAILQ_INSERT_TAIL(&cur_lun->attr_list, nv, links); } free(str); @@ -3467,8 +3462,6 @@ retry: printf("LUN Backend %18s %4s %-16s %-16s\n", "Size (Blocks)", "BS", "Serial Number", "Device ID"); STAILQ_FOREACH(lun, &devlist.lun_list, links) { - struct cctl_lun_nv *nv; - if ((backend != NULL) && (strcmp(lun->backend_type, backend) != 0)) continue; @@ -3481,9 +3474,7 @@ retry: if (verbose == 0) continue; - STAILQ_FOREACH(nv, &lun->attr_list, links) { - printf(" %s=%s\n", nv->name, nv->value); - } + print_nvlist(lun->attr_list, " %s=%s\n"); } bailout: free(lun_str); @@ -3501,9 +3492,9 @@ struct cctl_port { char *name; int pp, vp; char *target, *port, *lun_map; - STAILQ_HEAD(,cctl_lun_nv) init_list; - STAILQ_HEAD(,cctl_lun_nv) lun_list; - STAILQ_HEAD(,cctl_lun_nv) attr_list; + nvlist_t *init_list; + nvlist_t *lun_list; + nvlist_t *attr_list; STAILQ_ENTRY(cctl_port) links; }; @@ -3554,9 +3545,9 @@ cctl_start_pelement(void *user_data, const char *name, const char **attr) portlist->num_ports++; portlist->cur_port = cur_port; - STAILQ_INIT(&cur_port->init_list); - STAILQ_INIT(&cur_port->lun_list); - STAILQ_INIT(&cur_port->attr_list); + cur_port->init_list = nvlist_create(0); + cur_port->lun_list = nvlist_create(0); + cur_port->attr_list = nvlist_create(NV_FLAG_NO_UNIQUE); cur_port->port_id = portlist->cur_id; STAILQ_INSERT_TAIL(&portlist->port_list, cur_port, links); } @@ -3567,7 +3558,9 @@ cctl_end_pelement(void *user_data, const char *name) { struct cctl_portlist_data *portlist; struct cctl_port *cur_port; + char idname[16]; char *str; + int error; portlist = (struct cctl_portlist_data *)user_data; cur_port = portlist->cur_port; @@ -3622,31 +3615,28 @@ cctl_end_pelement(void *user_data, const char *name) portlist->cur_port = NULL; } else if (strcmp(name, "ctlportlist") == 0) { /* Nothing. */ + } else if (strcmp(name, "initiator") == 0) { + snprintf(idname, sizeof(idname), "%ju", portlist->cur_id); + nvlist_move_string(cur_port->init_list, idname, str); + error = nvlist_error(cur_port->init_list); + if (error != 0) + errc(1, error, "%s: can't add initiator nv pair", + __func__); + str = NULL; + } else if (strcmp(name, "lun") == 0) { + snprintf(idname, sizeof(idname), "%ju", portlist->cur_id); + nvlist_move_string(cur_port->lun_list, idname, str); + error = nvlist_error(cur_port->lun_list); + if (error != 0) + errc(1, error, "%s: can't add LUN nv pair", __func__); + str = NULL; } else { - struct cctl_lun_nv *nv; - - nv = calloc(1, sizeof(*nv)); - if (nv == NULL) - err(1, "%s: can't allocate %zd bytes for nv pair", - __func__, sizeof(*nv)); - - if (strcmp(name, "initiator") == 0 || - strcmp(name, "lun") == 0) - asprintf(&nv->name, "%ju", portlist->cur_id); - else - nv->name = strdup(name); - if (nv->name == NULL) - err(1, "%s: can't allocated %zd bytes for string", - __func__, strlen(name)); - - nv->value = str; + nvlist_move_string(cur_port->attr_list, name, str); + error = nvlist_error(cur_port->attr_list); + if (error != 0) + errc(1, error, "%s: can't add lun attribute nv pair", + __func__); str = NULL; - if (strcmp(name, "initiator") == 0) - STAILQ_INSERT_TAIL(&cur_port->init_list, nv, links); - else if (strcmp(name, "lun") == 0) - STAILQ_INSERT_TAIL(&cur_port->lun_list, nv, links); - else - STAILQ_INSERT_TAIL(&cur_port->attr_list, nv, links); } free(str); @@ -3763,8 +3753,6 @@ retry: if (quiet == 0) printf("Port Online Frontend Name pp vp\n"); STAILQ_FOREACH(port, &portlist.port_list, links) { - struct cctl_lun_nv *nv; - if ((frontend != NULL) && (strcmp(port->frontend_type, frontend) != 0)) continue; @@ -3780,27 +3768,22 @@ retry: if (init || verbose) { if (port->target) printf(" Target: %s\n", port->target); - STAILQ_FOREACH(nv, &port->init_list, links) { - printf(" Initiator %s: %s\n", - nv->name, nv->value); - } + print_nvlist(port->init_list, " Initiator %s: %s\n"); } if (lun || verbose) { if (port->lun_map) { - STAILQ_FOREACH(nv, &port->lun_list, links) - printf(" LUN %s: %s\n", - nv->name, nv->value); - if (STAILQ_EMPTY(&port->lun_list)) + if (nvlist_empty(port->lun_list)) printf(" No LUNs mapped\n"); + else + print_nvlist(port->lun_list, + " LUN %s: %s\n"); } else printf(" All LUNs mapped\n"); } if (verbose) { - STAILQ_FOREACH(nv, &port->attr_list, links) { - printf(" %s=%s\n", nv->name, nv->value); - } + print_nvlist(port->attr_list, " %s=%s\n"); } } bailout: