Thema: GCC Funktionen in Header "undefined reference"

[OS_3000]

Ich habe das Problem, dass ich einige Methoden eines Headers einfach nicht aufrufen kann. 

Zb. in meiner "AsmAcess.h":

Code: [Auswählen]

#ifndef ASMACCESS
#define ASMACCESS

#include <stdint.h>

inline uint8_t inb(uint16_t Port)
{
    unsigned char Data = 0;
    asm volatile ("inb %1, %0" : : "a" (Data), "Nd" (Port));
    return Data;
}
inline void outb(uint16_t Port, uint8_t Data)
{
    asm volatile ("outb %0, %1" : : "a" (Data), "Nd" (Port));
}
inline void halt()
{
    StopPoint:
    asm volatile ("cli");
    asm volatile ("hlt");
    goto StopPoint;
}

#endifDiese Datei binde ich dann zb. in meinem Kernel ein:

Code: [Auswählen]

#include "kernel\console.h"
#include "kernel\asmaccess.h"

void main()
{
    Console_ClearScreen();
    Console_WriteLine("Hallo!");
    Console_WriteLine("Ich heiße Compi!");
    Console_WriteLine("Ich bin ein Computer");
    Console_WriteLine("und lasse hier diesen sinnfreien Text anzeigen.");
    halt();
}
Wenn ich das Ganze dann mit folgendem Code kompilieren will:

Code: [Auswählen]

del obj\*.o
SET C_COMPILE=i586-elf-gcc -m32 -ffreestanding -Wall -nostdinc -std=gnu99 -I stdlib
SET ASM_COMPILE=nasm -f elf

%ASM_COMPILE% -o obj\start_S.o start.S
%C_COMPILE% -o obj\kernel_c.o -c kernel.c

%C_COMPILE% -o obj\beep_c.o -c kernel\beep.c
%C_COMPILE% -o obj\console_c.o -c kernel\console.c

%C_COMPILE% -o obj\rand_c.o -c stdlib\rand.c

i586-elf-ld -T linkconf.ld -o obj\kernel.bin obj\*.o -Ttext=0x100000

copy obj\kernel.bin cd\kernel.bin
mkisofs -R -J -b boot/grub/stage2_eltorito -no-emul-boot -boot-load-size 4 -boot-info-table -o cdrom.iso cd/Kommt beim Linken immer diese Fehlermeldung:

Code: [Auswählen]

obj\kernel.o: In Funktion "[i]main[/i]":
kernel.c(.text+0x57): undefined reference to "halt"
Ich bin absolut ratlos, was das Problem auslöst.
Eigentlich müsste der Präprozessor "#include "kernel\asmaccess.h"" mit dem Code in der Datei ersetzen und abschliessend das Ganze inlinen.
Doch weshalb tut er das nicht? 

Ich hoffe ihr könnt mir helfen. 

[rizor]

Der Fehler ist komisch, aber nimm besser mal das goto raus. Du hast den Punkt nirgendwo definiert und hlt; stoppt die CPU.

[OS_3000]

Hat leider nicht geholfen. 

Der Fehler tritt übrigens auch bei dem Aufruf der In\Out-Funktion auf.

Die "MoveCurser"-Funktion der "Console"-Datei verursacht so sogar 8 Linkerfehler.

Code: [Auswählen]

void Console_MoveCurser()
{
    if (Console_ShowCurser)
    {
        //Curser auf Position setzen
        outb(0x3D4, 14);
        outb(0x3D5, (uint8_t)(Console_Curser >> 8));
        outb(0x3D4, 15);
        outb(0x3D5, (uint8_t)Console_Curser);
    }
    else
    {
        //Curser aus dem Bildschirm setzen
        outb(0x3D4, 14);
        outb(0x3D5, (uint8_t)(Console_CharCount >> 8));
        outb(0x3D4, 15);
        outb(0x3D5, (uint8_t)Console_CharCount);
    }
}

