Re: Converting kernel printf() to aprint_*() or log()

[Izaak <yitzhak%inbox.ru@localhost>]

On Tue, Nov 11, 2014 at 06:35:48PM -0500, Christos Zoulas wrote:
> On Nov 11, 11:11pm, martin%duskware.de@localhost (Martin Husemann) wrote:
> -- Subject: Re: Converting kernel printf() to aprint_*() or log()
> 
> | That sounds like a good plan, please correct me if I understood
> | wrong:
> | 
> |  - create a log_dev(device_t, const char *fmt, ...) function and use that
> |    in drivers instead of printf() when not in the driver's *_attach()
> |    function (or functions only called from that); use log(9) where
> |    appropriate instead for printfs not using device_xname().
> 
> Yes, it should be:
> 
>     void log_dev(device_t dev, int level, const char *fmt, ...);
> 
> as originally proposed.
> 
> |  - find simple printf-family calls inside driver's *_attach() functions
> |    (or functions only called from those) and convert them to aprint_*.
> 
> Right.
So, is this the final decision then? What about Jared McNeill's comment and
the questions raised by Taylor R Campbell? 

-- 
Izaak