[ofa-general] Re: [PATCH 20/21] RDS: Kconfig and Makefile
From: Roland Dreier <hidden>
Date: 2009-01-29 05:14:27
From: Roland Dreier <hidden>
Date: 2009-01-29 05:14:27
> So the solution is just to base debug message output on a variable, > instead of a config option? RDS actually does do this a little already, > so converting totally isn't hard. I hadn't seen mention this was > preferable -- indeed, tons of drivers and subsystems have options for > compile-time debug statements, should these be converted? My experience is definitely that compile-time switches are a big pain when you actually have to debug something that can only be reproduced on someone else's setup (which will happen once users start using your stuff). You probably can use the dynamic_printk stuff that went in recently to make this all very clean and standard. - R.