Wenn ein Zeichen mehr im String ist, funktioniert das Programm nicht mehr

[Mathias O. (m-obi)]

Hallo,

wenn ich in meinem Programm

uart_puts_P("abcdefghijklmn\r\n");

stehen habe, funktioniert mein Programm (ich hab ein Blinken in der 
Hauptschleife zum testen).
Wenn ich aber

uart_puts_P("abcdefghijklmno\r\n");

schreibe, funktioniert es nicht mehr.

weiß jemand womit das zusammenhängen kann? Maximale Stringlänge?

[Holger (Gast)]

Die Wahrheit wird sich in uart_puts() finden ;)

[Hellseher (Gast)]

Zeile 274, da ist der Fehler. ;-)

[Test (Gast)]

Welche lib verwendst du ? Der Sende-Puffer wird zu klein sein (16byte)

[Oldie (Gast)]

Woraus leitet denn dein Programm das Blinken her?

Mein Programm blinkt nur nach der Frage "Bist du OK?\r\n",
allerdings denn auch nur, wenn es wirklich OK ist...

[Mathias O. (m-obi)]

Mein blinken ist einfach das toggeln einer LED. Das was wahrscheinlich 
jeder nimmt.

    while(1)
    {
//        eth_get_data();
        _delay_ms(500);
        PORTD ^= (1 << PD5);
    }

Ich nutze die Lib von Fleury.

Natürlich der Puffer. Das macht Sinn.

/** Size of the circular receive buffer, must be power of 2 */
#ifndef UART_RX_BUFFER_SIZE
#define UART_RX_BUFFER_SIZE 32
#endif
/** Size of the circular transmit buffer, must be power of 2 */
#ifndef UART_TX_BUFFER_SIZE
#define UART_TX_BUFFER_SIZE 32
#endif

Danke.

[NurEinGast (Gast)]

Hellseher schrieb:
> Zeile 274, da ist der Fehler. ;-)

Man soll immer erst in Zeile 42 schauen. Da liegt es meistens dran :-)

[Mathias O. (m-obi)]

NurEinGast schrieb:
> Hellseher schrieb:
>> Zeile 274, da ist der Fehler. ;-)
>
> Man soll immer erst in Zeile 42 schauen. Da liegt es meistens dran :-)

Er kennt wahrscheinlich den Film nicht ;)

[NurEinGast (Gast)]

In deiner Main-Schleife ist keine uart Ausgabe zu finden.  Zeig doch mal 
das ganze ....

[Hellseher (Gast)]

NurEinGast schrieb:
> Hellseher schrieb:
>> Zeile 274, da ist der Fehler. ;-)
>
> Man soll immer erst in Zeile 42 schauen. Da liegt es meistens dran :-)

BuSh, da hast du natürlich recht. Der erste Fehler ist in Zeile 42. Wie 
konnte ich das überhellsehen. ;-)

[DirkB (Gast)]

Mathias O. schrieb:
> Er kennt wahrscheinlich den Film nicht ;)

Es ist ein Hörspiel fürs Radio.
Dann kamen Bücher ...
... und eine Fernsehserie.

Den Film muss man aber nicht kennen.

[Ulrich H. (lurchi)]

Ein Zeichen mehr kann das Fass zum Überlaufen bringen, so das 1 byte 
beim RAM fehlt.

[Mathias O. (m-obi)]

Nagut, damit man es glaubt.

void main(void)
{
//    MCUCR |= (1 << ISC01);
//    GICR |= (1 << INT0);

    DDRD |= (1 << PD5)|(1 << PD6)|(1 << PD7);

    PORTD |= (1 << PD5);
    PORTD |= (1 << PD6);
    _delay_ms(200);
    PORTD &= ~(1 << PD5);
    PORTD &= ~(1 << PD6);
    _delay_ms(200);

    uart_init(UART_BAUD_SELECT(BAUD,F_CPU));

    uart_puts_P("--- Start ---\r\n");

    //Applikationen starten
//    stack_init();
//    testapp_init();

//    uart_puts_P("Stack initialisiert\r\n");
    uart_puts_P("abcdefghijklmno\r\n");

    sei();

    while(1)
    {
//        eth_get_data();
        _delay_ms(500);
        PORTD ^= (1 << PD5);
    }
}

