| To: | Jeff Garzik <jgarzik@xxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2.6.0-test11] Net device error logging |
| From: | Greg KH <greg@xxxxxxxxx> |
| Date: | Sat, 6 Dec 2003 01:04:14 -0800 |
| Cc: | Jim Keniston <jkenisto@xxxxxxxxxx>, LKML <linux-kernel@xxxxxxxxxxxxxxx>, netdev <netdev@xxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx>, "Feldman, Scott" <scott.feldman@xxxxxxxxx>, Larry Kessler <kessler@xxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxx> |
| In-reply-to: | <3FD0E498.8070703@xxxxxxxxx> |
| References: | <3FD0E1FE.1D5B1883@xxxxxxxxxx> <3FD0E498.8070703@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.4.1i |
On Fri, Dec 05, 2003 at 03:03:36PM -0500, Jeff Garzik wrote: > I discussed this a bit with David. My personal feelings are that I > prefer just leaving all the printk's as they are. But Linus and GregKH > have been accepting patches into other parts of the tree like this one, > and logging additional already-computer-parsed information is probably > not a bad thing long-term, so perhaps I've been being a bit of a Luddite > on this issue. To be fair, the patches I've taken (dev_err and friends) are _much_ simpler than these, so accepting them was not that big of a deal. It enabled the subsystems that have started to use them (USB and I2C) to log better messages (we now know exactly which device caused the errors, instead of just which driver). So please judge this patch on its own, and feel no pressure due to the dev_*() calls :) thanks, greg k-h |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [patch] remove com20020-isa.c unused variables, Adrian Bunk |
|---|---|
| Next by Date: | HTB 2.6.0-test10 oops related patch, devik |
| Previous by Thread: | Re: [PATCH 2.6.0-test11] Net device error logging, Jeff Garzik |
| Next by Thread: | [PATCH] IPv6 MIB:ipv6Prefix netlink notification, Shirley Ma |
| Indexes: | [Date] [Thread] [Top] [All Lists] |