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

[Hal Rosenstock]

On Mon, Sep 21, 2009 at 10:28 AM, Sasha Khapyorsky <sashak at voltaire.com>wrote:

> 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.


What do you mean full load ?


> In "ping" case the only one thread
> is loaded and in only request processing time.
>

I'm not following what you mean by this.


>
> >
> >
> > > 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.
>

Separate patch ?

-- Hal


>
> Sasha
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/general/attachments/20090921/ccc003b0/attachment.html>