Re: [PATCH RFC 0/3] Support standard SRIOV configuration for IB VFs

2015-05-25 Thread Jason Gunthorpe
On Tue, May 26, 2015 at 06:33:15AM +0300, Or Gerlitz wrote: > > This allows generic code to work with the LLADDR - for instance 'ip > > link set vf mac' should have checked the size of the IFLA_ADDRESS and > > demanded that the address argument is the same number of bytes. It is > > very broken the

Re: [PATCH RFC 0/3] Support standard SRIOV configuration for IB VFs

2015-05-25 Thread Or Gerlitz
On Tue, May 26, 2015 at 1:32 AM, Jason Gunthorpe wrote: > On Tue, May 26, 2015 at 12:50:52AM +0300, Or Gerlitz wrote: >> On Tue, May 26, 2015 at 12:14 AM, Jason Gunthorpe >> wrote: >> > On Mon, May 25, 2015 at 11:04:41PM +0300, Or Gerlitz wrote: >> > >> >> OK, so rewinding a bit, the IB VF [1] id

Re: [PATCH RFC 0/3] Support standard SRIOV configuration for IB VFs

2015-05-25 Thread Jason Gunthorpe
On Tue, May 26, 2015 at 12:50:52AM +0300, Or Gerlitz wrote: > On Tue, May 26, 2015 at 12:14 AM, Jason Gunthorpe > wrote: > > On Mon, May 25, 2015 at 11:04:41PM +0300, Or Gerlitz wrote: > > > >> OK, so rewinding a bit, the IB VF [1] identity is their 8 bytes port > >> GUID, and as Jason noted the u

Re: [PATCH RFC 0/3] Support standard SRIOV configuration for IB VFs

2015-05-25 Thread Or Gerlitz
On Tue, May 26, 2015 at 12:14 AM, Jason Gunthorpe wrote: > On Mon, May 25, 2015 at 11:04:41PM +0300, Or Gerlitz wrote: > >> OK, so rewinding a bit, the IB VF [1] identity is their 8 bytes port >> GUID, and as Jason noted the user/kernel API allows to deliver up to >> 32 bytes between user and kern

Re: [PATCH RFC 0/3] Support standard SRIOV configuration for IB VFs

2015-05-25 Thread Jason Gunthorpe
On Mon, May 25, 2015 at 11:04:41PM +0300, Or Gerlitz wrote: > OK, so rewinding a bit, the IB VF [1] identity is their 8 bytes port > GUID, and as Jason noted the user/kernel API allows to deliver up to > 32 bytes between user and kernel under the set_vf_mac flow > (do_setvfinfo() in net/core/rtnet

Re: [PATCH RFC 0/3] Support standard SRIOV configuration for IB VFs

2015-05-25 Thread Or Gerlitz
On Fri, May 22, 2015 at 12:11 AM, Doug Ledford wrote: > On Thu, 2015-05-21 at 22:55 +0300, Or Gerlitz wrote: >> On Thu, May 21, 2015 at 7:40 PM, Doug Ledford wrote: >> >> > The MAC/GUID mapping isn't the only thing that has to be faked here >> >> Exactly nothing is faked here, Virtualization syst

Re: [PATCH V3 for-next 1/3] IB/uverbs: Enable device removal when there are active user space applications

2015-05-25 Thread Jason Gunthorpe
On Wed, May 13, 2015 at 02:10:36PM +0300, Yishai Hadas wrote: > + struct srcu_struct disassociate_srcu; There is no need for rcu for this, use a rw sem. > @@ -1326,6 +1327,13 @@ ssize_t ib_uverbs_create_comp_channel(struct > ib_uverbs_file *file, > return

Re: [RESEND PATCH V3 for-next 0/3] HW Device hot-removal support

2015-05-25 Thread Yishai Hadas
On 5/19/2015 7:17 PM, Liran Liss wrote: From: Hefty, Sean [mailto:sean.he...@intel.com] these remaining resources may be device-specific. The proposed framework first of all allows a provider to indicate whether hot-removal is supported (i.e., the presence of the 'disassociate_ucontext' callba