Die Schleife hab ich nur wegen dem ominösen Blinken gepostet.

[Mathias O. (m-obi)]

DirkB schrieb:
> Mathias O. schrieb:
>> Er kennt wahrscheinlich den Film nicht ;)
>
> Es ist ein Hörspiel fürs Radio.
> Dann kamen Bücher ...
> ... und eine Fernsehserie.
>
> Den Film muss man aber nicht kennen.

Ach danke. Hatte immer im Hinterstübchen, dass es ein Film war. Stimmt 
das waren ja eher Romane.

[NurEinGast (Gast)]

Verdachtig wäre schon mal, dass Du die Interrupts erst freigibst nachdem 
Du schon 32 Zeichen in den Buffer geschoben hast ....

[Sina A. (sinapse)]

pack mal sei() vor die uart puts

[TomA (Gast)]

Hallo Mathias,

kann es sein, daß dein Compiler für jedes Zeichen im string zwei Bytes 
belegt?

Gruß. Tom

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Sina Anargo schrieb:
> pack mal sei() vor die uart puts
Nicht, dass das wirklich was ändern würde oder gar könnte. Der zu kleine 
Puffer wird trotzdem überlaufen. Denn sogar wenn sei() vorher käme, ist 
puts() idR. schneller als die serielle Schnitte.

TomA schrieb:
> kann es sein, daß dein Compiler für jedes Zeichen im string zwei Bytes
> belegt?
Kann, ist aber nicht so. Mit 32 ASCII Zeichen ist ein 32 Byte großer 
Puffer einfach voll.


Und die Antwort auf die Frage ist nicht in der Zeile 42, sondern die 
Antwort selbst ist 42. Das kommt aber nur im Buch so richtig rüber. In 
den Filmen geht das ein wenig unter...

[Bastler (Gast)]

Das geht auch mit kleinerem Puffer. Nur muß dieser auch geleert werden, 
sonst wartet man ewig, bis wieder Platz frei ist. Und zum Leeren Brauch 
man den Interrupt! Ein ganz simpler Deadlock.

[Mathias O. (m-obi)]

Bastler schrieb:
> Das geht auch mit kleinerem Puffer. Nur muß dieser auch geleert werden,
> sonst wartet man ewig, bis wieder Platz frei ist. Und zum Leeren Brauch
> man den Interrupt! Ein ganz simpler Deadlock.
Danke. sei vor den ersten puts gepackt und siehe da, schon funktioniert 
es auch mit langen Strings. Oh man oh man. Ab einer gewissen Uhrzeit 
sollte man ins Bett einfach gehen.

[chris (Gast)]

Lothar Miller schrieb:
> Nicht, dass das wirklich was ändern würde oder gar könnte. Der zu kleine
> Puffer wird trotzdem überlaufen. Denn sogar wenn sei() vorher käme, ist
> puts() idR. schneller als die serielle Schnitte.

Bei der Fleury-Lib ist puts() blockierend, wenn im Puffer kein Platz 
mehr ist. Und der Puffer wird im Interrupt geleert. Also ist sei() 
vorher nötig

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

chris schrieb:
> Bei der Fleury-Lib ist puts() blockierend, wenn im Puffer kein Platz
> mehr ist. Und der Puffer wird im Interrupt geleert. Also ist sei()
> vorher nötig
Gut, dass man den Sourcecode lesen kann.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

chris schrieb:
> Bei der Fleury-Lib
Du kannst aber gut raten...

> ist puts() blockierend, wenn im Puffer kein Platz
> mehr ist. Und der Puffer wird im Interrupt geleert. Also ist sei()
> vorher nötig
Das kann man sicher im Sourcecode nachsehen.

[chris (Gast)]

Lothar Miller schrieb:
> Du kannst aber gut raten...

ne, gut lesen :)

Mathias O. schrieb:
> Ich nutze die Lib von Fleury.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

chris schrieb:
> ne, gut lesen :)
Uuups...  :-]