[ofa-general] Re: [PATCH] infiniband-diags/ibtracert: print lids in decimal form

[Ira Weiny]

On Thu, 15 Nov 2007 06:30:17 -0800
Hal Rosenstock <hrosenstock at xsigo.com> wrote:

> On Thu, 2007-11-15 at 12:37 +0200, Sasha Khapyorsky wrote:
> > As stated in bug#504 (https://bugs.openfabrics.org/show_bug.cgi?id=504)
> > lid output format unification is needed. Print LIDs as decimal in
> > ibtracert.
> 
> I'd prefer to see this done as some sort of option. Also, I think hex is
> better for MLIDs.

While I support the output of decimal for the diags I don't know if this
changes any of the scripts.  We should check that and I don't have the time
right now.

Sasha do any of the scripts use ibtracert?  Does anyone else on the list use
the output?

Ira

> It would be best to see the bigger picture for what will change to
> support decimal LIDs in management and ibutils but this is a first trial
> balloon IMO.
> 
> -- Hal
> 
> > 
> > Signed-off-by: Sasha Khapyorsky <sashak at voltaire.com>
> > ---
> >  infiniband-diags/src/ibtracert.c |   10 +++++-----
> >  1 files changed, 5 insertions(+), 5 deletions(-)
> > 
> > diff --git a/infiniband-diags/src/ibtracert.c b/infiniband-diags/src/ibtracert.c
> > index 010f45f..284ae2a 100644
> > --- a/infiniband-diags/src/ibtracert.c
> > +++ b/infiniband-diags/src/ibtracert.c
> > @@ -207,7 +207,7 @@ dump_endnode(int dump, char *prompt, Node *node, Port *port)
> >  
> >  	nodename = remap_node_name(node_name_map, node->nodeguid, node->nodedesc);
> >  
> > -	fprintf(f, "%s %s {0x%016" PRIx64 "} portnum %d lid 0x%x-0x%x \"%s\"\n",
> > +	fprintf(f, "%s %s {0x%016" PRIx64 "} portnum %d lid %u-%u \"%s\"\n",
> >  		prompt,
> >  		(node->type <= IB_NODE_MAX ? node_type_str[node->type] : "???"),
> >  		node->nodeguid, node->type == IB_NODE_SWITCH ? 0 : port->portnum,
> > @@ -231,7 +231,7 @@ dump_route(int dump, Node *node, int outport, Port *port)
> >  		fprintf(f, "[%d] -> {0x%016" PRIx64 "}[%d]\n",
> >  			outport, port->portguid, port->portnum);
> >  	else
> > -		fprintf(f, "[%d] -> %s port {0x%016" PRIx64 "}[%d] lid 0x%x-0x%x \"%s\"\n",
> > +		fprintf(f, "[%d] -> %s port {0x%016" PRIx64 "}[%d] lid %u-%u \"%s\"\n",
> >  			outport,
> >  			(node->type <= IB_NODE_MAX ? node_type_str[node->type] : "???"),
> >  			port->portguid, port->portnum,
> > @@ -640,7 +640,7 @@ dump_mcpath(Node *node, int dumplevel)
> >  	nodename = remap_node_name(node_name_map, node->nodeguid, node->nodedesc);
> >  
> >  	if (!node->dist) {
> > -		printf("From %s 0x%" PRIx64 " port %d lid 0x%x-0x%x \"%s\"\n",
> > +		printf("From %s 0x%" PRIx64 " port %d lid %u-%u \"%s\"\n",
> >  			(node->type <= IB_NODE_MAX ? node_type_str[node->type] : "???"),
> >  			node->nodeguid, node->ports->portnum, node->ports->lid,
> >  			node->ports->lid + (1 << node->ports->lmc) - 1,
> > @@ -655,7 +655,7 @@ dump_mcpath(Node *node, int dumplevel)
> >  				(node->type <= IB_NODE_MAX ? node_type_str[node->type] : "???"),
> >  				node->nodeguid, node->upport);
> >  		else
> > -			printf("[%d] -> %s 0x%" PRIx64 "[%d] lid 0x%x \"%s\"\n",
> > +			printf("[%d] -> %s 0x%" PRIx64 "[%d] lid %u \"%s\"\n",
> >  				node->ports->remoteport->portnum,
> >  				(node->type <= IB_NODE_MAX ? node_type_str[node->type] : "???"),
> >  				node->nodeguid, node->upport,
> > @@ -664,7 +664,7 @@ dump_mcpath(Node *node, int dumplevel)
> >  
> >  	if (node->dist < 0)
> >  	/* target node */
> > -		printf("To %s 0x%" PRIx64 " port %d lid 0x%x-0x%x \"%s\"\n",
> > +		printf("To %s 0x%" PRIx64 " port %d lid %u-%u \"%s\"\n",
> >  			(node->type <= IB_NODE_MAX ? node_type_str[node->type] : "???"),
> >  			node->nodeguid, node->ports->portnum, node->ports->lid,
> >  			node->ports->lid + (1 << node->ports->lmc) - 1,
> _______________________________________________
> general mailing list
> general at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
> 
> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general