netdev - Re: [PATCH 00/33] Network Devices Ops (v0.3)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20081119.140444.179873784.davem@davemloft.net>
Date:	Wed, 19 Nov 2008 14:04:44 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	jeffrey.t.kirsher@...el.com
Cc:	shemminger@...tta.com, netdev@...r.kernel.org
Subject: Re: [PATCH 00/33] Network Devices Ops (v0.3)

From: "Jeff Kirsher" <jeffrey.t.kirsher@...el.com>
Date: Wed, 19 Nov 2008 13:55:12 -0800

> On Mon, Nov 17, 2008 at 5:35 PM, Stephen Hemminger
> <shemminger@...tta.com> wrote:
> > On Mon, 17 Nov 2008 17:27:05 -0800
> > "Jeff Kirsher" <jeffrey.t.kirsher@...el.com> wrote:
> >
> >> On Mon, Nov 17, 2008 at 3:42 PM, Stephen Hemminger
> >> <shemminger@...tta.com> wrote:
> >> > This sequence of patches breaks the network device adminstration
> >> > function table away from the network device data structure. It is
> >> > a transitional API change (both keep working), and this set of patches
> >> > hits a range of commonly used drivers. This should be enough to make sure
> >> > both old and new interfaces get used and tested.
> >> >
> >> >
> >>
> >> Did I miss a patch, or did you intentionally not convert ixgbe?
> >
> > Since it was so similar to ixgb, both files are patched in same patch bundle.
> > --
> >
> 
> Looking at this further, what is the benefit of this change?  For this
> massive of a change, I would expect some sort of significant benefit.
> 
> What is the driving force behind this change?

If you have 1,000 virtual MAC vlan devices, you have 1,000 copies of
these ops pointers being accessed by the CPU, that's stupid.

It's just one of many examples.
--
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