Commit e15ae2dd authored by Jesper Juhl's avatar Jesper Juhl Committed by Linus Torvalds

[PATCH] Whitespace and CodingStyle cleanup for lib/idr.c

Cleanup trailing whitespace, blank lines, CodingStyle issues etc, for
lib/idr.c
Signed-off-by: default avatarJesper Juhl <jesper.juhl@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 850b9247
...@@ -6,20 +6,20 @@ ...@@ -6,20 +6,20 @@
* Modified by George Anzinger to reuse immediately and to use * Modified by George Anzinger to reuse immediately and to use
* find bit instructions. Also removed _irq on spinlocks. * find bit instructions. Also removed _irq on spinlocks.
* *
* Small id to pointer translation service. * Small id to pointer translation service.
* *
* It uses a radix tree like structure as a sparse array indexed * It uses a radix tree like structure as a sparse array indexed
* by the id to obtain the pointer. The bitmap makes allocating * by the id to obtain the pointer. The bitmap makes allocating
* a new id quick. * a new id quick.
* *
* You call it to allocate an id (an int) an associate with that id a * You call it to allocate an id (an int) an associate with that id a
* pointer or what ever, we treat it as a (void *). You can pass this * pointer or what ever, we treat it as a (void *). You can pass this
* id to a user for him to pass back at a later time. You then pass * id to a user for him to pass back at a later time. You then pass
* that id to this code and it returns your pointer. * that id to this code and it returns your pointer.
* You can release ids at any time. When all ids are released, most of * You can release ids at any time. When all ids are released, most of
* the memory is returned (we keep IDR_FREE_MAX) in a local pool so we * the memory is returned (we keep IDR_FREE_MAX) in a local pool so we
* don't need to go to the memory "store" during an id allocate, just * don't need to go to the memory "store" during an id allocate, just
* so you don't need to be too concerned about locking and conflicts * so you don't need to be too concerned about locking and conflicts
* with the slab allocator. * with the slab allocator.
*/ */
...@@ -77,7 +77,7 @@ int idr_pre_get(struct idr *idp, gfp_t gfp_mask) ...@@ -77,7 +77,7 @@ int idr_pre_get(struct idr *idp, gfp_t gfp_mask)
while (idp->id_free_cnt < IDR_FREE_MAX) { while (idp->id_free_cnt < IDR_FREE_MAX) {
struct idr_layer *new; struct idr_layer *new;
new = kmem_cache_alloc(idr_layer_cache, gfp_mask); new = kmem_cache_alloc(idr_layer_cache, gfp_mask);
if(new == NULL) if (new == NULL)
return (0); return (0);
free_layer(idp, new); free_layer(idp, new);
} }
...@@ -107,7 +107,7 @@ static int sub_alloc(struct idr *idp, void *ptr, int *starting_id) ...@@ -107,7 +107,7 @@ static int sub_alloc(struct idr *idp, void *ptr, int *starting_id)
if (m == IDR_SIZE) { if (m == IDR_SIZE) {
/* no space available go back to previous layer. */ /* no space available go back to previous layer. */
l++; l++;
id = (id | ((1 << (IDR_BITS*l))-1)) + 1; id = (id | ((1 << (IDR_BITS * l)) - 1)) + 1;
if (!(p = pa[l])) { if (!(p = pa[l])) {
*starting_id = id; *starting_id = id;
return -2; return -2;
...@@ -161,7 +161,7 @@ static int idr_get_new_above_int(struct idr *idp, void *ptr, int starting_id) ...@@ -161,7 +161,7 @@ static int idr_get_new_above_int(struct idr *idp, void *ptr, int starting_id)
{ {
struct idr_layer *p, *new; struct idr_layer *p, *new;
int layers, v, id; int layers, v, id;
id = starting_id; id = starting_id;
build_up: build_up:
p = idp->top; p = idp->top;
...@@ -225,6 +225,7 @@ static int idr_get_new_above_int(struct idr *idp, void *ptr, int starting_id) ...@@ -225,6 +225,7 @@ static int idr_get_new_above_int(struct idr *idp, void *ptr, int starting_id)
int idr_get_new_above(struct idr *idp, void *ptr, int starting_id, int *id) int idr_get_new_above(struct idr *idp, void *ptr, int starting_id, int *id)
{ {
int rv; int rv;
rv = idr_get_new_above_int(idp, ptr, starting_id); rv = idr_get_new_above_int(idp, ptr, starting_id);
/* /*
* This is a cheap hack until the IDR code can be fixed to * This is a cheap hack until the IDR code can be fixed to
...@@ -259,6 +260,7 @@ EXPORT_SYMBOL(idr_get_new_above); ...@@ -259,6 +260,7 @@ EXPORT_SYMBOL(idr_get_new_above);
int idr_get_new(struct idr *idp, void *ptr, int *id) int idr_get_new(struct idr *idp, void *ptr, int *id)
{ {
int rv; int rv;
rv = idr_get_new_above_int(idp, ptr, 0); rv = idr_get_new_above_int(idp, ptr, 0);
/* /*
* This is a cheap hack until the IDR code can be fixed to * This is a cheap hack until the IDR code can be fixed to
...@@ -306,11 +308,10 @@ static void sub_remove(struct idr *idp, int shift, int id) ...@@ -306,11 +308,10 @@ static void sub_remove(struct idr *idp, int shift, int id)
free_layer(idp, **paa); free_layer(idp, **paa);
**paa-- = NULL; **paa-- = NULL;
} }
if ( ! *paa ) if (!*paa)
idp->layers = 0; idp->layers = 0;
} else { } else
idr_remove_warning(id); idr_remove_warning(id);
}
} }
/** /**
...@@ -326,9 +327,8 @@ void idr_remove(struct idr *idp, int id) ...@@ -326,9 +327,8 @@ void idr_remove(struct idr *idp, int id)
id &= MAX_ID_MASK; id &= MAX_ID_MASK;
sub_remove(idp, (idp->layers - 1) * IDR_BITS, id); sub_remove(idp, (idp->layers - 1) * IDR_BITS, id);
if ( idp->top && idp->top->count == 1 && if (idp->top && idp->top->count == 1 && (idp->layers > 1) &&
(idp->layers > 1) && idp->top->ary[0]) { // We can drop a layer
idp->top->ary[0]){ // We can drop a layer
p = idp->top->ary[0]; p = idp->top->ary[0];
idp->top->bitmap = idp->top->count = 0; idp->top->bitmap = idp->top->count = 0;
...@@ -337,7 +337,6 @@ void idr_remove(struct idr *idp, int id) ...@@ -337,7 +337,6 @@ void idr_remove(struct idr *idp, int id)
--idp->layers; --idp->layers;
} }
while (idp->id_free_cnt >= IDR_FREE_MAX) { while (idp->id_free_cnt >= IDR_FREE_MAX) {
p = alloc_layer(idp); p = alloc_layer(idp);
kmem_cache_free(idr_layer_cache, p); kmem_cache_free(idr_layer_cache, p);
return; return;
...@@ -391,8 +390,8 @@ void *idr_find(struct idr *idp, int id) ...@@ -391,8 +390,8 @@ void *idr_find(struct idr *idp, int id)
} }
EXPORT_SYMBOL(idr_find); EXPORT_SYMBOL(idr_find);
static void idr_cache_ctor(void * idr_layer, static void idr_cache_ctor(void * idr_layer, kmem_cache_t *idr_layer_cache,
kmem_cache_t *idr_layer_cache, unsigned long flags) unsigned long flags)
{ {
memset(idr_layer, 0, sizeof(struct idr_layer)); memset(idr_layer, 0, sizeof(struct idr_layer));
} }
...@@ -400,7 +399,7 @@ static void idr_cache_ctor(void * idr_layer, ...@@ -400,7 +399,7 @@ static void idr_cache_ctor(void * idr_layer,
static int init_id_cache(void) static int init_id_cache(void)
{ {
if (!idr_layer_cache) if (!idr_layer_cache)
idr_layer_cache = kmem_cache_create("idr_layer_cache", idr_layer_cache = kmem_cache_create("idr_layer_cache",
sizeof(struct idr_layer), 0, 0, idr_cache_ctor, NULL); sizeof(struct idr_layer), 0, 0, idr_cache_ctor, NULL);
return 0; return 0;
} }
......
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