git: pmap: Warning -> Debug

Aggelos Economopoulos aoiko at cc.ece.ntua.gr
Tue Mar 16 10:47:56 PDT 2010


Matthew Dillon wrote:
> :    pmap: Warning -> Debug
> :    
> :    This message does not concern the system administrator,
> :    try not to get them worried for no reason.
> :
> :Summary of changes:
> : sys/platform/pc32/i386/pmap.c |    2 +-
> 
>     It can just be removed now, it's proven that the case occurs.

That's what I set out to do, but thought I'd play it safe for now.

FWIW, this is not the first time someone on irc asked about those "make
sure this case is actually hit" kprintfs. I think it would be good
practice to label them "Debug:" instead of "Warning:" in the future. The
latter implies that the administrator should do something, which is
definitely not the case.

(I've just noticed I commented on that exact warning when you committed
it a few weeks ago and you suggested changing the message. My memory for
these things is terrible :-)

Aggelos





More information about the Commits mailing list