Re: [PATCH 06/16] IB uverbs: memory pinning implementation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Roland Dreier <[email protected]> wrote:
>
> Add support for pinning userspace memory regions and returning a list
> of pages in the region.  This includes tracking pinned memory against
> vm_locked and preventing unprivileged users from exceeding RLIMIT_MEMLOCK.
> 

Can you tell us a bit more about the design ideas here?  What's it doing,
how and why?

We should look at these things and also decide whether some of this should
live in mm/*.

> +int ib_umem_get(struct ib_device *dev, struct ib_umem *mem,
> +		void *addr, size_t size, int write)
> +{
> ...
> +	if (!can_do_mlock())
> +		return -EPERM;
> +
> ...
> +	if ((locked > lock_limit) && !capable(CAP_IPC_LOCK)) {

The capable() test is redundant.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux