[ofa-general] Re: [PATCH 1/2] opensm: Handle trap repress on trap 144 generation

[Sasha Khapyorsky]

On 12:55 Fri 06 Mar     , Hal Rosenstock wrote:
> 
> It could be subsequent patch but thought it best to include it. Do you
> want this separate ?

Without clear usage case I would prefer to not have it at all - finally
it is on a fast path.

> 
> > BR is not used anywhere in OpenSM.
> 
> No, but someone might use ib_types.h to build BM.

We cannot know what will be needed then - this someone will need to care
anyway.

> > And this function which should process TrapRepress method does nothing,
> > right?
> 
> Just some validation; It doesn't need to do anything (just retire the
> transaction).

Then I don't understand - how is trap 144 repress handled? And why
those changes in trap_rcv_process_response() were needed?

Sasha