[ofa-general] Re: [PATCH] osmtest: Add SA get PathRecord stress test

[Sasha Khapyorsky]

On 09:20 Mon 21 Sep     , Hal Rosenstock wrote:
> <snip...>
> 
> > +
> > +     /*
> > +      * Do a blocking query for the PathRecord.
> > +      */
> > +     status = osmtest_get_path_rec_by_lid_pair(p_osmt, slid, dlid,
> &context);
> > +     if (status != IB_SUCCESS) {
> > +             OSM_LOG(&p_osmt->log, OSM_LOG_ERROR, "ERR 000A: "
> > +                     "osmtest_get_path_rec_by_lid_pair failed (%s)\n",
> > +                     ib_get_err_str(status));
> > +             goto Exit;
> > +     }
> 
> It is not really "stress" testing, just pinging.
> 
> 
> So are the other tests (additionally those use RMPP). Isn't repetitive
> pinging a stress of a kind ?

No. "Stress" test assumes full load. In "ping" case the only one thread
is loaded and in only request processing time.

> 
> 
> > Shouldn't it be clarified in test description?
> 
> 
> Same level of description as other tests. They all could be made more
> descriptive.

Agree. And we need to start somewhere.

Sasha