Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/dev/ic



On Sat, 19 Apr 2008, Joerg Sonnenberger wrote:
> Ignore the error counting part for a moment. What is wrong with 
> aprint_debug
> aprint_verbose
> aprint_normal
> aprint_error
> in decreasing order of verbosity?

These functions were intended for messages about device attachment
and detachment.  Using them for errors like "failed to initialise the
device" is perfectly fine.  But using them for errors like "a packet
received from the network had a bad format" seems like a major departure
from the original intention, and I don't think it's appropriate for boot
flags (like RB_VERBOSE, RB_QUIET, etc.) to have any influence at all on
such messages.

--apb (Alan Barrett)


Home | Main Index | Thread Index | Old Index