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
| ||
|
Date: Thu, 20 Jun 2013 20:54:45 +0100 From: Ben Hutchings <bhutchings@...arflare.com> To: "Nelson, Shannon" <shannon.nelson@...el.com> CC: "Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>, "davem@...emloft.net" <davem@...emloft.net>, "Brandeburg, Jesse" <jesse.brandeburg@...el.com>, "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "gospo@...hat.com" <gospo@...hat.com>, "sassmann@...hat.com" <sassmann@...hat.com>, "Waskiewicz Jr, Peter P" <peter.p.waskiewicz.jr@...el.com>, "e1000-devel@...ts.sourceforge.net" <e1000-devel@...ts.sourceforge.net> Subject: Re: [net-next 1/8] i40e: main driver core On Thu, 2013-06-20 at 19:46 +0000, Nelson, Shannon wrote: > > From: Ben Hutchings [mailto:bhutchings@...arflare.com] > > Sent: Wednesday, June 19, 2013 11:13 AM > > > > On Thu, 2013-06-13 at 20:55 -0700, Jeff Kirsher wrote: > > > From: Jesse Brandeburg <jesse.brandeburg@...el.com> [...] > > > +/** > > > + * i40e_get_vsi_stats_struct - Get System Network Statistics > > > + * @vsi: the VSI we care about > > > + * > > > + * Returns the address of the device statistics structure. > > > + * The statistics are actually updated from the service task. > > > + **/ > > > +struct net_device_stats *i40e_get_vsi_stats_struct(struct i40e_vsi > > *vsi) > > > +{ > > > + /* only return the current stats */ > > > + /* It is possible for a VSIs to not have a netdev */ > > > + if (vsi->netdev) > > > + return &vsi->netdev->stats; > > > + else > > > + return &vsi->net_stats; > > > +} > > > > You should be using rtnl_device_stats64; at 40G a 32-bit counter is > > ridiculous (using this in a 32-bit machine is a bit ridiculous too, > > though...) > > You mean rtnl_link_stats64? Sure. Right, yes. > Yes, this driver is really not meant for a 32-bit machine. We've > discussed making sure it would only build for 64-bit, but hadn't quite > gotten around to implementing the restriction. It seems the right > thing to do - would anyone in the community get bent out of shape if > we added that? I'm afraid they probably would. [...] > > > +/** > > > + * i40e_config_rss - Prepare for RSS if used > > > + * @pf: board private structure > > > + **/ > > > +static s32 i40e_config_rss(struct i40e_pf *pf) > > > +{ > > > + struct i40e_hw *hw = &pf->hw; > > > + u32 lut = 0; > > > + int i, j; > > > + u64 hena; > > > + /* Set of random keys generated using kernel random number > > generator */ > > > + static const u32 seed[I40E_PFQF_HKEY_MAX_INDEX + 1] = {0x41b01687, > > > + 0x183cfd8c, 0xce880440, 0x580cbc3c, 0x35897377, > > > + 0x328b25e1, 0x4fa98922, 0xb7d90c14, 0xd5bad70d, > > > + 0xcd15a2c1, 0xe8580225, 0x4a1e9d11, 0xfe5731be}; > > > > Chosen by a fair dice roll? > > Well, as fair as /dev/random gets. [...] My point is: once you turn a random choice into a constant, it's no longer random. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- 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