| To: | Jim Keniston <jkenisto@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/4] Net device error logging, revised |
| From: | Greg KH <greg@xxxxxxxxx> |
| Date: | Tue, 26 Aug 2003 11:32:21 -0700 |
| Cc: | LKML <linux-kernel@xxxxxxxxxxxxxxx>, netdev <netdev@xxxxxxxxxxx>, Jeff Garzik <jgarzik@xxxxxxxxx>, "Feldman, Scott" <scott.feldman@xxxxxxxxx>, Larry Kessler <kessler@xxxxxxxxxx>, Randy Dunlap <rddunlap@xxxxxxxx>, Alan Cox <alan@xxxxxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx> |
| In-reply-to: | <3F4A8027.6FE3F594@us.ibm.com> |
| References: | <3F4A8027.6FE3F594@us.ibm.com> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.4.1i |
On Mon, Aug 25, 2003 at 02:31:19PM -0700, Jim Keniston wrote:
> +int __netdev_printk(const char *sevlevel, const struct net_device *netdev,
> + int msglevel, const char *format, ...)
> +{
> + if (!netdev || !format) {
> + return -EINVAL;
> + }
> + if (msglevel == NETIF_MSG_ALL || (netdev->msg_enable & msglevel)) {
> + char msg[512];
512 bytes on the stack? Any way to prevent this from happening? With
the push to make the stack even smaller in 2.7, people will not like
this.
thanks,
greg k-h
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [bk patches] net driver updates, Jeff Garzik |
|---|---|
| Next by Date: | Re: [OOPS] less /proc/net/igmp, Owen Ford |
| Previous by Thread: | Subject: [PATCH 3/4] Net device error logging, revised (e1000), Jim Keniston |
| Next by Thread: | Re: [PATCH 1/4] Net device error logging, revised, Jim Keniston |
| Indexes: | [Date] [Thread] [Top] [All Lists] |