netdev - Re: [net-next PATCH v3] igbvf: add new driver to support 82576 virtual functions

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20090326023343.GA8138@yzhao-otc.sh.intel.com>
Date:	Thu, 26 Mar 2009 10:33:43 +0800
From:	Yu Zhao <yu.zhao@...el.com>
To:	"shemminger@...tta.com" <shemminger@...tta.com>,
	Alexander Duyck <alexander.h.duyck@...el.com>
Cc:	David Miller <davem@...emloft.net>,
	"shemminger@...tta.com" <shemminger@...tta.com>,
	"Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"gospo@...hat.com" <gospo@...hat.com>
Subject: Re: [net-next PATCH v3] igbvf: add new driver to support 82576
	virtual functions

On Thu, Mar 26, 2009 at 08:54:55AM +0800, Alexander Duyck wrote:
> David Miller wrote:
> > From: Alexander Duyck <alexander.h.duyck@...el.com>
> > Date: Wed, 25 Mar 2009 15:33:31 -0700
> > 
> >> The sysfs part of this is already in net-next as it isn't part of
> >> the igbvf driver it is part of igb.  It was added in over a month
> >> ago: http://patchwork.ozlabs.org/patch/23471/
> >>
> >> I will see what we can do to make use of a netlink interface.  I'm
> >> honestly not even sure if it will work well for this kind of setup
> >> since the VF driver won't even be running on the same OS in most
> >> cases.  For now the sysfs code in igb is actually quite minimal, and
> >> the main goal was to keep the interface as simple as possible which
> >> is what I have done.
> > 
> > I have to agree with Stephen that the way to create new links
> > of virtual and similar devices is to use rtnl_link.
> > 
> > We have very good infrastructure for this, and if I understand
> > things correctly iproute2 might even work with an igb using
> > rtnl_link with zero modifications to the tool sources.
> > 
> > Please fix this up, it's a very reasonable request for such
> > a large new piece of driver infrastructure.  And since this is
> > a new driver, you have no time constraints for submission either.
> > 
> 
> Thanks for allowing me more time, but the issue isn't the igbvf driver. 
>   The part that Stephen is referencing will be an issue in the igb 
> driver.  That is why I pointed him to 
> http://patchwork.ozlabs.org/patch/23471/ since it is the part that 
> contains the code that he has issues with.
> 
> Since the issue isn't the igbvf driver there is no reason for it to be 
> held up.  If you want to revert the offending patch feel free to and 
> what I will do is send a patch out that just statically set the number 
> of VFs to 7 if the kernel and device supports enabling SR-IOV.  I didn't 
> want to take that approach since it means that it disables multi-queue 
> for igb on those devices but I feel like I have been backed into a 
> corner on this and don't have many other options.

We do have some powerful tools for network device configuration, but they
still can't cover the multi queue and SR-IOV NICs (also David Woodhouse's
Solos DSL, etc.) configuration requirement. Unless someone improves the
existing tools to support configuring these new hardware features, people
would keep trying to use the sysfs which is the easiest way so far.

Thanks,
Yu
--
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