mlx4: Remove page_is_free() safety check
authorBarret Rhoden <brho@cs.berkeley.edu>
Thu, 27 Oct 2016 00:11:28 +0000 (20:11 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Tue, 29 Nov 2016 16:27:40 +0000 (11:27 -0500)
With the arena allocator, it won't be easy to query the state of a given
page.  We actually could do that, if we wanted, with an arena helper
that looks up the btag for a given address, but it's a pain, it won't be
fast, and it will probably not work well with NUMA.

Considering this style of page pinning needs to change anyways, we might
as well remove it.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/drivers/net/udrvr/compat.c

index 9c258ef..6bf2aa9 100644 (file)
@@ -55,13 +55,6 @@ int map_upage_at_addr(struct proc *p, physaddr_t paddr, uintptr_t addr, int ptep
                return -1;
        }
 
-       /* Ensure kernel has not put such special pages into free pool */
-       if (page_is_free(paddr >> PAGE_SHIFT)) {
-               printk("[akaros]: map_upage_at_addr(): FreePA=0x%llx\n",
-                   paddr);
-               return -1;
-       }
-
        pp = pa2page(paddr);
 
        /* __vmr_free_pgs() refcnt's pagemap pages differently */