lkml.org 
[lkml]   [2019]   [Aug]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v2 03/14] PCI/P2PDMA: Add constants for not-supported result upstream_bridge_distance()


    On 2019-08-06 11:54 p.m., Christoph Hellwig wrote:
    > On Tue, Jul 30, 2019 at 10:35:34AM -0600, Logan Gunthorpe wrote:
    >> Add constant flags to indicate two devices are not supported or whether
    >> the data path goes through the host bridge instead of using the negative
    >> values -1 and -2.
    >>
    >> This helps annotate the code better, but the main reason is so we
    >> can use the information to store the required mapping method in an
    >> xarray.
    >>
    >> Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
    >> Reviewed-by: Christian König <christian.koenig@amd.com>
    >
    > Is there really no way to keep the distance separate from the type of
    > the connection as I requested? I think that would avoid a lot of
    > confusion down the road.

    Well I separated it in the xarray and the interface. It only stores the
    type of mapping, not the distance and uses pci_p2pdma_map_type() to
    retrieve it.

    We only calculate it at the same time as we calculate the distance. This
    is necessary because, to calculate the type, we have to walk the tree
    and check the ACS bits. If we separated it, we'd have to walk the tree
    twice in a very similar way just to determine both the distance and the
    mapping type.

    I'll apply your other feedback to a v3 next week.

    Logan

    \
     
     \ /
      Last update: 2019-08-07 17:59    [W:2.758 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site