On Tuesday 01 June 2010 17:04, Mike Heinz wrote: > + logger -i "Set node_desc for ${hca}: ${NODE_DESC_HOSTNAME} > HCA-\${hca_id}" > + echo -n "${NODE_DESC_HOSTNAME} HCA-${hca_id}" >> > ${sysdir}/${hca}/node_desc >
I don't see NODE_DESC_HOSTNAME defined anywhere. Don't you need a line like: > + echo -n "${NODE_DESC_HOSTNAME:-...@} HCA-${hca_id}" >> > ${sysdir}/${hca}/node_desc So that you will have the "@" that the driver looks for if NODE_DESC_HOSTNAME is not set in the environment, or will use some other string if the user wishes to override by setting NODE_DESC_HOSTNAME to something else in the environment? Also, I am concerned about the trailing \0 character. It looks to me that your function ib_build_node_desc does not deal with this. For example, if the node name is very long, and the 64 bytes does not reach the end of the string stored in sysfs due to the length difference when replacing "@" with the node name. -Jack _______________________________________________ ewg mailing list ewg@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg