Changeset 137691a in mainline


Ignore:
Timestamp:
2010-02-26T14:07:36Z (14 years ago)
Author:
Martin Decky <martin@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
91001e2, aa85487
Parents:
e0b0c25
Message:

remove unused variables and operations (as suggested by Clang static analyzer)

Location:
kernel
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • kernel/genarch/src/drivers/i8042/i8042.c

    re0b0c25 r137691a  
    6767        i8042_instance_t *instance = irq->instance;
    6868        i8042_t *dev = instance->i8042;
    69         uint8_t status;
    7069       
    71         if (((status = pio_read_8(&dev->status)) & i8042_BUFFER_FULL_MASK)) {
     70        if (pio_read_8(&dev->status) & i8042_BUFFER_FULL_MASK) {
    7271                uint8_t data = pio_read_8(&dev->data);
    7372                indev_push_character(instance->kbrdin, data);

  • kernel/generic/include/bitops.h

    re0b0c25 r137691a  
    6565        }
    6666       
    67         if (arg >> 1) {
    68                 arg >>= 1;
     67        if (arg >> 1)
    6968                n += 1;
    70         }
    7169       
    7270        return n;

  • kernel/generic/src/console/kconsole.c

    re0b0c25 r137691a  
    224224                printf("\n");
    225225                pos = NULL;
    226                 while ((hint = cmdtab_search_one(name, &pos))) {
     226                while (cmdtab_search_one(name, &pos)) {
    227227                        cmd_info_t *hlp = list_get_instance(pos, cmd_info_t, link);
    228228                        printf("%s (%s)\n", hlp->name, hlp->description);

  • kernel/generic/src/debug/symtab.c

    re0b0c25 r137691a  
    239239                printf("\n");
    240240                pos = 0;
    241                 while ((hint = symtab_search_one(name, &pos))) {
     241                while (symtab_search_one(name, &pos)) {
    242242                        printf("%s\n", symbol_table[pos].symbol_name);
    243243                        pos++;

  • kernel/generic/src/lib/elf.c

    re0b0c25 r137691a  
    155155    as_t *as, int flags)
    156156{
    157         char *interp;
    158 
    159157        switch (entry->p_type) {
    160158        case PT_NULL:
    … …  
    165163        case PT_DYNAMIC:
    166164        case PT_INTERP:
    167                 interp = (char *)elf + entry->p_offset;
    168                 /* FIXME */
    169                 /*if (memcmp((uintptr_t)interp, (uintptr_t)ELF_INTERP_ZSTR,
     165                // FIXME
     166                /*
     167                char *interp = (char *) elf + entry->p_offset;
     168                if (memcmp((uintptr_t) interp, (uintptr_t) ELF_INTERP_ZSTR,
    170169                    ELF_INTERP_ZLEN) != 0) {
    171170                        return EE_UNSUPPORTED;
    172                 }*/
     171                } */
    173172                if ((flags & ELD_F_LOADER) == 0) {
    174173                        return EE_LOADER;

  • kernel/generic/src/mm/as.c

    re0b0c25 r137691a  
    784784{
    785785        as_area_t *area;
    786         uintptr_t base;
    787786        link_t *cur;
    788787        ipl_t ipl;
    … …  
    813812                return ENOTSUP;
    814813        }
    815 
    816         base = area->base;
    817814
    818815        /*

  • kernel/generic/src/mm/backend_elf.c

    re0b0c25 r137691a  
    232232void elf_frame_free(as_area_t *area, uintptr_t page, uintptr_t frame)
    233233{
    234         elf_header_t *elf = area->backend_data.elf;
    235234        elf_segment_header_t *entry = area->backend_data.segment;
    236         uintptr_t base, start_anon;
    237         size_t i;
     235        uintptr_t start_anon;
    238236
    239237        ASSERT((page >= ALIGN_DOWN(entry->p_vaddr, PAGE_SIZE)) &&
    240238            (page < entry->p_vaddr + entry->p_memsz));
    241         i = (page - ALIGN_DOWN(entry->p_vaddr, PAGE_SIZE)) >> PAGE_WIDTH;
    242         base = (uintptr_t) (((void *) elf) +
    243             ALIGN_DOWN(entry->p_offset, FRAME_SIZE));
    244239        start_anon = entry->p_vaddr + entry->p_filesz;
    245240
    … …  
    257252                 * lower part is backed by the ELF image and the upper is
    258253                 * anonymous). In any case, a frame needs to be freed.
    259                  */ 
     254                 */
    260255                frame_free(frame);
    261256        }

  • kernel/generic/src/proc/thread.c

    re0b0c25 r137691a  
    264264
    265265        atomic_inc(&nrdy);
     266        // FIXME: Why is the avg value never read?
    266267        avg = atomic_get(&nrdy) / config.cpu_active;
    267268        atomic_inc(&cpu->nrdy);

  • kernel/generic/src/sysinfo/sysinfo.c

    re0b0c25 r137691a  
    4646                return NULL;
    4747       
    48         while (subtree != NULL) {
     48        while (subtree != NULL) {
    4949                int i = 0;
    5050                char *a = (char *) name;
    … …  
    6868                /* No matches try next */
    6969                subtree = subtree->next;
    70                 i = 0;
    7170        }
    7271        return NULL;
    … …  
    159158                        item->subinfo_type = SYSINFO_SUBINFO_NONE;
    160159                        return item;
    161                 } else {
     160                } else
    162161                        subtree = subtree->next;
    163                         i = 0;
    164                 }       
    165162        }
    166163

  • kernel/generic/src/udebug/udebug_ops.c

    re0b0c25 r137691a  
    8080static int _thread_op_begin(thread_t *t, bool being_go)
    8181{
    82         task_id_t taskid;
    8382        ipl_t ipl;
    84 
    85         taskid = TASK->taskid;
    8683
    8784        mutex_lock(&TASK->udebug.lock);
Note: See TracChangeset for help on using the changeset viewer.