Commit e036c200 authored by Markus Elfring's avatar Markus Elfring Committed by Doug Ledford

IB/hfi1: Remove intermediate var in hfi1_user_sdma_alloc_queues()

* Pass a product for a call of the function "vmalloc_user" without storing
  it in an intermediate variable.

* Delete the local variable "memsize" which became unnecessary with
  this refactoring.
Signed-off-by: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent 147d84e1
...@@ -376,7 +376,6 @@ int hfi1_user_sdma_alloc_queues(struct hfi1_ctxtdata *uctxt, struct file *fp) ...@@ -376,7 +376,6 @@ int hfi1_user_sdma_alloc_queues(struct hfi1_ctxtdata *uctxt, struct file *fp)
{ {
struct hfi1_filedata *fd; struct hfi1_filedata *fd;
int ret = 0; int ret = 0;
unsigned memsize;
char buf[64]; char buf[64];
struct hfi1_devdata *dd; struct hfi1_devdata *dd;
struct hfi1_user_sdma_comp_q *cq; struct hfi1_user_sdma_comp_q *cq;
...@@ -444,8 +443,8 @@ int hfi1_user_sdma_alloc_queues(struct hfi1_ctxtdata *uctxt, struct file *fp) ...@@ -444,8 +443,8 @@ int hfi1_user_sdma_alloc_queues(struct hfi1_ctxtdata *uctxt, struct file *fp)
if (!cq) if (!cq)
goto cq_nomem; goto cq_nomem;
memsize = PAGE_ALIGN(sizeof(*cq->comps) * hfi1_sdma_comp_ring_size); cq->comps = vmalloc_user(PAGE_ALIGN(sizeof(*cq->comps)
cq->comps = vmalloc_user(memsize); * hfi1_sdma_comp_ring_size));
if (!cq->comps) if (!cq->comps)
goto cq_comps_nomem; goto cq_comps_nomem;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment