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

Hal Rosenstock halr at voltaire.com
Wed Jul 20 03:47:31 PDT 2005


On Wed, 2005-07-20 at 03:35, Michael S. Tsirkin wrote:
> Quoting r. Hal Rosenstock <halr at voltaire.com>:
> > Subject: Re: [PATCHv3] ucm: Only print messages when debug level set
> > 
> > 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.
> > 
> > -- Hal
> 
> I always thought that in C global storage is zero-initialized.
> What am I missing?

Right:
 
"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).

-- Hal




More information about the general mailing list