WebSVN – HelenOS – Path Comparison – / – /trunk/kernel/ Rev 2744 and /trunk/kernel/ Rev 2745

Ignore whitespace Rev 2744 → Rev 2745

 /trunk/kernel/test/test.c
 ,7 → 58,11
 #include <print/print1.def>
 #include <thread/thread1.def>
 #include <sysinfo/sysinfo1.def>
-    {NULL, NULL, NULL}
+    {
+        .name = NULL,
+        .desc = NULL,
+        .entry = NULL
+    }
 };
 /** @}

 /trunk/kernel/test/avltree/avltree1.c
 ,7 → 246,7
 static void test_tree_delmin(avltree_t *tree, count_t node_count, bool quiet)
 {
-    int i = 0;
+    unsigned int i = 0;
     if (!quiet)
         printf("Deleting minimum nodes...");

 /trunk/kernel/test/synch/rwlock5.c
 ,7 → 69,7
 char * test_rwlock5(bool quiet)
 {
     int i, j, k;
-    count_t readers, writers;
+    long readers, writers;
     waitq_initialize(&can_start);
     rwlock_initialize(&rwlock);

 /trunk/kernel/test/thread/thread1.c
 ,7 → 78,7
     thread_sleep(10);
     atomic_set(&finish, 0);
-    while (atomic_get(&threads_finished) < total) {
+    while (atomic_get(&threads_finished) < ((long) total)) {
         if (!quiet)
             printf("Threads left: %d\n", total - atomic_get(&threads_finished));
         thread_sleep(1);

 /trunk/kernel/test/mm/falloc2.c
 ,7 → 88,7
                 printf("Thread #%llu (cpu%d): Deallocating ... \n", THREAD->tid, CPU->id);
             for (i = 0; i < allocated; i++) {
-                for (k = 0; k <= ((FRAME_SIZE << order) - 1); k++) {
+                for (k = 0; k <= (((index_t) FRAME_SIZE << order) - 1); k++) {
                     if (((uint8_t *) frames[i])[k] != val) {
                         if (!sh_quiet)
                             printf("Thread #%llu (cpu%d): Unexpected data (%d) in block %p offset %#zx\n", THREAD->tid, CPU->id, ((char *) frames[i])[k], frames[i], k);

 /trunk/kernel/test/fpu/mips2.c
 ,7 → 151,7
     thread_sleep(1);
     waitq_wakeup(&can_start, WAKEUP_ALL);
-    while (atomic_get(&threads_ok) != total) {
+    while (atomic_get(&threads_ok) != (long) total) {
         if (!quiet)
             printf("Threads left: %d\n", total - atomic_get(&threads_ok));
         thread_sleep(1);

 /trunk/kernel/test/fpu/fpu1.c
 ,7 → 215,7
     thread_sleep(1);
     waitq_wakeup(&can_start, WAKEUP_ALL);
-    while (atomic_get(&threads_ok) != total) {
+    while (atomic_get(&threads_ok) != (long) total) {
         if (!quiet)
             printf("Threads left: %d\n", total - atomic_get(&threads_ok));
         thread_sleep(1);

 /trunk/kernel/test/fpu/sse1.c
 ,7 → 151,7
     thread_sleep(1);
     waitq_wakeup(&can_start, WAKEUP_ALL);
-    while (atomic_get(&threads_ok) != total) {
+    while (atomic_get(&threads_ok) != (long) total) {
         if (!quiet)
             printf("Threads left: %d\n", total - atomic_get(&threads_ok));
         thread_sleep(1);

 /trunk/kernel/genarch/src/ofw/ebus.c
 ,7 → 57,7
     ranges = prop->size / sizeof(ofw_ebus_range_t);
     range = prop->value;
-    int i;
+    unsigned int i;
     for (i = 0; i < ranges; i++) {
         if (reg->space != range[i].child_space)
 ,7 → 102,7
     uint32_t addr = reg->addr & intr_mask->addr_mask;
     uint32_t intr = interrupt & intr_mask->intr_mask;
-    int i;
+    unsigned int i;
     for (i = 0; i < count; i++) {
         if ((intr_map[i].space == space) && (intr_map[i].addr == addr)
             && (intr_map[i].intr == intr))

 /trunk/kernel/genarch/src/ofw/fhc.c
 ,7 → 55,7
     ranges = prop->size / sizeof(ofw_fhc_range_t);
     range = prop->value;
-    int i;
+    unsigned int i;
     for (i = 0; i < ranges; i++) {
         if (overlaps(reg->addr, reg->size, range[i].child_base, range[i].size)) {
 ,7 → 97,7
     ranges = prop->size / sizeof(ofw_central_range_t);
     range = prop->value;
-    int i;
+    unsigned int i;
     for (i = 0; i < ranges; i++) {
         if (overlaps(reg->addr, reg->size, range[i].child_base, range[i].size)) {

 /trunk/kernel/genarch/src/ofw/ofw_tree.c
 ,7 → 61,7
  */
 ofw_tree_property_t *ofw_tree_getprop(const ofw_tree_node_t *node, const char *name)
 {
-    int i;
+    unsigned int i;
     for (i = 0; i < node->properties; i++) {
         if (strcmp(node->property[i].name, name) == 0)

 /trunk/kernel/genarch/src/ofw/pci.c
 ,7 → 65,7
     ranges = prop->size / sizeof(ofw_pci_range_t);
     range = prop->value;
-    int i;
+    unsigned int i;
     for (i = 0; i < ranges; i++) {
         if ((reg->space & PCI_SPACE_MASK) != (range[i].space & PCI_SPACE_MASK))
 ,7 → 100,7
     assigned_addresses = prop->size / sizeof(ofw_pci_reg_t);
     assigned_address = prop->value;
-    int i;
+    unsigned int i;
     for (i = 0; i < assigned_addresses; i++) {
         if ((assigned_address[i].space & PCI_REG_MASK) == (reg->space & PCI_REG_MASK)) {

 /trunk/kernel/genarch/src/ofw/sbus.c
 ,7 → 61,7
     ranges = prop->size / sizeof(ofw_sbus_range_t);
     range = prop->value;
-    int i;
+    unsigned int i;
     for (i = 0; i < ranges; i++) {
         if (overlaps(reg->addr, reg->size, range[i].child_base,

 /trunk/kernel/generic/include/lib/elf.h
 ,7 → 336,7
 typedef struct elf64_symbol elf_symbol_t;
 #endif
-extern char *elf_error(int rc);
+extern char *elf_error(unsigned int rc);
 #endif

 /trunk/kernel/generic/include/mm/slab.h
 ,7 → 53,7
 #define SLAB_INSIDE_SIZE   (PAGE_SIZE >> 3)
 /** Maximum wasted space we allow for cache */
-#define SLAB_MAX_BADNESS(cache)   ((PAGE_SIZE << (cache)->order) >> 2)
+#define SLAB_MAX_BADNESS(cache)   (((unsigned int) PAGE_SIZE << (cache)->order) >> 2)
 /* slab_reclaim constants */
 ,8 → 99,8
     int flags;
     /* Computed values */
-    uint8_t order;      /**< Order of frames to be allocated */
-    int objects;        /**< Number of objects that fit in */
+    uint8_t order;              /**< Order of frames to be allocated */
+    unsigned int objects;       /**< Number of objects that fit in */
     /* Statistics */
     atomic_t allocated_slabs;

 /trunk/kernel/generic/include/mm/as.h
 ,7 → 299,7
 extern mem_backend_t elf_backend;
 extern mem_backend_t phys_backend;
-extern int elf_load(elf_header_t *header, as_t *as);
+extern unsigned int elf_load(elf_header_t *header, as_t *as);
 /* Address space area related syscalls. */
 extern unative_t sys_as_area_create(uintptr_t address, size_t size, int flags);

 /trunk/kernel/generic/src/synch/futex.c
 ,7 → 324,7
     for (cur = TASK->futexes.leaf_head.next;
         cur != &TASK->futexes.leaf_head; cur = cur->next) {
         btree_node_t *node;
-        int i;
+        unsigned int i;
         node = list_get_instance(cur, btree_node_t, leaf_link);
         for (i = 0; i < node->keys; i++) {

 /trunk/kernel/generic/src/main/main.c
 ,7 → 86,7
 /** Initial user-space tasks */
 init_t init = {
+    .cnt = 0
 };
 /** Boot allocations. */

 /trunk/kernel/generic/src/debug/symtab.c
 ,7 → 70,7
  */
 static char * symtab_search_one(const char *name, int *startpos)
 {
-    int namelen = strlen(name);
+    unsigned int namelen = strlen(name);
     char *curname;
     int i,j;
     int colonoffset = -1;

/trunk/kernel/generic/src/interrupt/interrupt.c
103,6 → 103,7
/** kconsole cmd - print all exceptions */
static int exc_print_cmd(cmd_arg_t *argv)
{
#if (IVT_ITEMS > 0)
unsigned int i;
char *symbol;

138,6 → 139,7
}

spinlock_unlock(&exctbl_lock);
#endif

return 1;
}

 /trunk/kernel/generic/src/time/clock.c
 ,7 → 137,7
     timeout_handler_t f;
     void *arg;
     count_t missed_clock_ticks = CPU->missed_clock_ticks;
-    int i;
+    unsigned int i;
     /*
      * To avoid lock ordering problems,

 /trunk/kernel/generic/src/printf/printf_core.c
 ,7 → 93,7
 static int printf_putnchars(const char * buf, size_t count,
     struct printf_spec *ps)
 {
-    return ps->write((void *)buf, count, ps->data);
+    return ps->write((void *) buf, count, ps->data);
 }
 /** Print a string without adding a newline.
 ,8 → 177,8
  *
  * @return      Number of characters printed, negative value on failure.
  */
-static int print_string(char *s, int width, int precision, uint64_t flags,
-    struct printf_spec *ps)
+static int print_string(char *s, int width, unsigned int precision,
+    uint64_t flags, struct printf_spec *ps)
 {
     int counter = 0;
     size_t size;

 /trunk/kernel/generic/src/proc/task.c
 ,7 → 244,7
 {
     as_t *as;
     as_area_t *a;
-    int rc;
+    unsigned int rc;
     thread_t *t;
     task_t *task;
     uspace_arg_t *kernel_uarg;

 /trunk/kernel/generic/src/lib/elf.c
 ,7 → 69,7
  * @param as Created and properly mapped address space
  * @return EE_OK on success
  */
-int elf_load(elf_header_t *header, as_t * as)
+unsigned int elf_load(elf_header_t *header, as_t * as)
 {
     int i, rc;
 ,7 → 131,7
  *
  * @return NULL terminated description of error.
  */
-char *elf_error(int rc)
+char *elf_error(unsigned int rc)
 {
     ASSERT(rc < sizeof(error_codes) / sizeof(char *));

 /trunk/kernel/generic/src/lib/memstr.c
 ,7 → 59,7
  */
 void *_memcpy(void * dst, const void *src, size_t cnt)
 {
-    int i, j;
+    unsigned int i, j;
     if (ALIGN_UP((uintptr_t) src, sizeof(unative_t)) != (uintptr_t) src ||
         ALIGN_UP((uintptr_t) dst, sizeof(unative_t)) != (uintptr_t) dst) {
 ,7 → 89,7
  */
 void _memsetb(uintptr_t dst, size_t cnt, uint8_t x)
 {
-    int i;
+    unsigned int i;
     uint8_t *p = (uint8_t *) dst;
     for (i = 0; i < cnt; i++)
 ,7 → 108,7
  */
 void _memsetw(uintptr_t dst, size_t cnt, uint16_t x)
 {
-    int i;
+    unsigned int i;
     uint16_t *p = (uint16_t *) dst;
     for (i = 0; i < cnt; i++)

 /trunk/kernel/generic/src/lib/sort.c
 ,14 → 96,17
 void _qsort(void * data, count_t n, size_t e_size, int (* cmp) (void * a, void * b), void *tmp, void *pivot)
 {
     if (n > 4) {
-        int i = 0, j = n - 1;
+        unsigned int i = 0, j = n - 1;
         memcpy(pivot, data, e_size);
         while (1) {
-            while ((cmp(data + i * e_size, pivot) < 0) && i < n) i++;
-            while ((cmp(data + j * e_size, pivot) >=0) && j > 0) j--;
-            if (i<j) {
+            while ((cmp(data + i * e_size, pivot) < 0) && (i < n))
+                i++;
+            while ((cmp(data + j * e_size, pivot) >= 0) && (j > 0))
+                j--;
+            if (i < j) {
                 memcpy(tmp, data + i * e_size, e_size);
                 memcpy(data + i * e_size, data + j * e_size, e_size);
                 memcpy(data + j * e_size, tmp, e_size);

 /trunk/kernel/generic/src/lib/func.c
 ,9 → 139,9
  */
 int strncmp(const char *src, const char *dst, size_t len)
 {
-    int i;
+    unsigned int i;
-    for (i = 0; *src && *dst && i < len; src++, dst++, i++) {
+    for (i = 0; (*src) && (*dst) && (i < len); src++, dst++, i++) {
         if (*src < *dst)
             return -1;
         if (*src > *dst)
 ,7 → 168,7
  */
 void strncpy(char *dest, const char *src, size_t len)
 {
-    int i;
+    unsigned int i;
     for (i = 0; i < len; i++) {
         if (!(dest[i] = src[i]))
             return;

 /trunk/kernel/generic/src/mm/slab.c
 ,7 → 172,7
     void *data;
     slab_t *slab;
     size_t fsize;
-    int i;
+    unsigned int i;
     unsigned int zone = 0;
     data = frame_alloc_generic(cache->order, FRAME_KA | flags, &zone);
 ,8 → 191,8
     }
     /* Fill in slab structures */
-    for (i=0; i < (1 << cache->order); i++)
-        frame_set_parent(ADDR2PFN(KA2PA(data))+i, slab, zone);
+    for (i = 0; i < ((unsigned int) 1 << cache->order); i++)
+        frame_set_parent(ADDR2PFN(KA2PA(data)) + i, slab, zone);
     slab->start = data;
     slab->available = cache->objects;
 ,7 → 199,7
     slab->nextavail = 0;
     slab->cache = cache;
-    for (i=0; i<cache->objects;i++)
+    for (i = 0; i < cache->objects; i++)
         *((int *) (slab->start + i*cache->size)) = i+1;
     atomic_inc(&cache->allocated_slabs);
 ,10 → 371,10
 static count_t magazine_destroy(slab_cache_t *cache,
                 slab_magazine_t *mag)
 {
-    int i;
+    unsigned int i;
     count_t frames = 0;
-    for (i=0;i < mag->busy; i++) {
+    for (i = 0; i < mag->busy; i++) {
         frames += slab_obj_destroy(cache, mag->objs[i], NULL);
         atomic_dec(&cache->cached_objs);
     }
 ,7 → 527,7
 /* Slab cache functions */
 /** Return number of objects that fit in certain cache size */
-static int comp_objects(slab_cache_t *cache)
+static unsigned int comp_objects(slab_cache_t *cache)
 {
     if (cache->flags & SLAB_CACHE_SLINSIDE)
         return ((PAGE_SIZE << cache->order) - sizeof(slab_t)) / cache->size;
 ,16 → 536,16
 }
 /** Return wasted space in slab */
-static int badness(slab_cache_t *cache)
+static unsigned int badness(slab_cache_t *cache)
 {
-    int objects;
-    int ssize;
+    unsigned int objects;
+    unsigned int ssize;
     objects = comp_objects(cache);
     ssize = PAGE_SIZE << cache->order;
     if (cache->flags & SLAB_CACHE_SLINSIDE)
         ssize -= sizeof(slab_t);
-    return ssize - objects*cache->size;
+    return ssize - objects * cache->size;
 }
 /**
 ,16 → 553,15
  */
 static void make_magcache(slab_cache_t *cache)
 {
-    int i;
+    unsigned int i;
     ASSERT(_slab_initialized >= 2);
     cache->mag_cache = malloc(sizeof(slab_mag_cache_t)*config.cpu_count,0);
-    for (i=0; i < config.cpu_count; i++) {
+    for (i = 0; i < config.cpu_count; i++) {
         memsetb((uintptr_t)&cache->mag_cache[i],
             sizeof(cache->mag_cache[i]), 0);
-        spinlock_initialize(&cache->mag_cache[i].lock,
-                    "slab_maglock_cpu");
+        spinlock_initialize(&cache->mag_cache[i].lock, "slab_maglock_cpu");
     }
 }
 ,7 → 653,7
  */
 static count_t _slab_reclaim(slab_cache_t *cache, int flags)
 {
-    int i;
+    unsigned int i;
     slab_magazine_t *mag;
     count_t frames = 0;
     int magcount;
 ,7 → 674,7
     if (flags & SLAB_RECLAIM_ALL) {
         /* Free cpu-bound magazines */
         /* Destroy CPU magazines */
-        for (i=0; i<config.cpu_count; i++) {
+        for (i = 0; i < config.cpu_count; i++) {
             spinlock_lock(&cache->mag_cache[i].lock);
             mag = cache->mag_cache[i].current;

 /trunk/kernel/generic/src/mm/tlb.c
 ,7 → 81,7
 void tlb_shootdown_start(tlb_invalidate_type_t type, asid_t asid,
     uintptr_t page, count_t count)
 {
-    int i;
+    unsigned int i;
     CPU->tlb_active = 0;
     spinlock_lock(&tlblock);
 ,7 → 144,7
     asid_t asid;
     uintptr_t page;
     count_t count;
-    int i;
+    unsigned int i;
     ASSERT(CPU);

 /trunk/kernel/generic/src/mm/backend_anon.c
 ,7 → 98,7
             ALIGN_DOWN(addr, PAGE_SIZE) - area->base, &leaf);
         if (!frame) {
             bool allocate = true;
-            int i;
+            unsigned int i;
             /*
              * Zero can be returned as a valid frame address.
 ,13 → 193,13
     for (cur = area->used_space.leaf_head.next;
         cur != &area->used_space.leaf_head; cur = cur->next) {
         btree_node_t *node;
-        int i;
+        unsigned int i;
         node = list_get_instance(cur, btree_node_t, leaf_link);
         for (i = 0; i < node->keys; i++) {
             uintptr_t base = node->key[i];
             count_t count = (count_t) node->value[i];
-            int j;
+            unsigned int j;
             for (j = 0; j < count; j++) {
                 pte_t *pte;

 /trunk/kernel/generic/src/mm/as.c
 ,7 → 431,7
                 uintptr_t b = node->key[node->keys - 1];
                 count_t c =
                     (count_t) node->value[node->keys - 1];
-                int i = 0;
+                unsigned int i = 0;
                 if (overlaps(b, c * PAGE_SIZE, area->base,
                     pages * PAGE_SIZE)) {
 ,7 → 561,7
     for (cur = area->used_space.leaf_head.next;
         cur != &area->used_space.leaf_head; cur = cur->next) {
         btree_node_t *node;
-        int i;
+        unsigned int i;
         node = list_get_instance(cur, btree_node_t, leaf_link);
         for (i = 0; i < node->keys; i++) {
 ,7 → 1097,7
 {
     as_area_t *a;
     btree_node_t *leaf, *lnode;
-    int i;
+    unsigned int i;
     a = (as_area_t *) btree_search(&as->as_area_btree, va, &leaf);
     if (a) {
 ,7 → 1155,7
 {
     as_area_t *a;
     btree_node_t *leaf, *node;
-    int i;
+    unsigned int i;
     /*
      * We don't want any area to have conflicts with NULL page.
 ,7 → 1264,7
 {
     btree_node_t *leaf, *node;
     count_t pages;
-    int i;
+    unsigned int i;
     ASSERT(page == ALIGN_DOWN(page, PAGE_SIZE));
     ASSERT(count);
 ,7 → 1546,7
 {
     btree_node_t *leaf, *node;
     count_t pages;
-    int i;
+    unsigned int i;
     ASSERT(page == ALIGN_DOWN(page, PAGE_SIZE));
     ASSERT(count);
 ,7 → 1734,7
         for (cur = sh_info->pagemap.leaf_head.next;
             cur != &sh_info->pagemap.leaf_head; cur = cur->next) {
             btree_node_t *node;
-            int i;
+            unsigned int i;
             node = list_get_instance(cur, btree_node_t, leaf_link);
             for (i = 0; i < node->keys; i++)
 ,7 → 1795,7
         node = list_get_instance(cur, btree_node_t, leaf_link);
-        int i;
+        unsigned int i;
         for (i = 0; i < node->keys; i++) {
             as_area_t *area = node->value[i];

 /trunk/kernel/generic/src/mm/frame.c
 ,7 → 120,7
 static inline int frame_index_valid(zone_t *zone, index_t index)
 {
-    return (index >= 0) && (index < zone->count);
+    return (index < zone->count);
 }
 /** Compute pfn_t from frame_t pointer & zone pointer */
 ,7 → 210,7
     spinlock_lock(&zones.lock);
-    if (hint >= zones.count || hint < 0)
+    if (hint >= zones.count)
         hint = 0;
     i = hint;
 ,7 → 719,7
     ipl = interrupts_disable();
     spinlock_lock(&zones.lock);
-    if (z1 < 0 || z1 >= zones.count || z2 < 0 || z2 >= zones.count)
+    if ((z1 >= zones.count) || (z2 >= zones.count))
         goto errout;
     /* We can join only 2 zones with none existing inbetween */
     if (z2-z1 != 1)

 /trunk/kernel/generic/src/mm/backend_elf.c
 ,7 → 103,7
         frame = (uintptr_t) btree_search(&area->sh_info->pagemap,
             ALIGN_DOWN(addr, PAGE_SIZE) - area->base, &leaf);
         if (!frame) {
-            int i;
+            unsigned int i;
             /*
              * Workaround for valid NULL address.
 ,7 → 290,7
     mutex_lock(&area->sh_info->lock);
     for (cur = &node->leaf_link; cur != &area->used_space.leaf_head;
         cur = cur->next) {
-        int i;
+        unsigned int i;
         node = list_get_instance(cur, btree_node_t, leaf_link);
 ,7 → 297,7
         for (i = 0; i < node->keys; i++) {
             uintptr_t base = node->key[i];
             count_t count = (count_t) node->value[i];
-            int j;
+            unsigned int j;
             /*
              * Skip read-only areas of used space that are backed

 /trunk/kernel/generic/src/ipc/sysipc.c
 ,7 → 164,7
 {
     int phoneid;
-    if (IPC_GET_RETVAL(answer->data) == EHANGUP) {
+    if ((native_t) IPC_GET_RETVAL(answer->data) == EHANGUP) {
         /* In case of forward, hangup the forwared phone,
          * not the originator
          */
 ,7 → 354,7
  */
 static void process_answer(call_t *call)
 {
-    if (IPC_GET_RETVAL(call->data) == EHANGUP &&
+    if (((native_t) IPC_GET_RETVAL(call->data) == EHANGUP) &&
         (call->flags & IPC_CALL_FORWARDED))
         IPC_SET_RETVAL(call->data, EFORWARD);

 /trunk/kernel/generic/src/ipc/irq.c
 ,7 → 65,7
  */
 static void code_execute(call_t *call, irq_code_t *code)
 {
-    int i;
+    unsigned int i;
     unative_t dstval = 0;
     if (!code)

 /trunk/kernel/Makefile
 ,7 → 43,7
     -DKERNEL
 GCC_CFLAGS = -I$(INCLUDES) -O$(OPTIMIZATION) \
-    -fno-builtin -fomit-frame-pointer -Wall -Wmissing-prototypes -Werror \
+    -fno-builtin -fomit-frame-pointer -Wall -Wextra -Wno-unused-parameter -Wmissing-prototypes -Werror \
     -nostdlib -nostdinc
 ICC_CFLAGS = -I$(INCLUDES) -O$(OPTIMIZATION) \

 /trunk/kernel/arch/sparc64/src/smp/ipi.c
 ,7 → 116,7
  */
 void ipi_broadcast_arch(int ipi)
 {
-    int i;
+    unsigned int i;
     void (* func)(void);

 /trunk/kernel/arch/sparc64/src/mm/tlb.c
 ,7 → 490,7
  */
 void tlb_invalidate_pages(asid_t asid, uintptr_t page, count_t cnt)
 {
-    int i;
+    unsigned int i;
     tlb_context_reg_t pc_save, ctx;
     /* switch to nucleus because we are mapped by the primary context */

 /trunk/kernel/arch/sparc64/src/mm/frame.c
 ,7 → 48,7
  */
 void frame_arch_init(void)
 {
-    int i;
+    unsigned int i;
     pfn_t confdata;
     if (config.cpu_active == 1) {

 /trunk/kernel/arch/sparc64/src/mm/page.c
 ,7 → 66,7
     } else {
 #ifdef CONFIG_SMP
-        int i;
+        unsigned int i;
         /*
          * Copy locked DTLB entries from the BSP.
 ,7 → 98,7
 uintptr_t hw_map(uintptr_t physaddr, size_t size)
 {
     unsigned int order;
-    int i;
+    unsigned int i;
     ASSERT(config.cpu_active == 1);

 /trunk/kernel/arch/ia64/src/ia64.c
 ,12 → 62,13
     /* Setup usermode init tasks. */
 //#ifdef I460GX
-    int i;
+    unsigned int i;
     init.cnt = bootinfo->taskmap.count;
-    for(i=0;i<init.cnt;i++)
-    {
-        init.tasks[i].addr = ((unsigned long)bootinfo->taskmap.tasks[i].addr)|VRN_MASK;
-        init.tasks[i].size = bootinfo->taskmap.tasks[i].size;
+    for (i = 0; i < init.cnt; i++) {
+        init.tasks[i].addr = ((unsigned long) bootinfo->taskmap.tasks[i].addr) | VRN_MASK;
+        init.tasks[i].size = bootinfo->taskmap.tasks[i].size;
     }
 /*
 #else

 /trunk/kernel/arch/ia64/src/mm/tlb.c
 ,7 → 59,7
     uintptr_t adr;
     uint32_t count1, count2, stride1, stride2;
-    int i, j;
+    unsigned int i, j;
     adr = PAL_PTCE_INFO_BASE();
     count1 = PAL_PTCE_INFO_COUNT1();
 ,8 → 69,8
     ipl = interrupts_disable();
-    for(i = 0; i < count1; i++) {
-        for(j = 0; j < count2; j++) {
+    for (i = 0; i < count1; i++) {
+        for (j = 0; j < count2; j++) {
             asm volatile (
                 "ptc.e %0 ;;"
                 :

 /trunk/kernel/arch/arm32/src/arm32.c
 ,7 → 55,7
 /** Performs arm32 specific initialization before main_bsp() is called. */
 void arch_pre_main(void)
 {
-    int i;
+    unsigned int i;
     init.cnt = bootinfo.cnt;

 /trunk/kernel/arch/arm32/src/debug/print.c
 ,10 → 56,10
  */
 static int debug_write(const char *str, size_t count, void *unused)
 {
-    int i;
-    for (i = 0; i < count; ++i) {
+    unsigned int i;
+    for (i = 0; i < count; ++i)
         putc(str[i]);
-    }
     return i;
 }

 /trunk/kernel/arch/arm32/src/cpu/cpu.c
 ,7 → 56,7
 };
 /** Length of the #imp_data array */
-static int imp_data_length = sizeof(imp_data) / sizeof(char *);
+static unsigned int imp_data_length = sizeof(imp_data) / sizeof(char *);
 /** Architecture names */
 static char *arch_data[] = {
 ,7 → 71,7
 };
 /** Length of the #arch_data array */
-static int arch_data_length = sizeof(arch_data) / sizeof(char *);
+static unsigned int arch_data_length = sizeof(arch_data) / sizeof(char *);
 /** Retrieves processor identification from CP15 register 0.

 /trunk/kernel/arch/ia32xen/src/smp/mps.c
 ,11 → 77,11
 struct __io_intr_entry *io_intr_entries = NULL;
 struct __l_intr_entry *l_intr_entries = NULL;
-int processor_entry_cnt = 0;
-int bus_entry_cnt = 0;
-int io_apic_entry_cnt = 0;
-int io_intr_entry_cnt = 0;
-int l_intr_entry_cnt = 0;
+unsigned int processor_entry_cnt = 0;
+unsigned int bus_entry_cnt = 0;
+unsigned int io_apic_entry_cnt = 0;
+unsigned int io_intr_entry_cnt = 0;
+unsigned int l_intr_entry_cnt = 0;
 waitq_t ap_completion_wq;
 ,7 → 417,7
 int mps_irq_to_pin(unsigned int irq)
 {
-    int i;
+    unsigned int i;
     for (i = 0; i < io_intr_entry_cnt; i++) {
         if (io_intr_entries[i].src_bus_irq == irq && io_intr_entries[i].intr_type == 0)

 /trunk/kernel/arch/ia32xen/src/smp/smp.c
 ,7 → 98,7
  */
 void kmp(void *arg)
 {
-    int i;
+    unsigned int i;
     ASSERT(ops != NULL);

 /trunk/kernel/arch/ia32xen/src/mm/tlb.c
 ,7 → 65,7
  */
 void tlb_invalidate_pages(asid_t asid, uintptr_t page, count_t cnt)
 {
-    int i;
+    unsigned int i;
     for (i = 0; i < cnt; i++)
         invlpg(page + i * PAGE_SIZE);

 /trunk/kernel/arch/amd64/src/mm/page.c
 ,7 → 82,7
 void page_arch_init(void)
 {
     uintptr_t cur;
-    int i;
+    unsigned int i;
     int identity_flags = PAGE_CACHEABLE | PAGE_EXEC | PAGE_GLOBAL | PAGE_WRITE;
     if (config.cpu_active == 1) {

 /trunk/kernel/arch/mips32/src/cpu/cpu.c
 ,22 → 104,20
 void cpu_print_report(cpu_t *m)
 {
     struct data_t *data;
-    int i;
+    unsigned int i;
     if (m->arch.imp_num & 0x80) {
         /* Count records */
-        for (i=0;imp_data80[i].vendor;i++)
-            ;
+        for (i = 0; imp_data80[i].vendor; i++);
         if ((m->arch.imp_num & 0x7f) >= i) {
-            printf("imp=%d\n",m->arch.imp_num);
+            printf("imp=%d\n", m->arch.imp_num);
             return;
         }
         data = &imp_data80[m->arch.imp_num & 0x7f];
     } else {
-        for (i=0;imp_data[i].vendor;i++)
-            ;
+        for (i = 0; imp_data[i].vendor; i++);
         if (m->arch.imp_num >= i) {
-            printf("imp=%d\n",m->arch.imp_num);
+            printf("imp=%d\n", m->arch.imp_num);
             return;
         }
         data = &imp_data[m->arch.imp_num];
 ,7 → 125,7
     printf("cpu%d: %s %s (rev=%d.%d, imp=%d)\n",
         m->id, data->vendor, data->model, m->arch.rev_num >> 4,
-           m->arch.rev_num & 0xf, m->arch.imp_num);
+        m->arch.rev_num & 0xf, m->arch.imp_num);
 }
 /** @}

 /trunk/kernel/arch/mips32/src/debugger.c
 ,7 → 208,7
     bpinfo_t *cur;
     ipl_t ipl;
-    if (argv->intval < 0 || argv->intval > BKPOINTS_MAX) {
+    if (argv->intval > BKPOINTS_MAX) {
         printf("Invalid breakpoint number.\n");
         return 0;
     }

 /trunk/kernel/arch/mips32/src/mm/tlb.c
 ,7 → 572,7
  */
 void tlb_invalidate_pages(asid_t asid, uintptr_t page, count_t cnt)
 {
-    int i;
+    unsigned int i;
     ipl_t ipl;
     entry_lo_t lo0, lo1;
     entry_hi_t hi, hi_save;
 ,7 → 583,7
     hi_save.value = cp0_entry_hi_read();
     ipl = interrupts_disable();
-    for (i = 0; i < cnt+1; i+=2) {
+    for (i = 0; i < cnt + 1; i += 2) {
         hi.value = 0;
         prepare_entry_hi(&hi, asid, page + i * PAGE_SIZE);
         cp0_entry_hi_write(hi.value);

 /trunk/kernel/arch/mips32/src/drivers/arc.c
 ,7 → 118,7
 static void arc_print_confdata(arc_component *c)
 {
     cm_resource_list *configdata;
-    int i;
+    unsigned int i;
     if (!c->configdatasize)
         return; /* No configuration data */
 ,7 → 161,7
 /** Print information about component */
 static void arc_print_component(arc_component *c)
 {
-    int i;
+    unsigned int i;
     printf("%s: ",ctypes[c->type]);
     for (i = 0; i < c->identifier_len; i++)

Subversion Repositories HelenOS

Compare Revisions

Ignore whitespace Rev 2744 → Rev 2745