On Fri, Mar 20, 2009 at 10:14 AM, Waskiewicz Jr, Peter P <[email protected]> wrote: > On Fri, 20 Mar 2009, Stephen Hemminger wrote: > >> On Fri, 20 Mar 2009 00:23:39 -0700 (Pacific Daylight Time) >> "Waskiewicz Jr, Peter P" <[email protected]> wrote: >> >> > On Thu, 19 Mar 2009, Stephen Hemminger wrote: >> > >> > > Convert ixgbe to use net_device_ops properly. >> > > Rather than changing the select_queue function pointer >> > > just check the flag. >> > > >> > > Signed-off-by: Stephen Hemminger <[email protected]> >> > >> > Thanks Stephen. I was looking at reassigning a DCB netdev_ops struct when >> > DCB is enabled, and then having a default netdev_ops struct when it's not >> > enabled. I agree the check is cleaner the way you have it below, but it's >> > another conditional check in the Tx hotpath, which we have too many of in >> > the first place. >> >> Changing number of tx queues is actually the fastest, since then >> indirection is not needed >> >> > On a related side note, why is the netdev_ops member of net_device >> > declared const? >> >> The purpose of having an ops structure is two fold. First, the ops >> can be in read-only section (if driver wants) to avoid cache issues. >> More importantly only one instance is necessary when there are multiple >> boards, or 1000's of vlans. > > Ah, gotcha. Clears up my questions. > > Acked-by: Peter P Waskiewicz Jr <[email protected]> > --
Acked-by: Jeff Kirsher <[email protected]> Dave - feel free to apply this patch when you take the first patch in this series, I am not adding this to me tree. -- Cheers, Jeff ------------------------------------------------------------------------------ Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com _______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel
