netdev
[Top] [All Lists]

RE: [PATCH] Net device error logging, revised

To: "Jim Keniston" <jkenisto@xxxxxxxxxx>, "LKML" <linux-kernel@xxxxxxxxxxxxxxx>, "netdev" <netdev@xxxxxxxxxxx>, "Jeff Garzik" <jgarzik@xxxxxxxxx>, "Larry Kessler" <kessler@xxxxxxxxxx>, "Greg KH" <greg@xxxxxxxxx>, "Randy Dunlap" <rddunlap@xxxxxxxx>, "Alan Cox" <alan@xxxxxxxxxxxxxxxxxxx>, "Andrew Morton" <akpm@xxxxxxxx>
Subject: RE: [PATCH] Net device error logging, revised
From: "Feldman, Scott" <scott.feldman@xxxxxxxxx>
Date: Tue, 16 Sep 2003 19:06:01 -0700
Cc: "David Brownell" <david-b@xxxxxxxxxxx>, "Stephen Hemminger" <shemminger@xxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
Thread-index: AcN73qqL6xcF06HFQk+v7BYiIraw8AA4GNpg
Thread-topic: [PATCH] Net device error logging, revised
> 3. A new macro, netdev_fatal, is included.  Given the call
>         netdev_fatal(dev, HW, "NIC fried!\n");
> the indicated message is always logged: the msglevel arg (HW, in this
> case) is NOT consulted.  In fact, the msglevel arg to netdev_fatal
> is ignored in this implementation.  (As previously discussed, in some
> future implementation, the msglevel could be logged to help indicate
> the circumstances under which the event was logged.)

I couldn't find the previous discussion on netdev_fatal, so sorry if
this has already been worked out.  It uses KERN_ERR; did you mean
something stronger?  If not, why not just use netdev_err(dev, ALL,
"...")?  What is the situation in the driver where we'd want to use
_fatal?  How do I know when to use _fatal and when to use _err?

-scott


<Prev in Thread] Current Thread [Next in Thread>