Hey, Thanks for the review. I'll make the fixes for the next version. On 30/08/18 06:34 PM, Randy Dunlap wrote: >> +With the client list in hand, the orchestrator may then call> +:c:func:`pci_p2pmem_find()` to obtain a published P2P memory provider >> +that is supported (behind the same root port) as all the clients. If more >> +than one provider is supported, the one nearest to all the clients will >> +be chosen first. If there are more than one provider is an equal distance >> +away, the one returned will be chosen at random. This function returns the PCI > > random or just arbitrarily? Randomly. See pci_p2pmem_find() in patch 1. We use prandom_u32_max() to select any of the supported devices. Logan