lkml.org 
[lkml]   [2022]   [Jul]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v7 20/21] PCI/P2PDMA: Introduce pci_mmap_p2pmem()
    On Wed, Jun 29, 2022 at 02:59:06PM -0300, Jason Gunthorpe wrote:
    > I've tried in the past, this is not a good idea. There is no way to
    > handle failures when a VMA is dup'd and if you rely on private_data
    > you almost certainly have to alloc here.
    >
    > Then there is the issue of making the locking work on invalidation
    > which is crazy ugly.
    >
    > > I was not a fan of the extra code for this either, but I was given to
    > > understand that it was the standard way to collect and cleanup VMAs.
    >
    > Christoph you tried tried to clean it once globally, what happened to
    > that?

    Al pointed out that there are various places that rely on having a
    separate file system. I might be able to go back to it and see
    if we could at least do it for some users.

    But what also really matters here: I don't want every user that
    wants to be able to mmap a character device to do all this work.
    The layering is simply wrong, it needs some character device
    based helpers, not be open code everywhere.

    In fact I'm not even sure this should be a character device, it seems
    to fit it way better with the PCI sysfs hierchacy, just like how we
    map MMIO resources, which these are anyway. And once it is on sysfs
    we do have a uniqueue inode and need none of the pseudofs stuff, and
    don't need all the glue code in nvme either.

    \
     
     \ /
      Last update: 2022-07-05 09:54    [W:2.456 / U:0.044 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site