summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--linux-core/xgi_fb.c43
-rw-r--r--linux-core/xgi_misc.c39
-rw-r--r--linux-core/xgi_pcie.c75
3 files changed, 45 insertions, 112 deletions
diff --git a/linux-core/xgi_fb.c b/linux-core/xgi_fb.c
index 56cc589b..32fde5ab 100644
--- a/linux-core/xgi_fb.c
+++ b/linux-core/xgi_fb.c
@@ -96,7 +96,6 @@ void xgi_fb_free(struct xgi_info * info, unsigned long bus_addr)
unsigned long offset = bus_addr - info->fb.base;
struct xgi_mem_pid *mempid_block;
struct xgi_mem_pid *mempid_freeblock = NULL;
- struct list_head *mempid_list;
if (offset < 0) {
XGI_INFO("free onscreen frame buffer successfully !\n");
@@ -111,16 +110,12 @@ void xgi_fb_free(struct xgi_info * info, unsigned long bus_addr)
}
/* manage mempid */
- mempid_list = xgi_mempid_list.next;
- while (mempid_list != &xgi_mempid_list) {
- mempid_block =
- list_entry(mempid_list, struct xgi_mem_pid, list);
+ list_for_each_entry(mempid_block, &xgi_mempid_list, list) {
if (mempid_block->location == LOCAL
&& mempid_block->bus_addr == bus_addr) {
mempid_freeblock = mempid_block;
break;
}
- mempid_list = mempid_list->next;
}
if (mempid_freeblock) {
list_del(&mempid_freeblock->list);
@@ -192,20 +187,15 @@ int xgi_fb_heap_init(struct xgi_info * info)
void xgi_fb_heap_cleanup(struct xgi_info * info)
{
- struct list_head *free_list, *temp;
+ struct list_head *free_list;
struct xgi_mem_block *block;
+ struct xgi_mem_block *next;
int i;
if (xgi_fb_heap) {
free_list = &xgi_fb_heap->free_list;
for (i = 0; i < 3; i++, free_list++) {
- temp = free_list->next;
- while (temp != free_list) {
- block =
- list_entry(temp, struct xgi_mem_block,
- list);
- temp = temp->next;
-
+ list_for_each_entry_safe(block, next, free_list, list) {
XGI_INFO
("No. %d block->offset: 0x%lx block->size: 0x%lx \n",
i, block->offset, block->size);
@@ -334,7 +324,6 @@ static void xgi_mem_delete_node(struct xgi_mem_list * list, struct xgi_mem_block
static struct xgi_mem_block *xgi_mem_alloc(struct xgi_info * info,
unsigned long originalSize)
{
- struct list_head *free_list;
struct xgi_mem_block *block, *free_block, *used_block;
unsigned long size = (originalSize + PAGE_SIZE - 1) & PAGE_MASK;
@@ -354,18 +343,14 @@ static struct xgi_mem_block *xgi_mem_alloc(struct xgi_info * info,
return (NULL);
}
- free_list = xgi_fb_heap->free_list.next;
-
- while (free_list != &xgi_fb_heap->free_list) {
+ list_for_each_entry(block, &xgi_fb_heap->free_list, list) {
XGI_INFO("free_list: 0x%px \n", free_list);
- block = list_entry(free_list, struct xgi_mem_block, list);
if (size <= block->size) {
break;
}
- free_list = free_list->next;
}
- if (free_list == &xgi_fb_heap->free_list) {
+ if (&block->list == &xgi_fb_heap->free_list) {
XGI_ERROR
("Can't allocate %ldk size from frame buffer memory !\n",
size / 1024);
@@ -408,23 +393,19 @@ static struct xgi_mem_block *xgi_mem_alloc(struct xgi_info * info,
static struct xgi_mem_block *xgi_mem_free(struct xgi_info * info, unsigned long offset)
{
- struct list_head *free_list, *used_list;
- struct xgi_mem_block *used_block = NULL, *block = NULL;
+ struct xgi_mem_block *used_block = NULL, *block;
struct xgi_mem_block *prev, *next;
unsigned long upper;
unsigned long lower;
- used_list = xgi_fb_heap->used_list.next;
- while (used_list != &xgi_fb_heap->used_list) {
- block = list_entry(used_list, struct xgi_mem_block, list);
+ list_for_each_entry(block, &xgi_fb_heap->used_list, list) {
if (block->offset == offset) {
break;
}
- used_list = used_list->next;
}
- if (used_list == &xgi_fb_heap->used_list) {
+ if (&block->list == &xgi_fb_heap->used_list) {
XGI_ERROR("can't find block: 0x%lx to free!\n", offset);
return (NULL);
}
@@ -439,16 +420,12 @@ static struct xgi_mem_block *xgi_mem_free(struct xgi_info * info, unsigned long
upper = used_block->offset + used_block->size;
lower = used_block->offset;
- free_list = xgi_fb_heap->free_list.next;
- while (free_list != &xgi_fb_heap->free_list) {
- block = list_entry(free_list, struct xgi_mem_block, list);
-
+ list_for_each_entry(block, &xgi_fb_heap->free_list, list) {
if (block->offset == upper) {
next = block;
} else if ((block->offset + block->size) == lower) {
prev = block;
}
- free_list = free_list->next;
}
XGI_INFO("next = 0x%p, prev = 0x%p\n", next, prev);
diff --git a/linux-core/xgi_misc.c b/linux-core/xgi_misc.c
index 96ad12ee..eecd717b 100644
--- a/linux-core/xgi_misc.c
+++ b/linux-core/xgi_misc.c
@@ -556,50 +556,45 @@ int xgi_get_cpu_id(struct cpu_info *arg)
extern struct list_head xgi_mempid_list;
void xgi_mem_collect(struct xgi_info * info, unsigned int *pcnt)
{
- struct xgi_mem_pid *mempid_block;
- struct list_head *mempid_list;
+ struct xgi_mem_pid *block;
+ struct xgi_mem_pid *next;
struct task_struct *p, *find;
unsigned int cnt = 0;
- mempid_list = xgi_mempid_list.next;
-
- while (mempid_list != &xgi_mempid_list) {
- mempid_block =
- list_entry(mempid_list, struct xgi_mem_pid, list);
- mempid_list = mempid_list->next;
+ list_for_each_entry_safe(block, next, &xgi_mempid_list, list) {
find = NULL;
XGI_SCAN_PROCESS(p) {
- if (p->pid == mempid_block->pid) {
+ if (p->pid == block->pid) {
XGI_INFO
("[!]Find active pid:%ld state:%ld location:%d addr:0x%lx! \n",
- mempid_block->pid, p->state,
- mempid_block->location,
- mempid_block->bus_addr);
+ block->pid, p->state,
+ block->location,
+ block->bus_addr);
find = p;
- if (mempid_block->bus_addr == 0xFFFFFFFF)
+ if (block->bus_addr == 0xFFFFFFFF)
++cnt;
break;
}
}
if (!find) {
- if (mempid_block->location == LOCAL) {
+ if (block->location == LOCAL) {
XGI_INFO
("Memory ProcessID free fb and delete one block pid:%ld addr:0x%lx successfully! \n",
- mempid_block->pid, mempid_block->bus_addr);
- xgi_fb_free(info, mempid_block->bus_addr);
- } else if (mempid_block->bus_addr != 0xFFFFFFFF) {
+ block->pid, block->bus_addr);
+ xgi_fb_free(info, block->bus_addr);
+ } else if (block->bus_addr != 0xFFFFFFFF) {
XGI_INFO
("Memory ProcessID free pcie and delete one block pid:%ld addr:0x%lx successfully! \n",
- mempid_block->pid, mempid_block->bus_addr);
- xgi_pcie_free(info, mempid_block->bus_addr);
+ block->pid, block->bus_addr);
+ xgi_pcie_free(info, block->bus_addr);
} else {
/*only delete the memory block */
- list_del(&mempid_block->list);
+ list_del(&block->list);
XGI_INFO
("Memory ProcessID delete one pcie block pid:%ld successfully! \n",
- mempid_block->pid);
- kfree(mempid_block);
+ block->pid);
+ kfree(block);
}
}
}
diff --git a/linux-core/xgi_pcie.c b/linux-core/xgi_pcie.c
index dd758013..e451ebd5 100644
--- a/linux-core/xgi_pcie.c
+++ b/linux-core/xgi_pcie.c
@@ -376,8 +376,9 @@ void xgi_pcie_heap_check(void)
void xgi_pcie_heap_cleanup(struct xgi_info * info)
{
- struct list_head *free_list, *temp;
+ struct list_head *free_list;
struct xgi_pcie_block *block;
+ struct xgi_pcie_block *next;
int j;
xgi_pcie_lut_cleanup(info);
@@ -386,23 +387,16 @@ void xgi_pcie_heap_cleanup(struct xgi_info * info)
if (xgi_pcie_heap) {
free_list = &xgi_pcie_heap->free_list;
for (j = 0; j < 3; j++, free_list++) {
- temp = free_list->next;
-
- while (temp != free_list) {
- block =
- list_entry(temp, struct xgi_pcie_block,
- list);
+ list_for_each_entry_safe(block, next, free_list, list) {
XGI_INFO
- ("No. %d block->offset: 0x%lx block->size: 0x%lx \n",
+ ("No. %d block offset: 0x%lx size: 0x%lx\n",
j, block->offset, block->size);
xgi_pcie_block_stuff_free(block);
block->bus_addr = 0;
block->hw_addr = 0;
- temp = temp->next;
//XGI_INFO("No. %d free block: 0x%p \n", j, block);
kmem_cache_free(xgi_pcie_cache_block, block);
- block = NULL;
}
}
@@ -421,7 +415,6 @@ static struct xgi_pcie_block *xgi_pcie_mem_alloc(struct xgi_info * info,
unsigned long originalSize,
enum PcieOwner owner)
{
- struct list_head *free_list;
struct xgi_pcie_block *block, *used_block, *free_block;
struct xgi_page_block *page_block, *prev_page_block;
struct page *page;
@@ -475,17 +468,13 @@ static struct xgi_pcie_block *xgi_pcie_mem_alloc(struct xgi_info * info,
}
/* Jong 05/30/2006; find next free list which has enough space */
- free_list = xgi_pcie_heap->free_list.next;
- while (free_list != &xgi_pcie_heap->free_list) {
- //XGI_INFO("free_list: 0x%px \n", free_list);
- block = list_entry(free_list, struct xgi_pcie_block, list);
+ list_for_each_entry(block, &xgi_pcie_heap->free_list, list) {
if (size <= block->size) {
break;
}
- free_list = free_list->next;
}
- if (free_list == &xgi_pcie_heap->free_list) {
+ if (&block->list == &xgi_pcie_heap->free_list) {
XGI_ERROR("Can't allocate %ldk size from PCIE memory !\n",
size / 1024);
return (NULL);
@@ -696,21 +685,17 @@ static struct xgi_pcie_block *xgi_pcie_mem_alloc(struct xgi_info * info,
static struct xgi_pcie_block *xgi_pcie_mem_free(struct xgi_info * info,
unsigned long offset)
{
- struct list_head *free_list, *used_list;
- struct xgi_pcie_block *used_block, *block = NULL;
+ struct xgi_pcie_block *used_block, *block;
struct xgi_pcie_block *prev, *next;
unsigned long upper, lower;
- used_list = xgi_pcie_heap->used_list.next;
- while (used_list != &xgi_pcie_heap->used_list) {
- block = list_entry(used_list, struct xgi_pcie_block, list);
+ list_for_each_entry(block, &xgi_pcie_heap->used_list, list) {
if (block->offset == offset) {
break;
}
- used_list = used_list->next;
}
- if (used_list == &xgi_pcie_heap->used_list) {
+ if (&block->list == &xgi_pcie_heap->used_list) {
XGI_ERROR("can't find block: 0x%lx to free!\n", offset);
return (NULL);
}
@@ -730,16 +715,12 @@ static struct xgi_pcie_block *xgi_pcie_mem_free(struct xgi_info * info,
upper = used_block->offset + used_block->size;
lower = used_block->offset;
- free_list = xgi_pcie_heap->free_list.next;
-
- while (free_list != &xgi_pcie_heap->free_list) {
- block = list_entry(free_list, struct xgi_pcie_block, list);
+ list_for_each_entry(block, &xgi_pcie_heap->free_list, list) {
if (block->offset == upper) {
next = block;
} else if ((block->offset + block->size) == lower) {
prev = block;
}
- free_list = free_list->next;
}
XGI_INFO("next = 0x%p, prev = 0x%p\n", next, prev);
@@ -839,7 +820,6 @@ void xgi_pcie_free(struct xgi_info * info, unsigned long bus_addr)
unsigned long offset = bus_addr - info->pcie.base;
struct xgi_mem_pid *mempid_block;
struct xgi_mem_pid *mempid_freeblock = NULL;
- struct list_head *mempid_list;
char isvertex = 0;
int processcnt;
@@ -850,15 +830,12 @@ void xgi_pcie_free(struct xgi_info * info, unsigned long bus_addr)
if (isvertex) {
/*check is there any other process using vertex */
processcnt = 0;
- mempid_list = xgi_mempid_list.next;
- while (mempid_list != &xgi_mempid_list) {
- mempid_block =
- list_entry(mempid_list, struct xgi_mem_pid, list);
+
+ list_for_each_entry(mempid_block, &xgi_mempid_list, list) {
if (mempid_block->location == NON_LOCAL
&& mempid_block->bus_addr == 0xFFFFFFFF) {
++processcnt;
}
- mempid_list = mempid_list->next;
}
if (processcnt > 1) {
return;
@@ -877,17 +854,13 @@ void xgi_pcie_free(struct xgi_info * info, unsigned long bus_addr)
xgi_pcie_vertex_block = NULL;
/* manage mempid */
- mempid_list = xgi_mempid_list.next;
- while (mempid_list != &xgi_mempid_list) {
- mempid_block =
- list_entry(mempid_list, struct xgi_mem_pid, list);
+ list_for_each_entry(mempid_block, &xgi_mempid_list, list) {
if (mempid_block->location == NON_LOCAL
&& ((isvertex && mempid_block->bus_addr == 0xFFFFFFFF)
|| (!isvertex && mempid_block->bus_addr == bus_addr))) {
mempid_freeblock = mempid_block;
break;
}
- mempid_list = mempid_list->next;
}
if (mempid_freeblock) {
list_del(&mempid_freeblock->list);
@@ -905,15 +878,11 @@ void xgi_pcie_free(struct xgi_info * info, unsigned long bus_addr)
struct xgi_pcie_block *xgi_find_pcie_block(struct xgi_info * info,
unsigned long address)
{
- struct list_head *used_list;
struct xgi_pcie_block *block;
int i;
- used_list = xgi_pcie_heap->used_list.next;
-
- while (used_list != &xgi_pcie_heap->used_list) {
- block = list_entry(used_list, struct xgi_pcie_block, list);
+ list_for_each_entry(block, &xgi_pcie_heap->used_list, list) {
if (block->bus_addr == address) {
return block;
}
@@ -927,7 +896,6 @@ struct xgi_pcie_block *xgi_find_pcie_block(struct xgi_info * info,
}
}
}
- used_list = used_list->next;
}
XGI_ERROR("could not find map for vm 0x%lx\n", address);
@@ -944,17 +912,13 @@ struct xgi_pcie_block *xgi_find_pcie_block(struct xgi_info * info,
*/
void *xgi_find_pcie_virt(struct xgi_info * info, unsigned long address)
{
- struct list_head *used_list = xgi_pcie_heap->used_list.next;
+ struct xgi_pcie_block *block;
const unsigned long offset_in_page = address & (PAGE_SIZE - 1);
- XGI_INFO("begin (used_list = 0x%p, address = 0x%lx, "
- "PAGE_SIZE - 1 = %lu, offset_in_page = %lu)\n",
- used_list, address, PAGE_SIZE - 1, offset_in_page);
-
- while (used_list != &xgi_pcie_heap->used_list) {
- struct xgi_pcie_block *block =
- list_entry(used_list, struct xgi_pcie_block, list);
+ XGI_INFO("begin (address = 0x%lx, offset_in_page = %lu)\n",
+ address, offset_in_page);
+ list_for_each_entry(block, &xgi_pcie_heap->used_list, list) {
XGI_INFO("block = 0x%p (hw_addr = 0x%lx, size=%lu)\n",
block, block->hw_addr, block->size);
@@ -973,9 +937,6 @@ void *xgi_find_pcie_virt(struct xgi_info * info, unsigned long address)
XGI_INFO("return 0x%p\n", ret);
return ret;
- } else {
- XGI_INFO("used_list = used_list->next;\n");
- used_list = used_list->next;
}
}