usr/src/lib/fm/topo/libtopo/common/topo_prop.c
author robj
Tue, 04 Dec 2007 13:52:58 -0800
changeset 5590 4c010356e39b
parent 5068 06d88145c7f7
child 5646 de282ace882c
permissions -rw-r--r--
6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node 6628101 libtopo: topo_xml.c::propset_contains() can return a false match 6627354 Need to provide FRU labels for chip-select topo nodes on AMD family 0x10 platforms 6628254 libtopo: g4_chip_label() segfaults on X4600 M2 with family 0x10 modules
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     1
/*
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     2
 * CDDL HEADER START
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     3
 *
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     4
 * The contents of this file are subject to the terms of the
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     5
 * Common Development and Distribution License (the "License").
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     6
 * You may not use this file except in compliance with the License.
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     7
 *
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     8
 * You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
     9
 * or http://www.opensolaris.org/os/licensing.
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    10
 * See the License for the specific language governing permissions
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    11
 * and limitations under the License.
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    12
 *
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    13
 * When distributing Covered Code, include this CDDL HEADER in each
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    14
 * file and include the License file at usr/src/OPENSOLARIS.LICENSE.
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    15
 * If applicable, add the following below this CDDL HEADER, with the
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    16
 * fields enclosed by brackets "[]" replaced with your own identifying
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    17
 * information: Portions Copyright [yyyy] [name of copyright owner]
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    18
 *
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    19
 * CDDL HEADER END
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    20
 */
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    21
/*
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    22
 * Copyright 2007 Sun Microsystems, Inc.  All rights reserved.
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    23
 * Use is subject to license terms.
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    24
 */
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    25
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    26
#pragma ident	"%Z%%M%	%I%	%E% SMI"
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    27
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    28
#include <strings.h>
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    29
#include <assert.h>
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    30
#include <fm/libtopo.h>
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    31
#include <topo_prop.h>
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    32
#include <topo_string.h>
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    33
#include <topo_alloc.h>
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    34
#include <topo_error.h>
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    35
#include <topo_method.h>
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    36
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    37
/*
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    38
 * Topology nodes are permitted to contain property information.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    39
 * Property information is organized according to property grouping.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    40
 * Each property group defines a name, a stability level for that name,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    41
 * a stability level for all underlying property data (name, type, values),
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    42
 * a version for the property group definition and and a list of uniquely
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    43
 * defined properties.  Property group versions are incremented when one of
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    44
 * the following changes occurs:
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    45
 *	- a property name changes
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    46
 *	- a property type changes
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    47
 *	- a property definition is removed from the group
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    48
 * Compatible changes such as new property definitions in the group do
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    49
 * not require version changes.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    50
 *
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    51
 * Each property defines a unique (within the group) name, a type and
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    52
 * a value.  Properties may be statically defined as int32, uint32, int64,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    53
 * uint64, fmri, string or arrays of each type.  Properties may also be
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    54
 * dynamically exported via module registered methods.  For example, a module
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    55
 * may register a method to export an ASRU property that is dynamically
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    56
 * contructed when a call to topo_node_fmri() is invoked for a particular
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    57
 * topology node.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    58
 *
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    59
 * Static properties are persistently attached to topology nodes during
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    60
 * enumeration by an enumeration module or as part of XML statements in a
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    61
 * toplogy map file using the topo_prop_set* family of routines.  Similarly,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    62
 * property methods are registered during enumeration or as part of
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    63
 * statements in topololgy map files.  Set-up of property methods is performed
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    64
 * by calling topo_prop_method_register().
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    65
 *
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    66
 * All properties, whether statically persisted in a snapshot or dynamically
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    67
 * obtained, may be read via the topo_prop_get* family of interfaces.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    68
 * Callers wishing to receive all property groups and properties for a given
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    69
 * node may use topo_prop_getall().  This routine returns a nested nvlist
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    70
 * of all groupings and property (name, type, value) sets.  Groupings
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    71
 * are defined by TOPO_PROP_GROUP (name, data stability, name stability and
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    72
 * version) and a nested nvlist of properties (TOPO_PROP_VAL).  Each property
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    73
 * value is defined by its name, type and value.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    74
 */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    75
