[openib-general] [PATCH] osm: fixing bugs in osmtest

Sasha Khapyorsky sashak at voltaire.com
Wed Sep 20 18:01:08 PDT 2006


On 08:38 Wed 20 Sep     , Hal Rosenstock wrote:
> Hi Yevgeny,
> 
> On Wed, 2006-09-20 at 02:36, Yevgeny Kliteynik wrote:
> > Hi Hal
> > 
> > I'm doing a major review of the osmtest.
> 
> Good. This has been long overdue.
> 
> > This patch is fixing a few bugs in osmtest where failures
> > were ignored. More precisely, osmtest was expecting error,
> > but got IB_SUCCESS and ignored the fact that it should have
> > gotten an error.
> > There are also a few changes to improve the code and osmtest
> > log readability.
> 
> Looks good at the code inspection level.
> 
> > More patches expected.
> 
> Thanks for the heads up.
> 
> > This patch is for trunk only.
> > 
> > I tested applying this patch before sending it. If you get the
> > patch rejected again - let me know.
> 
> It took the header file part but rejected all code blocks for osmtest.c
> :-(

It looks like modified and context lines have different numbers of
prefixed spaces.

Sasha




More information about the general mailing list