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

Michael S. Tsirkin mst at mellanox.co.il
Thu Mar 17 09:48:30 PST 2005


Quoting r. Hal Rosenstock <halr at voltaire.com>:
> Subject: Re: [PATCH] agent: Add IB ping server agent
> 
> On Thu, 2005-03-17 at 12:38, Sean Hefty wrote:
> > Roland Dreier wrote:
> > > I would suggest moving the ping server at least into its own source
> > > file, if not into its own module.  I'm not convinced that we want to
> > > have a vendor-specific MAD handler unconditionally compiled into the
> > > core MAD support.
> > 
> > I agree with this.  This feels like it should be a separate module.
> 
> OK but this will result in some code duplication. I should have this
> later today and will revert back the changes to agent.c and agent_priv.h
> of earlier today.
> 
> -- Hal

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

-- 
MST - Michael S. Tsirkin



More information about the general mailing list