lkml.org 
[lkml]   [2018]   [Aug]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v5 13/13] nvmet: Optionally use PCI P2P memory
Thanks, for the review.

On 30/08/18 06:25 PM, Sagi Grimberg wrote:
>> + if (req->port->p2p_dev) {
>> + if (!pci_p2pdma_assign_provider(req->port->p2p_dev,
>> + &ctrl->p2p_clients)) {
>> + pr_info("peer-to-peer memory on %s is not supported\n",
>> + pci_name(req->port->p2p_dev));
>> + goto free_devices;
>> + }
>> + ctrl->p2p_dev = pci_dev_get(req->port->p2p_dev);
>> + } else {
>
> When is port->p2p_dev == NULL? a little more documentation would help
> here...

In the configfs functions, if the user enables p2p (port->use_p2pmem)
using 'auto' or 'y' then port->p2p_dev will be NULL. If the user sets a
specific p2p_dev to use, port->p2p_dev will be set to that device. I can
add a couple comments in the next version.

Logan

\
 
 \ /
  Last update: 2018-08-31 17:43    [W:0.843 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site