[LittleFox]

Hi,

einbinden tut der die header aber, oder? Das würde ich mal nachprüfen.
Schrein einfach mal ein "#error bla" in einen der header, dann können wir das schonmal ausschließen.

Grüße, LittleFox

--
Handy ..

[micha]

Diese Datei binde ich dann zb. in meinem Kernel ein:

Code: [Auswählen]

#include "kernel\console.h"
#include "kernel\asmaccess.h"

void main()
{
    Console_ClearScreen();
    Console_WriteLine("Hallo!");
    Console_WriteLine("Ich heiße Compi!");
    Console_WriteLine("Ich bin ein Computer");
    Console_WriteLine("und lasse hier diesen sinnfreien Text anzeigen.");
    halt();
}

Liegt das vielleicht in deinem Kernelverzeichnis?

du solltest dann das nehmen:

Code: [Auswählen]

#include "console.h"
#include "asmaccess.h"
statt dem:

#include "kernel\console.h"
#include "kernel\asmaccess.h"

[OS_3000]

Bei "#Error Ich funktioniere nicht" kommt der Fehler beim kompilieren von "Console.c" und "Kernel.c" wie es sein soll.

Das das wegen dem Kernelverzeichnis streikt, schliess ich mal aus, da "kernel\Console.h" problemlos eingebunden werden kann.

Ich hab mal das Ganze angehängt, vielleicht hilfts bei der Fehlersuche. 
(Die Datei, ist eine 7z-Datei, da ich solche aber leider nicht hochladen darf, habe ich noch ein ".txt" drangehängt. Zum entpacken einfach wieder ".7z" daraus machen.  )

[MNemo]

ich kann dir leider nicht sagen warum es so nicht funktioniert, aber static inline hilft.

[OS_3000]

Ah, tausend Dank!
Kannst du mir auch noch sagen, was dieses "static" jetzt eigentlich bewirkt, also wozu es gut ist?

Bedauerlicherweiße kommt jetzt beim kompilieren von "Console.c" ein anderer Fehler:

Code: [Auswählen]

Temp\cckJeKaN.s: Assembler messages:
Temp\cckJeKaN.s:30: Error suffix or operands invalid for "out"
Was stimmt denn jetzt schonwieder nicht?
Meine Funktionen heißen schliesslich immer "outb".

[MNemo]

static sagt das eine funktion nur in der aktuellen datei benutzt wird. Wenn du zwei Dateien hast die asmaccess.h includen hättest du ohne static die halt funktion zweimal definiert und es käme zu konfliketen beim linken. Static behebt das.

@Assembeler error messages
Es scheint als wäre dein uint8_t kein echter 8-Bit integer. So dass dir gcc z.B.

Code: [Auswählen]

outb %eax, %dxstatt

Code: [Auswählen]

outb %al, %dxbaut. Dann passt b (= 8bit) nicht zu eax (=32bit).

[OS_3000]

Ich verstehe zwar nicht, weshalb das jetzt kein echter Integer sein soll,
aber auf jedenfall geht es jetzt. 
Noch mal vielen Dank. 

[MNemo]

Ein integer ist es schon, nur scheint er größer als 8-Bit zu sein.

Code: [Auswählen]

if (sizeof(uint8_t) != 1) {
  puts("uint8_t hält _nicht_ was er verspricht!");
}

Edit: Ich hab mal nachgeguckt. Dein uint8_t ist OK. aber dein uint16_t nicht. Es lag also an %edx statt %dx. Da du "Nd" hast ist es übrigens nicht ratsam das als "%%dx" hart zu codieren sondern uint16_t zu fixen. (short statt int) bzw. ich nehme:

Code: [Auswählen]

typedef unsigned uint8_t __attribute__((__mode__(__QI__)));
typedef unsigned uint16_t __attribute__((__mode__(__HI__)));
typedef unsigned uint32_t __attribute__((__mode__(__SI__)));
typedef unsigned uint64_t __attribute__((__mode__(__DI__)));

