[openib-general] Re: [PATCHv3] ucm: Only print messages when debug level set

Libor Michalek limichal at cisco.com
Wed Jul 20 11:17:23 PDT 2005


On Wed, Jul 20, 2005 at 06:47:31AM -0400, Hal Rosenstock wrote:
> On Wed, 2005-07-20 at 03:35, Michael S. Tsirkin wrote:
> > Quoting r. Hal Rosenstock <halr at voltaire.com>:
> > > On Tue, 2005-07-19 at 20:14, Libor Michalek wrote:
> > > > On Wed, Jul 13, 2005 at 07:40:07PM -0400, Hal Rosenstock wrote:
> > > > > ucm: Only print messages when debug level set
> > > > 
> > > >   I don't think you can assume that ucm_debug_level will always be 
> > > > initialized correctly, or is module_param_named clearing it? Otherwise
> > > > looks fine...
> > > 
> > > Same is true for IPoIB so it would need changing in both places if so.
> > 
> > I always thought that in C global storage is zero-initialized.
> > What am I missing?
>  
> "A static object not explicitly initialized is initialized as if it (or
> its members) were assigned the constant 0."
> 
> So this patch is OK to commit (and IPoIB is fine in terms of this).

  Got it, applied and checked in.

-Libor



More information about the general mailing list