static void topo_propval_destroy(topo_propval_t *);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    76
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    77
static topo_pgroup_t *
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    78
pgroup_get(tnode_t *node, const char *pgname)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    79
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    80
	topo_pgroup_t *pg;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    81
	/*
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    82
	 * Check for an existing pgroup
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    83
	 */
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    84
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    85
	    pg = topo_list_next(pg)) {
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
    86
		if (strcmp(pg->tpg_info->tpi_name, pgname) == 0) {
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    87
			return (pg);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    88
		}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    89
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    90
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    91
	return (NULL);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    92
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    93
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    94
static topo_propval_t *
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    95
propval_get(topo_pgroup_t *pg, const char *pname)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    96
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    97
	topo_proplist_t *pvl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
    98
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
    99
	if (pg == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   100
		return (NULL);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   101
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   102
	for (pvl = topo_list_next(&pg->tpg_pvals); pvl != NULL;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   103
	    pvl = topo_list_next(pvl)) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   104
		if (strcmp(pvl->tp_pval->tp_name, pname) == 0)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   105
			return (pvl->tp_pval);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   106
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   107
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   108
	return (NULL);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   109
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   110
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   111
static int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   112
method_geterror(nvlist_t *nvl, int err, int *errp)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   113
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   114
	if (nvl != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   115
		nvlist_free(nvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   116
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   117
	*errp = err;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   118
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   119
	return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   120
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   121
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   122
static int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   123
prop_method_get(tnode_t *node, topo_propval_t *pv, topo_propmethod_t *pm,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   124
    nvlist_t *pargs, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   125
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   126
	int ret;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   127
	nvlist_t *args, *nvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   128
	char *name;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   129
	topo_type_t type;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   130
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   131
	if (topo_hdl_nvalloc(pv->tp_hdl, &args, NV_UNIQUE_NAME) < 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   132
	    nvlist_add_nvlist(args, TOPO_PROP_ARGS, pm->tpm_args) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   133
		return (method_geterror(NULL, ETOPO_PROP_NVL, err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   134
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   135
	if (pargs != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   136
		if (nvlist_add_nvlist(args, TOPO_PROP_PARGS, pargs) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   137
			return (method_geterror(args, ETOPO_PROP_NVL, err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   138
5590
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   139
	/*
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   140
	 * Now, get the latest value
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   141
	 *
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   142
	 * Grab a reference to the property and then unlock the node.  This will
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   143
	 * allow property methods to safely re-enter the prop_get codepath,
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   144
	 * making it possible for property methods to access other property
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   145
	 * values on the same node w\o causing a deadlock.
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   146
	 */
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   147
	topo_prop_hold(pv);
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   148
	topo_node_unlock(node);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   149
	if (topo_method_call(node, pm->tpm_name, pm->tpm_version,
5590
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   150
	    args, &nvl, err) < 0) {
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   151
		topo_prop_rele(pv);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   152
		return (method_geterror(args, *err, err));
5590
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   153
	}
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   154
	topo_node_lock(node);
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   155
	topo_prop_rele(pv);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   156
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   157
	nvlist_free(args);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   158
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   159
	/* Verify the property contents */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   160
	ret = nvlist_lookup_string(nvl, TOPO_PROP_VAL_NAME, &name);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   161
	if (ret != 0 || strcmp(name, pv->tp_name) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   162
		return (method_geterror(nvl, ETOPO_PROP_NAME, err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   163
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   164
	ret = nvlist_lookup_uint32(nvl, TOPO_PROP_VAL_TYPE, (uint32_t *)&type);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   165
	if (ret != 0 || type != pv->tp_type)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   166
		return (method_geterror(nvl, ETOPO_PROP_TYPE, err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   167
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   168
	/* Release the last value and re-assign to the new value */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   169
	if (pv->tp_val != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   170
		nvlist_free(pv->tp_val);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   171
	pv->tp_val = nvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   172
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   173
	return (0);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   174
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   175
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   176
static topo_propval_t *
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   177
prop_get(tnode_t *node, const char *pgname, const char *pname, nvlist_t *pargs,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   178
    int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   179
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   180
	topo_propval_t *pv = NULL;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   181
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   182
	if ((pv = propval_get(pgroup_get(node, pgname), pname)) == NULL) {
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   183
		*err = ETOPO_PROP_NOENT;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   184
		return (NULL);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   185
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   186
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   187
	if (pv->tp_method != NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   188
		if (prop_method_get(node, pv, pv->tp_method, pargs, err) < 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   189
			return (NULL);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   190
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   191
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   192
	return (pv);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   193
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   194
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   195
static int
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   196
get_properror(tnode_t *node, int *errp, int err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   197
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   198
	topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   199
	*errp = err;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   200
	return (-1);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   201
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   202
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   203
static int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   204
prop_getval(tnode_t *node, const char *pgname, const char *pname, void *val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   205
    topo_type_t type, uint_t *nelems, int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   206
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   207
	int i, j, ret = 0;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   208
	topo_hdl_t *thp = node->tn_hdl;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   209
	topo_propval_t *pv;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   210
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   211
	topo_node_lock(node);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   212
	if ((pv = prop_get(node, pgname, pname, NULL, err))
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   213
	    == NULL)
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   214
		return (get_properror(node, err, *err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   215
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   216
	if (pv->tp_type != type)
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   217
		return (get_properror(node, err, ETOPO_PROP_TYPE));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   218
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   219
	switch (type) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   220
		case TOPO_TYPE_INT32:
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   221
			ret = nvlist_lookup_int32(pv->tp_val, TOPO_PROP_VAL_VAL,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   222
			    (int32_t *)val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   223
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   224
		case TOPO_TYPE_UINT32:
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   225
			ret = nvlist_lookup_uint32(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   226
			    TOPO_PROP_VAL_VAL, (uint32_t *)val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   227
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   228
		case TOPO_TYPE_INT64:
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   229
			ret = nvlist_lookup_int64(pv->tp_val, TOPO_PROP_VAL_VAL,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   230
			    (int64_t *)val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   231
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   232
		case TOPO_TYPE_UINT64:
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   233
			ret = nvlist_lookup_uint64(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   234
			    TOPO_PROP_VAL_VAL, (uint64_t *)val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   235
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   236
		case TOPO_TYPE_STRING: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   237
			char *str;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   238
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   239
			ret = nvlist_lookup_string(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   240
			    TOPO_PROP_VAL_VAL, &str);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   241
			if (ret == 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   242
				char *s2;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   243
				if ((s2 = topo_hdl_strdup(thp, str)) == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   244
					ret = -1;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   245
				else
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   246
					*(char **)val = s2;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   247
			}
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   248
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   249
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   250
		case TOPO_TYPE_FMRI: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   251
			nvlist_t *nvl;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   252
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   253
			ret = nvlist_lookup_nvlist(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   254
			    TOPO_PROP_VAL_VAL, &nvl);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   255
			if (ret == 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   256
				ret = topo_hdl_nvdup(thp, nvl,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   257
				    (nvlist_t **)val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   258
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   259
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   260
		case TOPO_TYPE_INT32_ARRAY: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   261
			int32_t *a1, *a2;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   262
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   263
			if ((ret = nvlist_lookup_int32_array(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   264
			    TOPO_PROP_VAL_VAL, &a2, nelems)) != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   265
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   266
			if ((a1 = topo_hdl_alloc(thp, sizeof (int32_t) *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   267
			    *nelems)) == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   268
				ret = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   269
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   270
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   271
			for (i = 0; i < *nelems; ++i)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   272
				a1[i] = a2[i];
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   273
			*(int32_t **)val = a1;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   274
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   275
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   276
		case TOPO_TYPE_UINT32_ARRAY: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   277
			uint32_t *a1, *a2;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   278
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   279
			if ((ret = nvlist_lookup_uint32_array(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   280
			    TOPO_PROP_VAL_VAL, &a2, nelems)) != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   281
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   282
			if ((a1 = topo_hdl_alloc(thp, sizeof (uint32_t) *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   283
			    *nelems)) == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   284
				ret = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   285
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   286
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   287
			for (i = 0; i < *nelems; ++i)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   288
				a1[i] = a2[i];
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   289
			*(uint32_t **)val = a1;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   290
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   291
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   292
		case TOPO_TYPE_INT64_ARRAY: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   293
			int64_t *a1, *a2;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   294
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   295
			if ((ret = nvlist_lookup_int64_array(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   296
			    TOPO_PROP_VAL_VAL, &a2, nelems)) != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   297
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   298
			if ((a1 = topo_hdl_alloc(thp, sizeof (int64_t) *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   299
			    *nelems)) == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   300
				ret = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   301
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   302
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   303
			for (i = 0; i < *nelems; ++i)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   304
				a1[i] = a2[i];
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   305
			*(int64_t **)val = a1;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   306
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   307
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   308
		case TOPO_TYPE_UINT64_ARRAY: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   309
			uint64_t *a1, *a2;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   310
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   311
			if ((ret = nvlist_lookup_uint64_array(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   312
			    TOPO_PROP_VAL_VAL, &a2, nelems)) != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   313
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   314
			if ((a1 = topo_hdl_alloc(thp, sizeof (uint64_t) *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   315
			    *nelems)) == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   316
				ret = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   317
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   318
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   319
			for (i = 0; i < *nelems; ++i)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   320
				a1[i] = a2[i];
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   321
			*(uint64_t **)val = a1;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   322
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   323
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   324
		case TOPO_TYPE_STRING_ARRAY: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   325
			char **a1, **a2;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   326
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   327
			if ((ret = nvlist_lookup_string_array(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   328
			    TOPO_PROP_VAL_VAL, &a2, nelems)) != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   329
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   330
			if ((a1 = topo_hdl_alloc(thp, sizeof (char *) *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   331
			    *nelems)) == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   332
				ret = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   333
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   334
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   335
			for (i = 0; i < *nelems; ++i) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   336
				if ((a1[i] = topo_hdl_strdup(thp, a2[i]))
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   337
				    == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   338
					for (j = 0; j < i; ++j)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   339
						topo_hdl_free(thp, a1[j],
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   340
						    sizeof (char *));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   341
					topo_hdl_free(thp, a1,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   342
					    sizeof (char *) * *nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   343
					break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   344
				}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   345
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   346
			*(char ***)val = a1;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   347
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   348
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   349
		case TOPO_TYPE_FMRI_ARRAY: {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   350
			nvlist_t **a1, **a2;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   351
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   352
			if ((ret = nvlist_lookup_nvlist_array(pv->tp_val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   353
			    TOPO_PROP_VAL_VAL, &a2, nelems)) != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   354
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   355
			if ((a1 = topo_hdl_alloc(thp, sizeof (nvlist_t *) *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   356
			    *nelems)) == NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   357
				ret = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   358
				break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   359
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   360
			for (i = 0; i < *nelems; ++i) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   361
				if (topo_hdl_nvdup(thp, a2[i], &a1[i]) < 0) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   362
					for (j = 0; j < i; ++j)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   363
						nvlist_free(a1[j]);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   364
					topo_hdl_free(thp, a1,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   365
					    sizeof (nvlist_t *) * *nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   366
					break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   367
				}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   368
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   369
			*(nvlist_t ***)val = a1;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   370
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   371
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   372
		default:
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   373
			ret = ETOPO_PROP_NOENT;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   374
	}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   375
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   376
	if (ret != 0)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   377
		if (ret == ENOENT)
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   378
			return (get_properror(node, err, ETOPO_PROP_NOENT));
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   379
		else if (ret < ETOPO_UNKNOWN)
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   380
			return (get_properror(node, err, ETOPO_PROP_NVL));
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   381
		else
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   382
			return (get_properror(node, err, ret));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   383
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   384
	topo_node_unlock(node);
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   385
	return (0);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   386
}
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   387
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   388
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   389
topo_prop_get_int32(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   390
    int32_t *val, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   391
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   392
	return (prop_getval(node, pgname, pname, (void *)val, TOPO_TYPE_INT32,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   393
	    NULL, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   394
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   395
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   396
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   397
topo_prop_get_uint32(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   398
    uint32_t *val, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   399
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   400
	return (prop_getval(node, pgname, pname, (void *)val, TOPO_TYPE_UINT32,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   401
	    NULL, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   402
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   403
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   404
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   405
topo_prop_get_int64(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   406
    int64_t *val, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   407
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   408
	return (prop_getval(node, pgname, pname, (void *)val, TOPO_TYPE_INT64,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   409
	    NULL, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   410
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   411
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   412
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   413
topo_prop_get_uint64(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   414
    uint64_t *val, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   415
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   416
	return (prop_getval(node, pgname, pname, (void *)val, TOPO_TYPE_UINT64,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   417
	    NULL, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   418
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   419
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   420
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   421
topo_prop_get_string(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   422
    char **val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   423
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   424
	return (prop_getval(node, pgname, pname, (void *)val, TOPO_TYPE_STRING,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   425
	    NULL, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   426
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   427
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   428
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   429
topo_prop_get_fmri(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   430
    nvlist_t **val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   431
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   432
	return (prop_getval(node, pgname, pname, (void *)val, TOPO_TYPE_FMRI,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   433
	    NULL, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   434
}
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   435
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   436
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   437
topo_prop_get_int32_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   438
    int32_t **val, uint_t *nelem, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   439
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   440
	return (prop_getval(node, pgname, pname, (void *)val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   441
	    TOPO_TYPE_INT32_ARRAY, nelem, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   442
}
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   443
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   444
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   445
topo_prop_get_uint32_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   446
    uint32_t **val, uint_t *nelem, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   447
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   448
	return (prop_getval(node, pgname, pname, (void *)val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   449
	    TOPO_TYPE_UINT32_ARRAY, nelem, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   450
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   451
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   452
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   453
topo_prop_get_int64_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   454
    int64_t **val, uint_t *nelem, int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   455
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   456
	return (prop_getval(node, pgname, pname, (void *)val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   457
	    TOPO_TYPE_INT64_ARRAY, nelem, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   458
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   459
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   460
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   461
topo_prop_get_uint64_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   462
    uint64_t **val, uint_t *nelem, int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   463
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   464
	return (prop_getval(node, pgname, pname, (void *)val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   465
	    TOPO_TYPE_UINT64_ARRAY, nelem, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   466
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   467
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   468
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   469
topo_prop_get_string_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   470
    char ***val, uint_t *nelem, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   471
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   472
	return (prop_getval(node, pgname, pname, (void *)val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   473
	    TOPO_TYPE_STRING_ARRAY, nelem, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   474
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   475
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   476
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   477
topo_prop_get_fmri_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   478
    nvlist_t ***val, uint_t *nelem, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   479
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   480
	return (prop_getval(node, pgname, pname, (void *)val,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   481
	    TOPO_TYPE_FMRI_ARRAY, nelem, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   482
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   483
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   484
static topo_propval_t *
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   485
set_seterror(tnode_t *node, topo_proplist_t *pvl, int *errp, int err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   486
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   487
	topo_hdl_t *thp = node->tn_hdl;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   488
	topo_propval_t *pv;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   489
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   490
	if (pvl != NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   491
		pv = pvl->tp_pval;
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   492
		topo_propval_destroy(pv);
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   493
		topo_hdl_free(thp, pvl, sizeof (topo_proplist_t));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   494
	}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   495
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   496
	topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   497
	*errp = err;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   498
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   499
	return (NULL);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   500
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   501
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   502
static topo_propval_t *
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   503
prop_create(tnode_t *node, const char *pgname, const char *pname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   504
    topo_type_t type, int flag, int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   505
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   506
	topo_hdl_t *thp = node->tn_hdl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   507
	topo_pgroup_t *pg;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   508
	topo_propval_t *pv;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   509
	topo_proplist_t *pvl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   510
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   511
	/*
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   512
	 * Replace existing prop value with new one
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   513
	 */
4328
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
   514
	if ((pg = pgroup_get(node, pgname)) == NULL) {
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
   515
		topo_node_unlock(node);
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
   516
		*err = ETOPO_PROP_NOENT;
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   517
		return (NULL);
4328
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
   518
	}
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   519
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   520
	if ((pv = propval_get(pg, pname)) != NULL) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   521
		if (pv->tp_type != type)
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   522
			return (set_seterror(node, NULL, err, ETOPO_PROP_TYPE));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   523
		else if (pv->tp_flag == TOPO_PROP_IMMUTABLE)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   524
			return (set_seterror(node, NULL, err, ETOPO_PROP_DEFD));
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   525
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   526
		nvlist_free(pv->tp_val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   527
		pv->tp_val = NULL;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   528
	} else {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   529
		if ((pvl = topo_hdl_zalloc(thp, sizeof (topo_proplist_t)))
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   530
		    == NULL)
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   531
			return (set_seterror(node, NULL, err, ETOPO_NOMEM));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   532
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   533
		if ((pv = topo_hdl_zalloc(thp, sizeof (topo_propval_t)))
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   534
		    == NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   535
			return (set_seterror(node, pvl, err, ETOPO_NOMEM));
4328
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
   536
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
   537
		pv->tp_hdl = thp;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   538
		pvl->tp_pval = pv;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   539
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   540
		if ((pv->tp_name = topo_hdl_strdup(thp, pname))
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   541
		    == NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   542
			return (set_seterror(node, pvl, err, ETOPO_NOMEM));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   543
		pv->tp_flag = flag;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   544
		pv->tp_type = type;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   545
		topo_prop_hold(pv);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   546
		topo_list_append(&pg->tpg_pvals, pvl);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   547
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   548
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   549
	return (pv);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   550
}
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   551
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   552
static int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   553
topo_prop_set(tnode_t *node, const char *pgname, const char *pname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   554
    topo_type_t type, int flag, void *val, int nelems, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   555
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   556
	int ret;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   557
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   558
	nvlist_t *nvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   559
	topo_propval_t *pv;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   560
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   561
	if (topo_hdl_nvalloc(thp, &nvl, NV_UNIQUE_NAME) < 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   562
		*err = ETOPO_PROP_NVL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   563
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   564
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   565
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   566
	ret = nvlist_add_string(nvl, TOPO_PROP_VAL_NAME, pname);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   567
	ret |= nvlist_add_uint32(nvl, TOPO_PROP_VAL_TYPE, type);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   568
	switch (type) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   569
		case TOPO_TYPE_INT32:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   570
			ret |= nvlist_add_int32(nvl, TOPO_PROP_VAL_VAL,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   571
			    *(int32_t *)val);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   572
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   573
		case TOPO_TYPE_UINT32:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   574
			ret |= nvlist_add_uint32(nvl, TOPO_PROP_VAL_VAL,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   575
			    *(uint32_t *)val);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   576
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   577
		case TOPO_TYPE_INT64:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   578
			ret |= nvlist_add_int64(nvl, TOPO_PROP_VAL_VAL,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   579
			    *(int64_t *)val);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   580
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   581
		case TOPO_TYPE_UINT64:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   582
			ret |= nvlist_add_uint64(nvl, TOPO_PROP_VAL_VAL,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   583
			    *(uint64_t *)val);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   584
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   585
		case TOPO_TYPE_STRING:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   586
			ret |= nvlist_add_string(nvl, TOPO_PROP_VAL_VAL,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   587
			    (char *)val);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   588
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   589
		case TOPO_TYPE_FMRI:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   590
			ret |= nvlist_add_nvlist(nvl, TOPO_PROP_VAL_VAL,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   591
			    (nvlist_t *)val);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   592
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   593
		case TOPO_TYPE_INT32_ARRAY:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   594
			ret |= nvlist_add_int32_array(nvl,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   595
			    TOPO_PROP_VAL_VAL, (int32_t *)val, nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   596
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   597
		case TOPO_TYPE_UINT32_ARRAY:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   598
			ret |= nvlist_add_uint32_array(nvl,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   599
			    TOPO_PROP_VAL_VAL, (uint32_t *)val, nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   600
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   601
		case TOPO_TYPE_INT64_ARRAY:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   602
			ret |= nvlist_add_int64_array(nvl,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   603
			    TOPO_PROP_VAL_VAL, (int64_t *)val, nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   604
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   605
		case TOPO_TYPE_UINT64_ARRAY:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   606
			ret |= nvlist_add_uint64_array(nvl,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   607
			    TOPO_PROP_VAL_VAL, (uint64_t *)val, nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   608
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   609
		case TOPO_TYPE_STRING_ARRAY:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   610
			ret |= nvlist_add_string_array(nvl,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   611
			    TOPO_PROP_VAL_VAL, (char **)val, nelems);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   612
			break;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   613
		case TOPO_TYPE_FMRI_ARRAY:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   614
			ret |= nvlist_add_nvlist_array(nvl,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   615
			    TOPO_PROP_VAL_VAL, (nvlist_t **)val, nelems);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   616
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   617
		default:
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   618
			*err = ETOPO_PROP_TYPE;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   619
			return (-1);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   620
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   621
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   622
	if (ret != 0) {
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   623
		nvlist_free(nvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   624
		if (ret == ENOMEM) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   625
			*err = ETOPO_PROP_NOMEM;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   626
			return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   627
		} else {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   628
			*err = ETOPO_PROP_NVL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   629
			return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   630
		}
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   631
	}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   632
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   633
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   634
	if ((pv = prop_create(node, pgname, pname, type, flag, err)) == NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   635
		nvlist_free(nvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   636
		return (-1); /* unlocked and err set */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   637
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   638
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   639
	pv->tp_val = nvl;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   640
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   641
	topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   642
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   643
	return (ret);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   644
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   645
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   646
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   647
topo_prop_set_int32(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   648
    int flag, int32_t val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   649
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   650
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_INT32, flag,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   651
	    &val, 1, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   652
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   653
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   654
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   655
topo_prop_set_uint32(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   656
    int flag, uint32_t val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   657
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   658
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_UINT32, flag,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   659
	    &val, 1, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   660
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   661
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   662
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   663
topo_prop_set_int64(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   664
    int flag, int64_t val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   665
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   666
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_INT64, flag,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   667
	    &val, 1, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   668
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   669
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   670
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   671
topo_prop_set_uint64(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   672
    int flag, uint64_t val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   673
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   674
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_UINT64, flag,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   675
	    &val, 1, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   676
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   677
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   678
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   679
topo_prop_set_string(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   680
    int flag, const char *val, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   681
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   682
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_STRING, flag,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   683
	    (void *)val, 1, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   684
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   685
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   686
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   687
topo_prop_set_fmri(tnode_t *node, const char *pgname, const char *pname,
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   688
    int flag, const nvlist_t *fmri, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   689
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   690
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_FMRI, flag,
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   691
	    (void *)fmri, 1, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   692
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   693
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   694
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   695
topo_prop_set_int32_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   696
    int flag, int32_t *val, uint_t nelems, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   697
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   698
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_INT32_ARRAY, flag,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   699
	    val, nelems, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   700
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   701
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   702
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   703
topo_prop_set_uint32_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   704
    int flag, uint32_t *val, uint_t nelems, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   705
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   706
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_UINT32_ARRAY, flag,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   707
	    val, nelems, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   708
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   709
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   710
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   711
topo_prop_set_int64_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   712
    int flag, int64_t *val, uint_t nelems, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   713
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   714
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_INT64_ARRAY, flag,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   715
	    val, nelems, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   716
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   717
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   718
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   719
topo_prop_set_uint64_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   720
    int flag, uint64_t *val, uint_t nelems, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   721
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   722
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_UINT64_ARRAY, flag,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   723
	    val, nelems, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   724
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   725
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   726
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   727
topo_prop_set_string_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   728
    int flag, const char **val, uint_t nelems, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   729
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   730
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_STRING_ARRAY, flag,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   731
	    (void *)val, nelems, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   732
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   733
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   734
int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   735
topo_prop_set_fmri_array(tnode_t *node, const char *pgname, const char *pname,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   736
    int flag, const nvlist_t **fmri, uint_t nelems, int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   737
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   738
	return (topo_prop_set(node, pgname, pname, TOPO_TYPE_FMRI_ARRAY, flag,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
   739
	    (void *)fmri, nelems, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   740
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   741
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   742
/*
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   743
 * topo_prop_setprop() is a private project function for fmtopo
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   744
 */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   745
int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   746
topo_prop_setprop(tnode_t *node, const char *pgname, nvlist_t *prop,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   747
    int flag, nvlist_t *pargs, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   748
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   749
	int ret;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   750
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   751
	topo_propval_t *pv;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   752
	nvlist_t *nvl, *args;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   753
	char *name;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   754
	topo_type_t type;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   755
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   756
	if (nvlist_lookup_string(prop, TOPO_PROP_VAL_NAME, &name) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   757
		*err = ETOPO_PROP_NAME;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   758
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   759
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   760
	if (nvlist_lookup_uint32(prop, TOPO_PROP_VAL_TYPE, (uint32_t *)&type)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   761
	    != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   762
		*err = ETOPO_PROP_TYPE;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   763
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   764
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   765
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   766
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   767
	if ((pv = prop_create(node, pgname, name, type, flag, err)) == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   768
		return (-1); /* unlocked and err set */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   769
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   770
	/*
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   771
	 * Set by method or set to new prop value.  If we fail, leave
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   772
	 * property in list with old value.
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   773
	 */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   774
	if (pv->tp_method != NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   775
		topo_propmethod_t *pm = pv->tp_method;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   776
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   777
		if (topo_hdl_nvalloc(pv->tp_hdl, &args, NV_UNIQUE_NAME) < 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   778
			topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   779
			*err = ETOPO_PROP_NOMEM;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   780
			return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   781
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   782
		ret = nvlist_add_nvlist(args, TOPO_PROP_ARGS, pm->tpm_args);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   783
		if (pargs != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   784
			ret |= nvlist_add_nvlist(args, TOPO_PROP_PARGS, pargs);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   785
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   786
		if (ret != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   787
			topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   788
			nvlist_free(args);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   789
			*err = ETOPO_PROP_NVL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   790
			return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   791
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   792
5590
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   793
		/*
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   794
		 *
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   795
		 * Grab a reference to the property and then unlock the node.
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   796
		 * This will allow property methods to safely re-enter the
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   797
		 * prop_get codepath, making it possible for property methods
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   798
		 * to access other property values on the same node w\o causing
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   799
		 * a deadlock.
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   800
		 *
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   801
		 * We don't technically need this now, since this interface is
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   802
		 * currently only used by fmtopo (which is single-threaded), but
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   803
		 * we may make this interface available to other parts of
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   804
		 * libtopo in the future, so best to make it MT-safe now.
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   805
		 */
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   806
		topo_prop_hold(pv);
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   807
		topo_node_unlock(node);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   808
		ret = topo_method_call(node, pm->tpm_name, pm->tpm_version,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   809
		    args, &nvl, err);
5590
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   810
		topo_node_lock(node);
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   811
		topo_prop_rele(pv);
4c010356e39b 6631732 libtopo: propmethods can cause deadlock if they access other properties on the same node
robj
parents: 5068
diff changeset
   812
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   813
		nvlist_free(args);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   814
	} else {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   815
		if ((ret = topo_hdl_nvdup(thp, prop, &nvl)) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   816
			*err = ETOPO_PROP_NOMEM;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   817
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   818
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   819
	if (ret != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   820
		topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   821
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   822
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   823
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   824
	pv->tp_val = nvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   825
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   826
	return (0);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   827
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   828
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   829
static int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   830
register_methoderror(tnode_t *node, topo_propmethod_t *pm, int *errp, int l,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   831
    int err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   832
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   833
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   834
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   835
	if (pm != NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   836
		if (pm->tpm_name != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   837
			topo_hdl_strfree(thp, pm->tpm_name);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   838
		if (pm->tpm_args != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   839
			nvlist_free(pm->tpm_args);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   840
		topo_hdl_free(thp, pm, sizeof (topo_propmethod_t));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   841
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   842
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   843
	*errp = err;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   844
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   845
	if (l != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   846
		topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   847
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   848
	return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   849
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   850
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   851
int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   852
prop_method_register(tnode_t *node, const char *pgname, const char *pname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   853
    topo_type_t ptype, const char *mname, topo_version_t version,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   854
    const nvlist_t *args, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   855
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   856
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   857
	topo_propmethod_t *pm = NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   858
	topo_propval_t *pv = NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   859
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   860
	if ((pm = topo_hdl_zalloc(thp, sizeof (topo_propmethod_t))) == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   861
		return (register_methoderror(node, pm, err, 1,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   862
		    ETOPO_PROP_NOMEM));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   863
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   864
	if ((pm->tpm_name = topo_hdl_strdup(thp, mname)) == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   865
		return (register_methoderror(node, pm, err, 1,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   866
		    ETOPO_PROP_NOMEM));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   867
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   868
	pm->tpm_version = version;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   869
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   870
	if (topo_hdl_nvdup(thp, (nvlist_t *)args, &pm->tpm_args) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   871
		return (register_methoderror(node, pm, err, 1,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   872
		    ETOPO_PROP_NOMEM));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   873
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   874
	if ((pv = prop_create(node, pgname, pname, ptype, TOPO_PROP_MUTABLE,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   875
	    err)) == NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   876
		/* node unlocked */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   877
		return (register_methoderror(node, pm, err, 0, *err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   878
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   879
5068
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   880
	if (pv->tp_method != NULL)
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   881
		return (register_methoderror(node, pm, err, 1,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   882
		    ETOPO_METHOD_DEFD));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   883
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   884
	pv->tp_val = NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   885
	pv->tp_method = pm;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   886
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   887
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   888
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   889
	return (0);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   890
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   891
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   892
int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   893
topo_prop_method_register(tnode_t *node, const char *pgname, const char *pname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   894
    topo_type_t ptype, const char *mname, const nvlist_t *args, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   895
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   896
	topo_imethod_t *mp;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   897
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   898
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   899
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   900
	if ((mp = topo_method_lookup(node, mname)) == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   901
		return (register_methoderror(node, NULL, err, 1,
5068
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   902
		    ETOPO_METHOD_NOTSUP)); /* node unlocked */
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   903
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   904
	topo_node_lock(node);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   905
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   906
	return (prop_method_register(node, pgname, pname, ptype, mname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   907
	    mp->tim_version, args, err)); /* err set and node unlocked */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   908
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   909
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   910
int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   911
topo_prop_method_version_register(tnode_t *node, const char *pgname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   912
    const char *pname, topo_type_t ptype, const char *mname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   913
    topo_version_t version, const nvlist_t *args, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   914
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   915
	topo_imethod_t *mp;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   916
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   917
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   918
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   919
	if ((mp = topo_method_lookup(node, mname)) == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   920
		return (register_methoderror(node, NULL, err, 1,
5068
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   921
		    ETOPO_METHOD_NOTSUP)); /* node unlocked */
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   922
06d88145c7f7 6492246 x64 cpu/mem topology/diagnosis should include FRU labels
robj
parents: 4328
diff changeset
   923
	topo_node_lock(node);
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   924
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   925
	if (version < mp->tim_version)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   926
		return (register_methoderror(node, NULL, err, 1,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   927
		    ETOPO_METHOD_VEROLD));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   928
	if (version > mp->tim_version)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   929
		return (register_methoderror(node, NULL, err, 1,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   930
		    ETOPO_METHOD_VERNEW));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   931
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   932
	return (prop_method_register(node, pgname, pname, ptype, mname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   933
	    version, args, err)); /* err set and node unlocked */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   934
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   935
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   936
void
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   937
topo_prop_method_unregister(tnode_t *node, const char *pgname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   938
    const char *pname)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   939
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   940
	topo_propval_t *pv;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   941
	topo_pgroup_t *pg;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   942
	topo_proplist_t *pvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   943
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   944
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   945
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   946
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   947
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   948
	    pg = topo_list_next(pg)) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   949
		if (strcmp(pg->tpg_info->tpi_name, pgname) == 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   950
			break;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   951
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   952
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   953
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   954
	if (pg == NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   955
		topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   956
		return;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   957
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   958
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   959
	for (pvl = topo_list_next(&pg->tpg_list); pvl != NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   960
	    pvl = topo_list_next(pvl)) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   961
		pv = pvl->tp_pval;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   962
		if (strcmp(pv->tp_name, pname) == 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   963
			topo_list_delete(&pg->tpg_pvals, pvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   964
			assert(pv->tp_refs == 1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   965
			topo_prop_rele(pv);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   966
			topo_hdl_free(thp, pvl, sizeof (topo_proplist_t));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   967
			break;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   968
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   969
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   970
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   971
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   972
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   973
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   974
static int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   975
inherit_seterror(tnode_t *node, int *errp, int err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   976
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   977
	topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   978
	topo_node_unlock(node->tn_parent);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   979
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   980
	*errp = err;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   981
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   982
	return (-1);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   983
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   984
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   985
int
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   986
topo_prop_inherit(tnode_t *node, const char *pgname, const char *name, int *err)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   987
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   988
	topo_hdl_t *thp = node->tn_hdl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   989
	tnode_t *pnode = node->tn_parent;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   990
	topo_pgroup_t *pg;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   991
	topo_propval_t *pv;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   992
	topo_proplist_t *pvl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   993
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   994
	topo_node_lock(pnode);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   995
	topo_node_lock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   996
	/*
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   997
	 * Check for an existing property group and prop val
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
   998
	 */
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
   999
	if ((pv = propval_get(pgroup_get(pnode, pgname), name)) == NULL)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1000
		return (inherit_seterror(node, err, ETOPO_PROP_NOENT));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1001
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1002
	/*
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1003
	 * Can this propval be inherited?
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1004
	 */
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1005
	if (pv->tp_flag != TOPO_PROP_IMMUTABLE)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1006
		return (inherit_seterror(node, err, ETOPO_PROP_NOINHERIT));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1007
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1008
	/*
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1009
	 * Property group should already exist: bump the ref count for this
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1010
	 * propval and add it to the node's property group
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1011
	 */
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1012
	if ((pg = pgroup_get(node, pgname)) == NULL)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1013
		return (inherit_seterror(node, err, ETOPO_PROP_NOENT));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1014
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1015
	if ((pvl = topo_hdl_zalloc(thp, sizeof (topo_proplist_t)))
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1016
	    == NULL)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1017
		return (inherit_seterror(node, err, ETOPO_NOMEM));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1018
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1019
	topo_prop_hold(pv);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1020
	pvl->tp_pval = pv;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1021
	topo_list_append(&pg->tpg_pvals, pvl);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1022
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1023
	topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1024
	topo_node_unlock(pnode);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1025
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1026
	return (0);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1027
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1028
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1029
topo_pgroup_info_t *
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1030
topo_pgroup_info(tnode_t *node, const char *pgname, int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1031
{
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1032
	topo_hdl_t *thp = node->tn_hdl;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1033
	topo_pgroup_t *pg;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1034
	topo_ipgroup_info_t *pip;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1035
	topo_pgroup_info_t *info;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1036
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1037
	topo_node_lock(node);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1038
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1039
	    pg = topo_list_next(pg)) {
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1040
		if (strcmp(pgname, pg->tpg_info->tpi_name) == 0) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1041
			if ((info = topo_hdl_alloc(thp,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1042
			    sizeof (topo_pgroup_info_t))) == NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1043
				return (NULL);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1044
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1045
			pip = pg->tpg_info;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1046
			if ((info->tpi_name =
4328
fe6ac87d8e60 6547176 libtopo leaks memory on failure
cindi
parents: 4087
diff changeset
  1047
			    topo_hdl_strdup(thp, pip->tpi_name)) == NULL) {
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1048
				*err = ETOPO_PROP_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1049
				topo_hdl_free(thp, info,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1050
				    sizeof (topo_pgroup_info_t));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1051
				topo_node_unlock(node);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1052
				return (NULL);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1053
			}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1054
			info->tpi_namestab = pip->tpi_namestab;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1055
			info->tpi_datastab = pip->tpi_datastab;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1056
			info->tpi_version = pip->tpi_version;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1057
			topo_node_unlock(node);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1058
			return (info);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1059
		}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1060
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1061
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1062
	*err = ETOPO_PROP_NOENT;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1063
	topo_node_unlock(node);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1064
	return (NULL);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1065
}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1066
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1067
static int
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1068
pgroup_seterr(tnode_t *node, topo_pgroup_t *pg, topo_ipgroup_info_t *pip,
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1069
    int *err)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1070
{
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1071
	topo_hdl_t *thp = node->tn_hdl;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1072
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1073
	if (pip != NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1074
		if (pip->tpi_name != NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1075
			topo_hdl_strfree(thp, (char *)pip->tpi_name);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1076
		topo_hdl_free(thp, pip, sizeof (topo_ipgroup_info_t));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1077
	}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1078
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1079
	topo_hdl_free(thp, pg, sizeof (topo_pgroup_t));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1080
	*err = ETOPO_NOMEM;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1081
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1082
	topo_node_unlock(node);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1083
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1084
	return (-1);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1085
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1086
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1087
int
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1088
topo_pgroup_create(tnode_t *node, const topo_pgroup_info_t *pinfo, int *err)
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1089
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1090
	topo_pgroup_t *pg;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1091
	topo_ipgroup_info_t *pip;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1092
	topo_hdl_t *thp = node->tn_hdl;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1093
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1094
	*err = 0;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1095
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1096
	topo_node_lock(node);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1097
	/*
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1098
	 * Check for an existing pgroup
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1099
	 */
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1100
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1101
	    pg = topo_list_next(pg)) {
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1102
		if (strcmp(pg->tpg_info->tpi_name, pinfo->tpi_name) == 0) {
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1103
			*err = ETOPO_PROP_DEFD;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1104
			topo_node_unlock(node);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1105
			return (-1);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1106
		}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1107
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1108
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1109
	if ((pg = topo_hdl_zalloc(thp, sizeof (topo_pgroup_t))) == NULL) {
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1110
		*err = ETOPO_NOMEM;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1111
		topo_node_unlock(node);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1112
		return (-1);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1113
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1114
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1115
	if ((pip = topo_hdl_zalloc(thp, sizeof (topo_ipgroup_info_t)))
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1116
	    == NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1117
		return (pgroup_seterr(node, pg, pip, err));
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1118
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1119
	if ((pip->tpi_name = topo_hdl_strdup(thp, pinfo->tpi_name))
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1120
	    == NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1121
		return (pgroup_seterr(node, pg, pip, err));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1122
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1123
	pip->tpi_namestab = pinfo->tpi_namestab;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1124
	pip->tpi_datastab = pinfo->tpi_datastab;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1125
	pip->tpi_version = pinfo->tpi_version;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1126
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1127
	pg->tpg_info = pip;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1128
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1129
	topo_list_append(&node->tn_pgroups, pg);
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1130
	topo_node_unlock(node);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1131
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1132
	return (0);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1133
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1134
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1135
void
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1136
topo_pgroup_destroy(tnode_t *node, const char *pname)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1137
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1138
	topo_hdl_t *thp = node->tn_hdl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1139
	topo_pgroup_t *pg;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1140
	topo_proplist_t *pvl;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1141
	topo_ipgroup_info_t *pip;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1142
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1143
	topo_node_lock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1144
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1145
	    pg = topo_list_next(pg)) {
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1146
		if (strcmp(pg->tpg_info->tpi_name, pname) == 0) {
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1147
			break;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1148
		}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1149
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1150
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1151
	if (pg == NULL) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1152
		topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1153
		return;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1154
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1155
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1156
	while ((pvl = topo_list_next(&pg->tpg_list)) != NULL) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1157
		topo_list_delete(&pg->tpg_pvals, pvl);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1158
		topo_prop_rele(pvl->tp_pval);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1159
		topo_hdl_free(thp, pvl, sizeof (topo_proplist_t));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1160
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1161
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1162
	topo_list_delete(&node->tn_pgroups, pg);
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1163
	topo_node_unlock(node);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1164
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1165
	pip = pg->tpg_info;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1166
	if (pip != NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1167
		if (pip->tpi_name != NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1168
			topo_hdl_strfree(thp, (char *)pip->tpi_name);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1169
		topo_hdl_free(thp, pip, sizeof (topo_ipgroup_info_t));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1170
	}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1171
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1172
	topo_hdl_free(thp, pg, sizeof (topo_pgroup_t));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1173
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1174
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1175
void
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1176
topo_pgroup_destroy_all(tnode_t *node)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1177
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1178
	topo_hdl_t *thp = node->tn_hdl;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1179
	topo_pgroup_t *pg;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1180
	topo_proplist_t *pvl;
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1181
	topo_ipgroup_info_t *pip;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1182
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1183
	topo_node_lock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1184
	while ((pg = topo_list_next(&node->tn_pgroups)) != NULL) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1185
		while ((pvl = topo_list_next(&pg->tpg_pvals)) != NULL) {
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1186
			topo_list_delete(&pg->tpg_pvals, pvl);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1187
			topo_prop_rele(pvl->tp_pval);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1188
			topo_hdl_free(thp, pvl, sizeof (topo_proplist_t));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1189
		}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1190
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1191
		topo_list_delete(&node->tn_pgroups, pg);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1192
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1193
		pip = pg->tpg_info;
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1194
		if (pip != NULL) {
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1195
			if (pip->tpi_name != NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1196
				topo_hdl_strfree(thp, (char *)pip->tpi_name);
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1197
			topo_hdl_free(thp, pip, sizeof (topo_pgroup_info_t));
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1198
		}
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1199
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1200
		topo_hdl_free(thp, pg, sizeof (topo_pgroup_t));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1201
	}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1202
	topo_node_unlock(node);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1203
}
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1204
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1205
static void
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1206
propmethod_destroy(topo_hdl_t *thp, topo_propval_t *pv)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1207
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1208
	topo_propmethod_t *pm;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1209
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1210
	pm = pv->tp_method;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1211
	if (pm != NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1212
		if (pm->tpm_name != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1213
			topo_hdl_strfree(thp, pm->tpm_name);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1214
		if (pm->tpm_args != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1215
			nvlist_free(pm->tpm_args);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1216
		topo_hdl_free(thp, pm, sizeof (topo_propmethod_t));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1217
		pv->tp_method = NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1218
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1219
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1220
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1221
static void
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1222
topo_propval_destroy(topo_propval_t *pv)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1223
{
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1224
	topo_hdl_t *thp;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1225
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1226
	if (pv == NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1227
		return;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1228
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1229
	thp = pv->tp_hdl;
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1230
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1231
	if (pv->tp_name != NULL)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1232
		topo_hdl_strfree(thp, pv->tp_name);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1233
3062
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1234
	if (pv->tp_val != NULL)
46d280f5351d 6396916 verification of dtd file name is wrong
cindi
parents: 1414
diff changeset
  1235
		nvlist_free(pv->tp_val);
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1236
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1237
	propmethod_destroy(thp, pv);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1238
1414
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1239
	topo_hdl_free(thp, pv, sizeof (topo_propval_t));
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1240
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1241
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1242
void
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1243
topo_prop_hold(topo_propval_t *pv)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1244
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1245
	pv->tp_refs++;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1246
}
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1247
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1248
void
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1249
topo_prop_rele(topo_propval_t *pv)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1250
{
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1251
	pv->tp_refs--;
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1252
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1253
	assert(pv->tp_refs >= 0);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1254
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1255
	if (pv->tp_refs == 0)
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1256
		topo_propval_destroy(pv);
b4126407ac5b PSARC 2006/020 FMA for Athlon 64 and Opteron Processors
cindi
parents:
diff changeset
  1257
}
4087
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1258
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1259
/*
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1260
 * topo_prop_getprop() and topo_prop_getprops() are private project functions
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1261
 * for fmtopo
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1262
 */
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1263
int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1264
topo_prop_getprop(tnode_t *node, const char *pgname, const char *pname,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1265
    nvlist_t *args, nvlist_t **prop, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1266
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1267
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1268
	topo_propval_t *pv;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1269
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1270
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1271
	if ((pv = prop_get(node, pgname, pname, args, err)) == NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1272
		(void) get_properror(node, err, *err);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1273
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1274
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1275
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1276
	if (topo_hdl_nvdup(thp, pv->tp_val, prop) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1277
		(void) get_properror(node, err, ETOPO_NOMEM);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1278
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1279
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1280
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1281
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1282
	return (0);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1283
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1284
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1285
static int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1286
prop_val_add(tnode_t *node, nvlist_t **nvl, topo_propval_t *pv, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1287
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1288
	if (pv->tp_method != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1289
		if (prop_method_get(node, pv, pv->tp_method, NULL, err) < 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1290
			return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1291
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1292
	if (pv->tp_val == NULL) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1293
		*err = ETOPO_PROP_NOENT;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1294
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1295
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1296
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1297
	if (topo_hdl_nvdup(pv->tp_hdl, pv->tp_val, nvl) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1298
		*err = ETOPO_PROP_NOMEM;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1299
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1300
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1301
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1302
	return (0);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1303
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1304
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1305
static int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1306
get_pgrp_seterror(tnode_t *node, nvlist_t *nvl, int *errp, int err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1307
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1308
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1309
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1310
	if (nvl != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1311
		nvlist_free(nvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1312
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1313
	*errp = err;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1314
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1315
	return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1316
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1317
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1318
int
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1319
topo_prop_getpgrp(tnode_t *node, const char *pgname, nvlist_t **pgrp,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1320
    int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1321
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1322
	int ret;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1323
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1324
	nvlist_t *nvl, *pvnvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1325
	topo_pgroup_t *pg;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1326
	topo_propval_t *pv;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1327
	topo_proplist_t *pvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1328
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1329
	if (topo_hdl_nvalloc(thp, &nvl, 0) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1330
		*err = ETOPO_NOMEM;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1331
		return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1332
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1333
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1334
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1335
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1336
	    pg = topo_list_next(pg)) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1337
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1338
		if (strcmp(pgname, pg->tpg_info->tpi_name) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1339
			continue;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1340
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1341
		if (nvlist_add_string(nvl, TOPO_PROP_GROUP_NAME,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1342
		    pg->tpg_info->tpi_name) != 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1343
		    nvlist_add_string(nvl, TOPO_PROP_GROUP_NSTAB,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1344
		    topo_stability2name(pg->tpg_info->tpi_namestab)) != 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1345
		    nvlist_add_string(nvl, TOPO_PROP_GROUP_DSTAB,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1346
		    topo_stability2name(pg->tpg_info->tpi_datastab)) != 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1347
		    nvlist_add_int32(nvl, TOPO_PROP_GROUP_VERSION,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1348
		    pg->tpg_info->tpi_version) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1349
			return (get_pgrp_seterror(node, nvl, err,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1350
			    ETOPO_PROP_NVL));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1351
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1352
		for (pvl = topo_list_next(&pg->tpg_pvals); pvl != NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1353
		    pvl = topo_list_next(pvl)) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1354
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1355
			pv = pvl->tp_pval;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1356
			if (prop_val_add(node, &pvnvl, pv, err) < 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1357
				return (get_pgrp_seterror(node, nvl, err,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1358
				    *err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1359
			}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1360
			if ((ret = nvlist_add_nvlist(nvl, TOPO_PROP_VAL,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1361
			    pvnvl)) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1362
				nvlist_free(pvnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1363
				return (get_pgrp_seterror(node, nvl, err, ret));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1364
			}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1365
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1366
			nvlist_free(pvnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1367
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1368
		topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1369
		*pgrp = nvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1370
		return (0);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1371
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1372
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1373
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1374
	*err = ETOPO_PROP_NOENT;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1375
	return (-1);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1376
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1377
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1378
static nvlist_t *
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1379
get_all_seterror(tnode_t *node, nvlist_t *nvl, int *errp, int err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1380
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1381
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1382
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1383
	if (nvl != NULL)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1384
		nvlist_free(nvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1385
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1386
	*errp = err;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1387
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1388
	return (NULL);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1389
}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1390
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1391
nvlist_t *
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1392
topo_prop_getprops(tnode_t *node, int *err)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1393
{
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1394
	int ret;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1395
	topo_hdl_t *thp = node->tn_hdl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1396
	nvlist_t *nvl, *pgnvl, *pvnvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1397
	topo_pgroup_t *pg;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1398
	topo_propval_t *pv;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1399
	topo_proplist_t *pvl;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1400
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1401
	topo_node_lock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1402
	if (topo_hdl_nvalloc(thp, &nvl, 0) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1403
		return (get_all_seterror(node, NULL, err, ETOPO_NOMEM));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1404
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1405
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1406
	for (pg = topo_list_next(&node->tn_pgroups); pg != NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1407
	    pg = topo_list_next(pg)) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1408
		if (topo_hdl_nvalloc(thp, &pgnvl, 0) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1409
			return (get_all_seterror(node, nvl, err, ETOPO_NOMEM));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1410
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1411
		if (nvlist_add_string(pgnvl, TOPO_PROP_GROUP_NAME,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1412
		    pg->tpg_info->tpi_name) != 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1413
		    nvlist_add_string(pgnvl, TOPO_PROP_GROUP_NSTAB,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1414
		    topo_stability2name(pg->tpg_info->tpi_namestab)) != 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1415
		    nvlist_add_string(pgnvl, TOPO_PROP_GROUP_DSTAB,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1416
		    topo_stability2name(pg->tpg_info->tpi_datastab)) != 0 ||
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1417
		    nvlist_add_int32(pgnvl, TOPO_PROP_GROUP_VERSION,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1418
		    pg->tpg_info->tpi_version) != 0)
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1419
			return (get_all_seterror(node, nvl, err,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1420
			    ETOPO_PROP_NVL));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1421
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1422
		for (pvl = topo_list_next(&pg->tpg_pvals); pvl != NULL;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1423
		    pvl = topo_list_next(pvl)) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1424
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1425
			pv = pvl->tp_pval;
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1426
			if (prop_val_add(node, &pvnvl, pv, err) < 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1427
				nvlist_free(pgnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1428
				return (get_all_seterror(node, nvl, err, *err));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1429
			}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1430
			if ((ret = nvlist_add_nvlist(pgnvl, TOPO_PROP_VAL,
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1431
			    pvnvl)) != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1432
				nvlist_free(pgnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1433
				nvlist_free(pvnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1434
				return (get_all_seterror(node, nvl, err, ret));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1435
			}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1436
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1437
			nvlist_free(pvnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1438
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1439
		if ((ret = nvlist_add_nvlist(nvl, TOPO_PROP_GROUP, pgnvl))
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1440
		    != 0) {
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1441
			nvlist_free(pgnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1442
			return (get_all_seterror(node, nvl, err, ret));
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1443
		}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1444
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1445
		nvlist_free(pgnvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1446
	}
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1447
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1448
	topo_node_unlock(node);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1449
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1450
	return (nvl);
b986877655d2 6521594 fmtopo '[-Cdev]' usage doesn't make sense
cindi
parents: 3062
diff changeset
  1451
}