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

Hal Rosenstock halr at voltaire.com
Wed Sep 20 05:38:18 PDT 2006


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
:-(

-- Hal





More information about the general mailing list