]> Zhao Yanbai Git Server - minix.git/commitdiff
Remove useless symbol declarations from headers, make symbols local where possible...
authorKees van Reeuwijk <reeuwijk@few.vu.nl>
Thu, 22 Apr 2010 07:49:40 +0000 (07:49 +0000)
committerKees van Reeuwijk <reeuwijk@few.vu.nl>
Thu, 22 Apr 2010 07:49:40 +0000 (07:49 +0000)
12 files changed:
kernel/arch/i386/apic.c
kernel/arch/i386/apic.h
kernel/arch/i386/breakpoints.c
kernel/arch/i386/exception.c
kernel/arch/i386/i8259.c
kernel/arch/i386/proto.h
kernel/glo.h
kernel/interrupt.c
kernel/main.c
kernel/proto.h
kernel/system.c
kernel/watchdog.c

index 9d39aa969e39cc6c1076ca68e33fb420025e4d0d..25406a64260b7821c058e5649a63f8f29699dd7f 100644 (file)
 
 #define VERBOSE_APIC(x) x
 
-PUBLIC int reboot_type;
-PUBLIC int ioapic_enabled;
-PUBLIC u32_t ioapic_id_mask[8], lapic_id_mask[8];
+PRIVATE int reboot_type;
+PRIVATE int ioapic_enabled;
+PRIVATE u32_t ioapic_id_mask[8];
+PRIVATE u32_t lapic_id_mask[8];
 PUBLIC u32_t lapic_addr_vaddr;
 PUBLIC vir_bytes lapic_addr;
 PUBLIC vir_bytes lapic_eoi_addr;
-PUBLIC u32_t lapic_taskpri_addr;
-PUBLIC int bsp_lapic_id;
+PRIVATE u32_t lapic_taskpri_addr;
 
 PRIVATE volatile int probe_ticks;
 PRIVATE        u64_t tsc0, tsc1;
 PRIVATE        u32_t lapic_tctr0, lapic_tctr1;
 
-u8_t apicid2cpuid[MAX_NR_APICIDS+1];
-unsigned apic_imcrp;
-unsigned nioapics;
-unsigned nbuses;
-unsigned nintrs;
-unsigned nlints;
+/* FIXME: this is only accessed from assembly, never from C. Move to asm? */
+PUBLIC u8_t apicid2cpuid[MAX_NR_APICIDS+1];  /* Accessed from asm */
+
+PRIVATE unsigned apic_imcrp;
+PRIVATE unsigned nioapics;
+PRIVATE unsigned nbuses;
+PRIVATE unsigned nintrs;
+PRIVATE const unsigned nlints = 0;
 
 /*
  * FIXME this should be a cpulocal variable but there are some problems with
@@ -100,7 +102,7 @@ PRIVATE int calib_clk_handler(irq_hook_t * UNUSED(hook))
        return 1;
 }
 
-PUBLIC void apic_calibrate_clocks(void)
+PRIVATE void apic_calibrate_clocks(void)
 {
        u32_t lvtt, val, lapic_delta;
        u64_t tsc_delta;
@@ -213,19 +215,19 @@ PUBLIC void lapic_stop_timer(void)
        lapic_write(LAPIC_LVTTR, lvtt | APIC_LVTT_MASK);
 }
 
-PUBLIC void lapic_microsec_sleep(unsigned count)
+PRIVATE void lapic_microsec_sleep(unsigned count)
 {
        lapic_set_timer_one_shot(count);
        while (lapic_read (LAPIC_TIMER_CCR));
 }
 
-PUBLIC  u32_t lapic_errstatus (void)
+PRIVATE  u32_t lapic_errstatus(void)
 {
        lapic_write(LAPIC_ESR, 0);
        return lapic_read(LAPIC_ESR);
 }
 
-PUBLIC void lapic_disable(void)
+PRIVATE void lapic_disable(void)
 {
        /* Disable current APIC and close interrupts from PIC */
        u32_t val;
@@ -432,7 +434,7 @@ PRIVATE void lapic_set_dummy_handlers(void)
 #endif
 
 /* Build descriptors for interrupt gates in IDT. */
