It is mostly useless and probably confusing.  I'm not even sure if the
linker had to put it directly after .data, or if it could put it wherever
it wanted before the .bss, since it was not inside the .data section.

Signed-off-by: Barret Rhoden <[email protected]>
---
 kern/arch/riscv/kernel.ld | 2 --
 kern/arch/x86/kernel64.ld | 2 --
 kern/include/ns.h         | 3 ---
 kern/src/monitor.c        | 3 +--
 4 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/kern/arch/riscv/kernel.ld b/kern/arch/riscv/kernel.ld
index c2c92057bc60..bfc87964880f 100644
--- a/kern/arch/riscv/kernel.ld
+++ b/kern/arch/riscv/kernel.ld
@@ -50,8 +50,6 @@ SECTIONS
                *(.sdata)
        }
 
-       PROVIDE(edata = .);
-
        .bss : {
                PROVIDE(__start_bss = .);
                *(.bss)
diff --git a/kern/arch/x86/kernel64.ld b/kern/arch/x86/kernel64.ld
index f37a095d7e50..d56eef300052 100644
--- a/kern/arch/x86/kernel64.ld
+++ b/kern/arch/x86/kernel64.ld
@@ -53,8 +53,6 @@ SECTIONS
                *(.data)
        }
 
-       PROVIDE(edata = .);
-
        .bss : {
                PROVIDE(__start_bss = .);
                *(.bss)
diff --git a/kern/include/ns.h b/kern/include/ns.h
index b55d7075b241..166158472114 100644
--- a/kern/include/ns.h
+++ b/kern/include/ns.h
@@ -70,9 +70,6 @@ static inline uint32_t getcallerpc(void *v)
        return 0;
 }
 
-extern char etext[];
-extern char edata[];
-extern char end[];
 extern int getfields(char *unused_char_p_t, char **unused_char_pp_t,
                                         int unused_int, int, char *);
 extern int tokenize(char *unused_char_p_t, char **unused_char_pp_t, int);
diff --git a/kern/src/monitor.c b/kern/src/monitor.c
index 048ef3b59900..2b2a075a9b57 100644
--- a/kern/src/monitor.c
+++ b/kern/src/monitor.c
@@ -93,12 +93,11 @@ int mon_ps(int argc, char** argv, struct hw_trapframe 
*hw_tf)
 
 int mon_kerninfo(int argc, char **argv, struct hw_trapframe *hw_tf)
 {
-       extern char _start[], etext[], edata[], end[];
+       extern char _start[], etext[], end[];
 
        cprintf("Special kernel symbols:\n");
        cprintf("  _start %016x (virt)  %016x (phys)\n", _start, 
(uintptr_t)(_start - KERNBASE));
        cprintf("  etext  %016x (virt)  %016x (phys)\n", etext, 
(uintptr_t)(etext - KERNBASE));
-       cprintf("  edata  %016x (virt)  %016x (phys)\n", edata, 
(uintptr_t)(edata - KERNBASE));
        cprintf("  end    %016x (virt)  %016x (phys)\n", end, (uintptr_t)(end - 
KERNBASE));
        cprintf("Kernel executable memory footprint: %dKB\n",
                (uint32_t)(end-_start+1023)/1024);
-- 
2.6.0.rc2.230.g3dd15c0

-- 
You received this message because you are subscribed to the Google Groups 
"Akaros" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to