[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090611.053830.39176651.davem@davemloft.net>
Date: Thu, 11 Jun 2009 05:38:30 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: dhollis@...ehollis.com
Cc: david-b@...bell.net, netdev@...r.kernel.org, mark@...x.com.tw,
donald@...x.com.tw, louis@...x.com.tw
Subject: Re: [RFC usbnet.c] - Add packet aggregation capability
From: David Hollis <dhollis@...ehollis.com>
Date: Wed, 10 Jun 2009 13:57:39 -0400
> One item that I have is how to handle the portion in usbnet_probe() that
> sets the hard_start_xmit handler with the change to netdevice_ops:
>
> - net->hard_start_xmit = usbnet_start_xmit;
> + if (info->tx_gather)
> + net->hard_start_xmit = usbnet_bundle_xmit;
> + else
> + net->hard_start_xmit = usbnet_start_xmit;
>
>
> Would it be the most appropriate to merge the
> usbnet_aggregate_skb_xmit() pieces into the existing usbnet_start_xmit
> with appropriate conditionals?
In the current tree you cannot even make this assignment.
Everything must go through a net_device_ops set of methods,
the pointer of which is const.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists