IB/hfi1: Remove unused struct mmu_rb_ops fields .insert, .invalidate
authorBrendan Cunningham <bcunningham@cornelisnetworks.com>
Fri, 19 May 2023 16:54:25 +0000 (12:54 -0400)
committerJason Gunthorpe <jgg@nvidia.com>
Thu, 1 Jun 2023 17:45:58 +0000 (14:45 -0300)
The struct mmu_rb_ops function pointers .insert, .invalidate were only
used to increment and decrement struct sdma_mmu_node.refcount.

With the deletion of struct sdma_mmu_node.refcount and the addition of
struct mmu_rb_node.refcount  these function pointers are not called and
there are no implementations of them. So it is safe to delete these from
struct mmu_rb_ops.

Link: https://lore.kernel.org/r/168451526508.3702129.8677714753157495310.stgit@awfm-02.cornelisnetworks.com
Signed-off-by: Brendan Cunningham <bcunningham@cornelisnetworks.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@cornelisnetworks.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/hfi1/mmu_rb.h

index 82c505a..751dc3f 100644 (file)
@@ -19,16 +19,11 @@ struct mmu_rb_node {
        struct kref refcount;
 };
 
-/*
- * NOTE: filter, insert, invalidate, and evict must not sleep.  Only remove is
- * allowed to sleep.
- */
+/* filter and evict must not sleep. Only remove is allowed to sleep. */
 struct mmu_rb_ops {
        bool (*filter)(struct mmu_rb_node *node, unsigned long addr,
                       unsigned long len);
-       int (*insert)(void *ops_arg, struct mmu_rb_node *mnode);
        void (*remove)(void *ops_arg, struct mmu_rb_node *mnode);
-       int (*invalidate)(void *ops_arg, struct mmu_rb_node *node);
        int (*evict)(void *ops_arg, struct mmu_rb_node *mnode,
                     void *evict_arg, bool *stop);
 };