Re: [PATCH V2 08/10] i40iw: Control debug error prints using env variable

Jason Gunthorpe
2016-12-15T18:50:34Z

On Thu, Dec 15, 2016 at 08:35:37PM +0200, Leon Romanovsky wrote:

> I had in mind much simplier infrastucture, just add pr_debug(..) call
> and allow every provider to place in any place in their code.

I think the bitmask thing has to be hoisted too.

> My main point is that I want to see all ENV variables in one place.

Why?

I really don't want to see util/ files include provider headers, for
instance, so I don't like your SET() macro idea..

At this point I'd settle for having all ENV vars *documented* in one
place :(

Jason