-PUBLIC void apic_idt_init(const int reset)
+PRIVATE void apic_idt_init(const int reset)
 {
        /* Set up idt tables for smp mode.
         */
index 45f8c9cb464bfd1893a601add2d1c7054da084b3..82bffa97c7cae511f8ac219d3e36e03ff6c368f4 100644 (file)
 
 #include "kernel/kernel.h"
 
-EXTERN int ioapic_enabled;
 EXTERN vir_bytes lapic_addr;
 EXTERN u32_t lapic_eoi_addr;
-EXTERN u32_t lapic_taskpri_addr;
-EXTERN int bsp_lapic_id;
 
 #define MAX_NR_IOAPICS                 32
 #define MAX_NR_BUSES                   32
@@ -104,39 +101,17 @@ EXTERN int bsp_lapic_id;
 #define MAX_NR_LCLINTS                 2
 
 EXTERN u8_t apicid2cpuid[MAX_NR_APICIDS+1];
-EXTERN unsigned apic_imcrp;
-EXTERN unsigned nioapics;
-EXTERN unsigned nbuses;
-EXTERN unsigned nintrs;
-EXTERN unsigned nlints;
-
-EXTERN u32_t ioapic_id_mask[8];
-EXTERN u32_t lapic_id_mask[8];
+
 EXTERN u32_t lapic_addr_vaddr; /* we remember the virtual address here until we
                                  switch to paging */
 
-_PROTOTYPE (void calc_bus_clock, (void));
-_PROTOTYPE (u32_t lapic_errstatus, (void));
 /*
 _PROTOTYPE (u32_t ioapic_read, (u32_t addr, u32_t offset));
 _PROTOTYPE (void ioapic_write, (u32_t addr, u32_t offset, u32_t data));
 _PROTOTYPE (void lapic_eoi, (void));
 */
-_PROTOTYPE (void lapic_microsec_sleep, (unsigned count));
-_PROTOTYPE (void smp_ioapic_unmask, (void));
-_PROTOTYPE (void ioapic_disable_irqs, (u32_t irq));
-_PROTOTYPE (void ioapic_enable_irqs, (u32_t irq));
-_PROTOTYPE (u32_t ioapic_irqs_inuse, (void));
-_PROTOTYPE (void smp_recv_ipi, (int arg));
-_PROTOTYPE (void ioapic_config_pci_irq, (u32_t data));
 
 _PROTOTYPE (int lapic_enable, (void));
-_PROTOTYPE (void lapic_disable, (void));
-
-_PROTOTYPE (void ioapic_disable_all, (void));
-_PROTOTYPE (int ioapic_enable_all, (void));
-
-_PROTOTYPE(void apic_idt_init, (int reset));
 
 _PROTOTYPE(int apic_single_cpu_init, (void));
 
index 189cb26c3fb47d5cf6b50268243cf5324785de32..2863a90f94da50f0083a9910be25c41388517a14 100644 (file)
@@ -3,7 +3,7 @@
 
 #include "debugreg.h"
 
-int breakpoint_set(phys_bytes linaddr, int bp, const int flags)
+PRIVATE int breakpoint_set(phys_bytes linaddr, int bp, const int flags)
 {
        u32_t dr7, dr7flags;
        
index 84cc18b3de258e03815fcf601c6d8142b6a55ef7..6a87223d59d70cf518968ce1c5c99fa3bdf81cda 100644 (file)
@@ -11,7 +11,7 @@
 #include "kernel/proc.h"
 #include "kernel/proto.h"
 
-extern int catch_pagefaults;
+extern int catch_pagefaults = 0;
 
 void pagefault( struct proc *pr,
                struct exception_frame * frame,
index cc0bc419edf42dceb39a85c6a53e7e0033aeb60c..a2b0b6c8bfc99868a246c82fcc20efc5cf4f5707 100644 (file)
 #define ICW4_PC_AEOI_SLAVE   0x0B /* not SFNM, buffered, auto EOI, 8086 */
 #define ICW4_PC_AEOI_MASTER  0x0F /* not SFNM, buffered, auto EOI, 8086 */
 
+/*===========================================================================*
+ *                             intr_disabled                                *
+ *===========================================================================*/
+PRIVATE int intr_disabled(void)
+{
+       if(!(read_cpu_flags() & X86_FLAG_I))
+               return 1;
+       return 0;
+}
+
 /*===========================================================================*
  *                             intr_init                                    *
  *===========================================================================*/
@@ -73,16 +83,6 @@ PUBLIC int intr_init(const int mine, const int auto_eoi)
   return OK;
 }
 
-/*===========================================================================*
- *                             intr_disabled                                *
- *===========================================================================*/
-PUBLIC int intr_disabled(void)
-{
-       if(!(read_cpu_flags() & X86_FLAG_I))
-               return 1;
-       return 0;
-}
-
 PUBLIC void irq_8259_unmask(const int irq)
 {
        const unsigned ctl_mask = irq < 8 ? INT_CTLMASK : INT2_CTLMASK;
index e421510ce1279937efdfd882ef36f595c8422163..041e9d447335f6137561c36403a50ef5bed1abb3 100644 (file)
@@ -170,8 +170,6 @@ _PROTOTYPE(void __user_copy_msg_pointer_failure, (void));
 #define BREAKPOINT_FLAG_MODE_LOCAL     (1 << 4)
 #define BREAKPOINT_FLAG_MODE_GLOBAL    (2 << 4)
 
-_PROTOTYPE(int breakpoint_set, (phys_bytes linaddr, int index, int flags));
-
 /* functions defined in architecture-independent kernel source. */
 #include "kernel/proto.h"
 
index d3042d3dd359b09f384f7791e4637cf9a7193326..0868c461a2c1ec21c52b110fe94b8443b034ae0d 100644 (file)
@@ -39,13 +39,12 @@ EXTERN int irq_use;                         /* map of all in-use irq's */
 EXTERN u32_t system_hz;                                /* HZ value */
 
 /* Miscellaneous. */
-EXTERN reg_t mon_ss, mon_sp;           /* boot monitor stack */
+EXTERN reg_t mon_sp;                   /* boot monitor stack */
 EXTERN int mon_return;                 /* true if we can return to monitor */
 EXTERN int do_serial_debug;
 EXTERN time_t boottime;
 EXTERN char params_buffer[512];                /* boot monitor parameters */
 EXTERN int minix_panicing;
-EXTERN int locklevel;
 EXTERN char fpu_presence;
 EXTERN char osfxsr_feature;             /* FXSAVE/FXRSTOR instructions support (SSEx) */
 EXTERN int verboseboot;                        /* verbose boot, init'ed in cstart */
index 00d0310da06dfefdf65111658041e13d914c91bc..d315c19e675bc161bd0bf4733fb300c197fb9c4d 100644 (file)
@@ -21,7 +21,8 @@
 #include "hw_intr.h"
 
 /* number of lists of IRQ hooks, one list per supported line. */
-PUBLIC irq_hook_t* irq_handlers[NR_IRQ_VECTORS] = {0};
+PRIVATE irq_hook_t* irq_handlers[NR_IRQ_VECTORS] = {0};
+
 /*===========================================================================*
  *                             put_irq_handler                              *
  *===========================================================================*/
index 500f685730a82d0341de182f5fe37c7665f8e46b..ffee52cf4d4e4f7eae12d929645418a6ea1645bd 100644 (file)
@@ -25,7 +25,7 @@ FORWARD _PROTOTYPE( void announce, (void));
 /*===========================================================================*
  *                             main                                         *
  *===========================================================================*/
-PUBLIC void main()
+PUBLIC void main(void)
 {
 /* Start the ball rolling. */
   struct boot_image *ip;       /* boot image pointer */
index da4e8c25a4683bdab33fe46e70165cab0fb0a28f..2cd5befc67bea199b81e70b547a9313c3cebdb78 100644 (file)
@@ -69,7 +69,6 @@ _PROTOTYPE( void system_init, (void)                                  );
        umap_local(proc_addr(proc_nr), D, (vir_addr), (bytes))
 _PROTOTYPE( phys_bytes umap_grant, (struct proc *, cp_grant_id_t, vir_bytes));
 _PROTOTYPE( void clear_endpoint, (struct proc *rc)                     );
-_PROTOTYPE( void clear_ipc, (struct proc *rc)                          );
 _PROTOTYPE( void clear_ipc_refs, (struct proc *rc, int caller_ret)     );
 _PROTOTYPE( phys_bytes umap_bios, (vir_bytes vir_addr, vir_bytes bytes));
 _PROTOTYPE( void kernel_call_resume, (struct proc *p));
@@ -136,8 +135,6 @@ _PROTOTYPE( void alloc_segments, (struct proc *rp)                      );
 _PROTOTYPE( void vm_init, (struct proc *first)                         );
 _PROTOTYPE( phys_bytes umap_local, (register struct proc *rp, int seg,
        vir_bytes vir_addr, vir_bytes bytes));
-_PROTOTYPE( void cp_mess, (int src,phys_clicks src_clicks,
-        vir_bytes src_offset, phys_clicks dst_clicks, vir_bytes dst_offset));
 _PROTOTYPE( phys_bytes umap_remote, (const struct proc* rp, int seg,
         vir_bytes vir_addr, vir_bytes bytes)                           );
 _PROTOTYPE( phys_bytes umap_virtual, (struct proc* rp,
@@ -148,7 +145,6 @@ _PROTOTYPE( int vm_phys_memset, (phys_bytes source, u8_t pattern,
 _PROTOTYPE( vir_bytes alloc_remote_segment, (u32_t *, segframe_t *,
         int, phys_bytes, vir_bytes, int));
 _PROTOTYPE( int intr_init, (int, int)                                  );
-_PROTOTYPE( int intr_disabled, (void)                                  );
 _PROTOTYPE( void halt_cpu, (void)                                      );
 _PROTOTYPE( void arch_init, (void)                                     );
 _PROTOTYPE( void ser_putc, (char)                                              );
index e49a20009cff61e0ba0c5b31568670c4e9bc16b0..5510112c325f7368732ff64406dbc96cf968f825 100644 (file)
@@ -49,8 +49,8 @@
  * because the dummy is declared extern. If an illegal call is given, the 
  * array size will be negative and this won't compile. 
  */
-PUBLIC int (*call_vec[NR_SYS_CALLS])(struct proc * caller, message *m_ptr);
-char *callnames[NR_SYS_CALLS];
+PRIVATE int (*call_vec[NR_SYS_CALLS])(struct proc * caller, message *m_ptr);
+PRIVATE char *callnames[NR_SYS_CALLS];
 
 #define map(call_nr, handler) \
     {extern int dummy[NR_SYS_CALLS>(unsigned)(call_nr-KERNEL_CALL) ? 1:-1];} \
@@ -473,39 +473,12 @@ vir_bytes bytes;                /* size */
        return ret;
 }
 
-/*===========================================================================*
- *                              clear_endpoint                              *
- *===========================================================================*/
-PUBLIC void clear_endpoint(rc)
-register struct proc *rc;              /* slot of process to clean up */
-{
-  if(isemptyp(rc)) panic("clear_proc: empty process: %d",  rc->p_endpoint);
-
-  /* Make sure that the exiting process is no longer scheduled. */
-  RTS_SET(rc, RTS_NO_ENDPOINT);
-  if (priv(rc)->s_flags & SYS_PROC)
-  {
-       priv(rc)->s_asynsize= 0;
-  }
-
-  /* If the process happens to be queued trying to send a
-   * message, then it must be removed from the message queues.
-   */
-  clear_ipc(rc);
-
-  /* Likewise, if another process was sending or receive a message to or from 
-   * the exiting process, it must be alerted that process no longer is alive.
-   * Check all processes. 
-   */
-  clear_ipc_refs(rc, EDEADSRCDST);
-
-}
-
 /*===========================================================================*
  *                              clear_ipc                                   *
  *===========================================================================*/
-PUBLIC void clear_ipc(rc)
-register struct proc *rc;              /* slot of process to clean up */
+PRIVATE void clear_ipc(
+  register struct proc *rc     /* slot of process to clean up */
+)
 {
 /* Clear IPC data for a given process slot. */
   struct proc **xpp;                   /* iterate over caller queue */
@@ -531,6 +504,34 @@ register struct proc *rc;          /* slot of process to clean up */
   rc->p_rts_flags &= ~RTS_RECEIVING;
 }
 
+/*===========================================================================*
+ *                              clear_endpoint                              *
+ *===========================================================================*/
+PUBLIC void clear_endpoint(rc)
+register struct proc *rc;              /* slot of process to clean up */
+{
+  if(isemptyp(rc)) panic("clear_proc: empty process: %d",  rc->p_endpoint);
+
+  /* Make sure that the exiting process is no longer scheduled. */
+  RTS_SET(rc, RTS_NO_ENDPOINT);
+  if (priv(rc)->s_flags & SYS_PROC)
+  {
+       priv(rc)->s_asynsize= 0;
+  }
+
+  /* If the process happens to be queued trying to send a
+   * message, then it must be removed from the message queues.
+   */
+  clear_ipc(rc);
+
+  /* Likewise, if another process was sending or receive a message to or from 
+   * the exiting process, it must be alerted that process no longer is alive.
+   * Check all processes. 
+   */
+  clear_ipc_refs(rc, EDEADSRCDST);
+
+}
+
 /*===========================================================================*
  *                            clear_ipc_refs                                *
  *===========================================================================*/
index c10b4d6d4c9313318eaef52ec7e0a95f36d2181d..489532e36fac0bab70f7c47bd4a95246c3c959d1 100644 (file)
@@ -6,7 +6,7 @@
 
 #include "watchdog.h"
 
-unsigned watchdog_local_timer_ticks;
+unsigned watchdog_local_timer_ticks = 0U;
 struct arch_watchdog *watchdog;
 int watchdog_enabled;