[openib-general] Re: [PATCH] agent: Add IB ping server agent

Roland Dreier roland at topspin.com
Thu Mar 17 10:43:10 PST 2005


    Michael> I think a compile option is simpler, and would be sufficient.

    Hal> I agree. Is this an acceptable approach ?

I think I'd prefer to avoid having this #ifdef'ed.  If we have OpenIB
vendor class handling nicely encapsulated, it's nice and maintainable,
and it makes it clear how vendor classes should be handled.  If we set
the precedent for handling for vendor classes in the core mad files,
then we're taking the first step on the road to insanity.

 - R.



More information about the general mailing list