On 12/27/2017 11:52 AM, Jason Gunthorpe wrote: > On Wed, Dec 27, 2017 at 09:27:20AM -0600, Daniel Jurgens wrote: >> On 12/24/2017 4:00 PM, Or Gerlitz wrote: >>> On Sun, Dec 24, 2017 at 2:57 PM, Leon Romanovsky <leon@xxxxxxxxxx> wrote: >>>> From: Daniel Jurgens <danielj@xxxxxxxxxxxx> >>>> >>>> Generate a unique 128bit identifier for each host and pass that value to >>>> firmware in the INIT_HCA command if it reports the sw_owner_id >>>> capo ability. This value is used by FW to determine if functions are in >>>> use by the same host. >>> "capo ability"? >>> >>> did you want to say the same driver instance? b/c multiple instances >>> of the driver can run on a host >> No, we want to set the same SW owner ID set for all devices for that >> host. Not per driver instance. > This commit message is confusing, can you explain it better please? > > I'm guessing the owner_id is used to match individual port's PCI > functions? > > So, any 'host' that can see multiple PCI function and share ports > should set the same ID? Right, ports can only be bound if they have the same software owner ID set. If the ID is not the same the bind command will fail. > What happens when a driver is detached? Eg I rebind the PCI function > to VFIO. Does something reset the ID? Yes, next time INIT_HCA is called the owner ID is set again. > Not entirely sure why this is being done at all, shouldn't the PCI > functions be rather un-ambiguous? It's to prevent binding a device as a slave port if it's passed through to a different VM. > Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html