netdev - Re: [RFC 12/19] RDMA/irdma: Implement device supported verb APIs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20190212222707.GU24692@ziepe.ca>
Date:   Tue, 12 Feb 2019 15:27:07 -0700
From:   Jason Gunthorpe <jgg@...pe.ca>
To:     Shiraz Saleem <shiraz.saleem@...el.com>
Cc:     dledford@...hat.com, davem@...emloft.net,
        linux-rdma@...r.kernel.org, netdev@...r.kernel.org,
        mustafa.ismail@...el.com, jeffrey.t.kirsher@...el.com
Subject: Re: [RFC 12/19] RDMA/irdma: Implement device supported verb APIs

On Tue, Feb 12, 2019 at 03:43:55PM -0600, Shiraz Saleem wrote:

> +/**
> + * irdma_disassociate_ucontext - Disassociate user context
> + * @context: ib user context
> + */
> +static void irdma_disassociate_ucontext(struct ib_ucontext *context)
> +{
> +	struct irdma_ucontext *ucontext = to_ucontext(context);
> +
> +	struct irdma_vma_data *vma_data, *n;
> +	struct vm_area_struct *vma;
> +
> +	irdma_dev_info(&ucontext->iwdev->rf->sc_dev, "called\n");
> +	mutex_lock(&ucontext->vma_list_mutex);
> +	list_for_each_entry_safe(vma_data, n, &ucontext->vma_list, list) {
> +		vma = vma_data->vma;
> +		zap_vma_ptes(vma, vma->vm_start, PAGE_SIZE);
> +
> +		vma->vm_flags &= ~(VM_SHARED | VM_MAYSHARE);
> +		vma->vm_ops = NULL;
> +		list_del(&vma_data->list);
> +		kfree(vma_data);
> +	}
> +	mutex_unlock(&ucontext->vma_list_mutex);
> +}

You need to study all the changes that have been done in the core code
and make sure this driver is using all the latest stuff, I do not want
to review a driver and find it is full of obsolete APIs like this
above.

Jason