From: PJ Waskiewicz <[EMAIL PROTECTED]>
Date: Thu, 28 Jun 2007 09:21:13 -0700
> -struct net_device *alloc_netdev(int sizeof_priv, const char *name,
> - void (*setup)(struct net_device *))
> +struct net_device *alloc_netdev_mq(int sizeof_priv, const char *name,
> + void (*setup)(struct net_device *), int queue_count)
> {
> void *p;
> struct net_device *dev;
> @@ -3557,7 +3564,9 @@ struct net_device *alloc_netdev(int sizeof_priv, const
> char *name,
> BUG_ON(strlen(name) >= sizeof(dev->name));
>
> /* ensure 32-byte alignment of both the device and private area */
> - alloc_size = (sizeof(*dev) + NETDEV_ALIGN_CONST) & ~NETDEV_ALIGN_CONST;
> + alloc_size = (sizeof(*dev) + NETDEV_ALIGN_CONST +
> + (sizeof(struct net_device_subqueue) * queue_count)) &
> + ~NETDEV_ALIGN_CONST;
> alloc_size += sizeof_priv + NETDEV_ALIGN_CONST;
>
> p = kzalloc(alloc_size, GFP_KERNEL);
> @@ -3573,12 +3582,14 @@ struct net_device *alloc_netdev(int sizeof_priv,
> const char *name,
> if (sizeof_priv)
> dev->priv = netdev_priv(dev);
>
> + dev->egress_subqueue_count = queue_count;
> +
> dev->get_stats = internal_stats;
> setup(dev);
> strcpy(dev->name, name);
> return dev;
> }
This isn't going to work.
The pointer returned from netdev_priv() doesn't take into account the
variable sized queues at the end of struct netdev, so we can stomp
over the queues with the private area.
This probably works by luck because of NETDEV_ALIGN.
The simplest fix is to just make netdev_priv() use dev->priv,
except when it's being initialized during allocation, and
that's what I'm going to do when I apply your patch.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
