netdev
[Top] [All Lists]

Re: [PATCH 1/4] Net device error logging, revised

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@xxxxxxxxxx>
References: <3F4A8027.6FE3F594@xxxxxxxxxx>
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>