Commit 0ed9a4a0 authored by Bryan O'Sullivan's avatar Bryan O'Sullivan Committed by Linus Torvalds

[PATCH] IB/ipath: use more appropriate gfp flags

This helps us to survive better when memory is fragmented.
Signed-off-by: default avatarDave Olson <dave.olson@qlogic.com>
Signed-off-by: default avatarBryan O'Sullivan <bryan.osullivan@qlogic.com>
Cc: "Michael S. Tsirkin" <mst@mellanox.co.il>
Cc: Roland Dreier <rolandd@cisco.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent a40f55fc
...@@ -705,6 +705,15 @@ static int ipath_create_user_egr(struct ipath_portdata *pd) ...@@ -705,6 +705,15 @@ static int ipath_create_user_egr(struct ipath_portdata *pd)
unsigned e, egrcnt, alloced, egrperchunk, chunk, egrsize, egroff; unsigned e, egrcnt, alloced, egrperchunk, chunk, egrsize, egroff;
size_t size; size_t size;
int ret; int ret;
gfp_t gfp_flags;
/*
* GFP_USER, but without GFP_FS, so buffer cache can be
* coalesced (we hope); otherwise, even at order 4,
* heavy filesystem activity makes these fail, and we can
* use compound pages.
*/
gfp_flags = __GFP_WAIT | __GFP_IO | __GFP_COMP;
egrcnt = dd->ipath_rcvegrcnt; egrcnt = dd->ipath_rcvegrcnt;
/* TID number offset for this port */ /* TID number offset for this port */
...@@ -721,10 +730,8 @@ static int ipath_create_user_egr(struct ipath_portdata *pd) ...@@ -721,10 +730,8 @@ static int ipath_create_user_egr(struct ipath_portdata *pd)
* memory pressure (creating large files and then copying them over * memory pressure (creating large files and then copying them over
* NFS while doing lots of MPI jobs), we hit some allocation * NFS while doing lots of MPI jobs), we hit some allocation
* failures, even though we can sleep... (2.6.10) Still get * failures, even though we can sleep... (2.6.10) Still get
* failures at 64K. 32K is the lowest we can go without waiting * failures at 64K. 32K is the lowest we can go without wasting
* more memory again. It seems likely that the coalescing in * additional memory.
* free_pages, etc. still has issues (as it has had previously
* during 2.6.x development).
*/ */
size = 0x8000; size = 0x8000;
alloced = ALIGN(egrsize * egrcnt, size); alloced = ALIGN(egrsize * egrcnt, size);
...@@ -745,12 +752,6 @@ static int ipath_create_user_egr(struct ipath_portdata *pd) ...@@ -745,12 +752,6 @@ static int ipath_create_user_egr(struct ipath_portdata *pd)
goto bail_rcvegrbuf; goto bail_rcvegrbuf;
} }
for (e = 0; e < pd->port_rcvegrbuf_chunks; e++) { for (e = 0; e < pd->port_rcvegrbuf_chunks; e++) {
/*
* GFP_USER, but without GFP_FS, so buffer cache can be
* coalesced (we hope); otherwise, even at order 4,
* heavy filesystem activity makes these fail
*/
gfp_t gfp_flags = __GFP_WAIT | __GFP_IO | __GFP_COMP;
pd->port_rcvegrbuf[e] = dma_alloc_coherent( pd->port_rcvegrbuf[e] = dma_alloc_coherent(
&dd->pcidev->dev, size, &pd->port_rcvegrbuf_phys[e], &dd->pcidev->dev, size, &pd->port_rcvegrbuf_phys[e],
...@@ -1167,9 +1168,10 @@ static int ipath_mmap(struct file *fp, struct vm_area_struct *vma) ...@@ -1167,9 +1168,10 @@ static int ipath_mmap(struct file *fp, struct vm_area_struct *vma)
ureg = dd->ipath_uregbase + dd->ipath_palign * pd->port_port; ureg = dd->ipath_uregbase + dd->ipath_palign * pd->port_port;
ipath_cdbg(MM, "ushare: pgaddr %llx vm_start=%lx, vmlen %lx\n", ipath_cdbg(MM, "pgaddr %llx vm_start=%lx len %lx port %u:%u\n",
(unsigned long long) pgaddr, vma->vm_start, (unsigned long long) pgaddr, vma->vm_start,
vma->vm_end - vma->vm_start); vma->vm_end - vma->vm_start, dd->ipath_unit,
pd->port_port);
if (pgaddr == ureg) if (pgaddr == ureg)
ret = mmap_ureg(vma, dd, ureg); ret = mmap_ureg(vma, dd, ureg);
......
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