[ofa-general] [infiniband-diags] add --loop_ports option to perfquery

[Hal Rosenstock]

Al,

On Tue, Oct 7, 2008 at 6:54 PM, Al Chu <chu11 at llnl.gov> wrote:
> Hey Sasha,
>
> We have a switch here that does not report the AllPortSelect flag as a
> capability.  It's pretty annoying typing each port on the switch or
> always having to script around this one oddball switch we have.  So I
> added an option --loop_ports for perfquery.  If you want to do something
> to all the ports on the CA/Switch, but AllPortSelect isn't available, it
> loops through all the available ports instead.

Why not add simulated AllPortSelect for multiple ports rather than add
another perquery option for this ?

> There was already a workaround in the tool for a CA that did not support
> the AllPortSelect flag.  I get the feeling the workaround may have been
> for a specific hardware, so I kept the workaround in there.

> Al
>
> --
> Albert Chu
> chu11 at llnl.gov
> Computer Scientist
> High Performance Systems Division
> Lawrence Livermore National Laboratory
>
> _______________________________________________
> 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
>

There are also 2 for loops which are not correct for some switches:
for (i = 1; i <= num_ports; i++)

-- Hal