Changeset 80d2bdb in mainline for generic


Ignore:
Timestamp:
2005-12-15T16:10:19Z (20 years ago)
Author:
Ondrej Palkovsky <ondrap@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
b4cad8b2
Parents:
7dd2561
Message:

We are now almost -Wall clean.

  • redefined atomic_t
  • deleted many, many unused variables
  • some minor code cleanups found using compiler warning.
Location:
generic
Files:
13 edited

Legend:

Unmodified
Added
Removed

  • generic/include/cpu.h

    r7dd2561 r80d2bdb  
    5454
    5555        #ifdef CONFIG_SMP
    56         int kcpulbstarted;
     56        atomic_t kcpulbstarted;
    5757        waitq_t kcpulb_wq;
    5858        #endif /* CONFIG_SMP */

  • generic/include/debug.h

    r7dd2561 r80d2bdb  
    3434#include <arch.h>
    3535
    36 #define CALLER       ((__address *)__builtin_return_address(0))
     36#define CALLER       ((__address)__builtin_return_address(0))
    3737
    3838#ifndef HERE

  • generic/include/synch/spinlock.h

    r7dd2561 r80d2bdb  
    3333#include <typedefs.h>
    3434#include <preemption.h>
     35#include <arch/atomic.h>
    3536
    3637#ifdef CONFIG_SMP
    … …  
    3940        char *name;
    4041#endif
    41         int val;
     42        atomic_t val;
    4243};
    4344
    … …  
    5657        spinlock_t slname = {           \
    5758                .name = #slname,        \
    58                 .val = 0                \
     59                .val = { 0 }            \
    5960        }
    6061#else
    6162#define SPINLOCK_INITIALIZE(slname)     \
    6263        spinlock_t slname = {           \
    63                 .val = 0                \
     64                .val = { 0 }            \
    6465        }
    6566#endif

  • generic/src/console/cmd.c

    r7dd2561 r80d2bdb  
    311311{
    312312        link_t *cur;
    313         ipl_t ipl;
    314313
    315314        spinlock_lock(&cmd_lock);
    … …  
    340339{
    341340        link_t *cur;
    342         ipl_t ipl;
    343341
    344342        spinlock_lock(&cmd_lock);
    … …  
    369367int cmd_symaddr(cmd_arg_t *argv)
    370368{
    371         __address symaddr;
    372         char *symbol;
    373 
    374369        symtab_print_search(argv->buffer);
    375370       
    … …  
    511506int cmd_set4(cmd_arg_t *argv)
    512507{
    513         char *symbol;
    514508        __u32 *addr ;
    515509        __u32 arg1 = argv[1].intval;

  • generic/src/console/kconsole.c

    r7dd2561 r80d2bdb  
    9292int cmd_register(cmd_info_t *cmd)
    9393{
    94         ipl_t ipl;
    9594        link_t *cur;
    9695       
    … …  
    161160        int namelen = strlen(name);
    162161        const char *curname;
    163         char *foundsym = NULL;
    164         int foundpos = 0;
    165162
    166163        spinlock_lock(&cmd_lock);
    … …  
    470467        cmd_info_t *cmd = NULL;
    471468        link_t *cur;
    472         ipl_t ipl;
    473469        int i;
    474470       

  • generic/src/debug/symtab.c

    r7dd2561 r80d2bdb  
    6767        char *curname;
    6868        int i,j;
    69         char *foundsym = NULL;
    70         int foundpos = 0;
    7169        int colonoffset = -1;
    7270

  • generic/src/main/kinit.c

    r7dd2561 r80d2bdb  
    6969void kinit(void *arg)
    7070{
     71        thread_t *t;
     72        int i;
     73#ifdef CONFIG_USERSPACE
    7174        vm_t *m;
    7275        vm_area_t *a;
    7376        task_t *u;
    74         thread_t *t;
    75         int i;
     77#endif
    7678
    7779        interrupts_disable();

  • generic/src/main/main.c

    r7dd2561 r80d2bdb  
    126126 */
    127127static void main_bsp_separated_stack(void);
     128#ifdef CONFIG_SMP
    128129static void main_ap_separated_stack(void);
     130#endif
    129131
    130132/** Bootstrap CPU main kernel routine

  • generic/src/mm/vm.c

    r7dd2561 r80d2bdb  
    189189void vm_install(vm_t *m)
    190190{
    191         link_t *l;
    192191        ipl_t ipl;
    193192       

  • generic/src/proc/scheduler.c

    r7dd2561 r80d2bdb  
    491491        ipl = interrupts_disable();
    492492        spinlock_lock(&CPU->lock);
    493         count = nrdy / config.cpu_active;
     493        count = atomic_get(&nrdy) / config.cpu_active;
    494494        count -= CPU->nrdy;
    495495        spinlock_unlock(&CPU->lock);
    … …  
    619619         * Tell find_best_thread() to wake us up later again.
    620620         */
    621         CPU->kcpulbstarted = 0;
     621        atomic_set(&CPU->kcpulbstarted,0);
    622622        goto loop;
    623623}

  • generic/src/proc/thread.c

    r7dd2561 r80d2bdb  
    9696{
    9797        THREAD = NULL;
    98         nrdy = 0;
     98        atomic_set(&nrdy,0);
    9999}
    100100
    … …  
    112112        runq_t *r;
    113113        ipl_t ipl;
    114         int i, avg, send_ipi = 0;
     114        int i, avg;
    115115
    116116        ipl = interrupts_disable();
    … …  
    136136
    137137        atomic_inc(&nrdy);
    138         avg = nrdy / config.cpu_active;
     138        avg = atomic_get(&nrdy) / config.cpu_active;
    139139
    140140        spinlock_lock(&cpu->lock);

  • generic/src/synch/rwlock.c

    r7dd2561 r80d2bdb  
    226226                                panic("_mutex_lock_timeout()==ESYNCH_OK_ATOMIC\n");
    227227                                break;
    228                         dafault:
     228                        default:
    229229                                panic("invalid ESYNCH\n");
    230230                                break;

  • generic/src/synch/spinlock.c

    r7dd2561 r80d2bdb  
    4646void spinlock_initialize(spinlock_t *sl, char *name)
    4747{
    48         sl->val = 0;
     48        atomic_set(&sl->val, 0);
    4949#ifdef CONFIG_DEBUG_SPINLOCK
    5050        sl->name = name;
    … …  
    152152void spinlock_unlock(spinlock_t *sl)
    153153{
    154         ASSERT(sl->val != 0);
     154        ASSERT(atomic_get(&sl->val) != 0);
    155155
    156156        /*
    … …  
    159159        CS_LEAVE_BARRIER();
    160160       
    161         sl->val = 0;
     161        atomic_set(&sl->val,0);
    162162        preemption_enable();
    163163}
Note: See TracChangeset for help on using the changeset viewer.