[micha]

nochmal zu dem outb:
deine funktionen in asmaccess.h sind falsch. Sie könnten (sollten) so aussehen:

Code: [Auswählen]

static inline void outb(uint16_t port, uint8_t data){
   asm("outb %0, %1" : : "a" (data), "Nd" (port));
}
static inline uint8_t inb(uint16_t port){
  unsigned char result = 0;
  asm("inb %1, %0" : "=a" (result) : "Nd" (port));
  return result;
}

[OS_3000]

Ja, stimmt, hab Blödsinn geschrieben.
Meinte natürlich, dass es eben kein 8-Bit Integer ist. 

Das mit dem "short" hab ich jetzt bereits selbst bemerkt, als die "Console.c" nicht so recht funktioniert hat.
Diese Definition hatte ich aus dem Internet.
Konnte ja nicht wissen, dass sie noch aus der 16-Bit-Zeit stammt. 

Code: [Auswählen]

typedef unsigned uint8_t __attribute__((__mode__(__QI__)));
typedef unsigned uint16_t __attribute__((__mode__(__HI__)));
typedef unsigned uint32_t __attribute__((__mode__(__SI__)));
typedef unsigned uint64_t __attribute__((__mode__(__DI__)));Wenn das besser ist, mach ichs so.

EDIT:
Wo ist jetzt der Fehler in "outb"?
Der einzige Unterschied ist in "inb" das Istgleichzeichen vor dem "a".

[MNemo]

Dein outb ist korrekt.

Ich benutzte diese (u)intNN_t Definitionen weil C mir z.B. nicht garantiert das long 32-bit ist. Es ist halt gcc-only (und auch nur neuere).  Aber ich glaub es geht auch irgendwie die stdint.h von gcc zu nehmen, das wäre dann noch besser.

[micha]

ich meinte ja eingentlich auch nur das inb.
Der Assembler hat mir nur Angezeit: Suffix or operands invalid for out
mit den Funktionen kommt das nicht
deshalb hab ich dir funktionierende Funktionen nochmal geschrieben

[OS_3000]

Eigentlich dachte ich, jetzt gehts endlich, als das heute wieder los ging. 
Jetzt kommt dauernd "undefined Reference to __moddi3", "undefined Reference to __divdi3", "undefined Reference to __umoddi3" und "undefined Reference to __udivdi3".

[LittleFox]

wenn ich mich richtig erinnere haben die Funktionen etwas mit uint64/int64 zu tun.
Ist vielleicht schonmal ein Ansatzpunkt.

Grüße

[OS_3000]

Hm...
Könnte gut sein, da der Fehler in "Console_WriteInt64" auftritt.
Mal noch der Quellcode:

Code: [Auswählen]

void Console_WriteInt64(sint64_t Number)
{
    char String[21];
    //Negative Zahlen extra behandeln
    bool Negative = Number < 0;
    if (Negative)
    {
        String[0] = '-';
        Number = 0 - Number;
    }
   
    //Nummer in Ziffern aufteilen
    char TempDigits[10];
    int Index = 0;
    while(Number > 0)
    {
        TempDigits[Index++] = Number % 10;
        Number /= 10;
    }
   
    //String zusammenbauen
    int StringIndex = Negative ? 1 : 0;
    for (Index--; Index >= 0; Index--)
    {
        String[StringIndex] = TempDigits[Index] + 0x30;
        StringIndex++;
    }
    String[StringIndex] = 0;
    Console_Write(String);
}
EDIT: Weshalb gibts hier eigentlich kein Syntaxhighlighting?
So ist Quellcode fast unleserlich.

[kevin]

Du musst die libgcc dazulinken.

[OS_3000]

Wo finde ich das bzw. wie linke ich das?
Hab eben ein wenig im Internet rumgeschaut, aber nichts dazu gefunden.