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
