[openib-general] [PATCH] Combine ib_agent into ib_mad module

Roland Dreier roland at topspin.com
Tue Oct 26 12:35:33 PDT 2004


    Hal> Because with just your changes, the port did not become
    Hal> active. It appeared that there needed to be some
    Hal> synchronization between the MAD layer completing its
    Hal> initialization and the agents completing theirs.  This was
    Hal> the simplest way I could think of. I'm sure there are other
    Hal> solutions too.

That's strange, my tree is working fine for me.  If you do 

	ib_register_client(&mad_client)
	ib_register_client(&agent_client)

then it's guaranteed that the MAD layer is done initializing each
device before the agent layer starts.

Oh well.

    Hal> Also, while your changes were fewer lines of code change,
    Hal> these changes are less lines of code total. Is there an issue
    Hal> with doing it this way ?

I guess the only downside I see is that it ties ib_mad.c and
ib_agent.c together event more tightly (there is agent initialization
code in ib_mad.c, rather than keeping all the agent code in ib_agent.c)

 - Roland



More information about the general mailing list