Problem mit FIFO Artikel

Gast #3338697
Lesenswert?
• ▲
▼
Hallo,

ich bin gerade am Nachprogrammieren und Anpassen des Codes aus dem 
µC.net-Artikel FIFO. Ich habe dabei gerade ein Verständnisproblem 
und komme einfach nicht drauf wo mein Denkfehler liegt.
Es geht um folgende Funktion:
1
uint8_t BufferIn(uint8_t byte)
2
{
3
  //if (buffer.write >= BUFFER_SIZE)
4
  //  buffer.write = 0; // erhöht sicherheit
5
 
6
  if (buffer.write + 1 == buffer.read || buffer.read == 0 && buffer.write + 1 == BUFFER_SIZE)
7
    return FAIL; // voll
8
 
9
  buffer.data[buffer.write] = byte;
10
 
11
  buffer.write = buffer.write + 1;
12
  if (buffer.write >= BUFFER_SIZE)
13
    buffer.write = 0;
14
  return SUCCESS;
15
 
16
}
und hier konkret um die Bedingung:
1
buffer.read == 0 && buffer.write + 1 == BUFFER_SIZE
Mal angenommen BUFFER_SIZE hat den Wert 3 und die Struktur ist wie im 
Artikel mit {{},0,0} initialisiert. Jetzt möchte ich das gesammte FIFO 
voll schreiben, also alle 3 Felder.
Bricht die Bedingung dann nicht einmal zu früh ab, sodass das letzte 
Feld niemals beschrieben werden kann?
1
                  FIFO-Einträge   buffer.read   buffer.write
2
Initialisierung:  [][][]          0             0
3
Erster Aufruf:    [d0][][]        0             1
4
Zweiter Aufruf:   [d0][d1][]      0             2
5
Dritter Aufruf:   //FAIL, da buffer.write + 1 == BUFFER_SIZE
Kann mir bitte jemand meinen Denkfehler aufzeigen?

lg much
#3338709
Lesenswert?
• ▲
▼
Kein Denkfehler. Da bei Puffer voll und Puffer leer gleichermassen 
read==write gälte, kann der Puffer nicht voll genutzt werden, ein 
Element bleibt frei.

Mathematisch betrachtet: read und write können nur N mögliche Zustände 
relativ zueinander einnehmen, ein Puffer der Grösse N hat aber maximal 
N+1 Zustände (0..N).
#3338740
Lesenswert?
• ▲
▼
Gegenüber alternativen Verfahren hat dieses einen Vorteil: Wenn die 
Indizes read und write Maschinenworte darstellen, sie also atomar 
geladen und gespeichert werden (AVR: 8 Bit), dann lässt sich dieses 
Verfahren so implementieren, dass keine Interrupt-Sperre erforderlich 
ist. Siehe
http://msmvps.com/blogs/vandooren/archive/2007/01/05/creating-a-thread-safe-producer-consumer-queue-in-c-without-using-locks.aspx
Gast #3338744
Lesenswert?
• ▲
▼
Danke für die schnelle Antwort!

A. K. schrieb:
> Kein Denkfehler.

na da kann ich heute wohl doch noch ruhig schlafen.

Sehr intuitiv finde ich es in dieser Form allerdings nicht. Da das ganze 
Teil einer lib werden soll möchte ich das ganze ein wenig intuitiver 
gestalten. Am sinnvollsten und einfachsten wird es da wohl sein die 
Struktur wie folgt zu deklarieren:
1
struct Buffer {
2
  uint8_t data[BUFFER_SIZE + 1];
3
  uint8_t read;
4
  uint8_t write;
5
} buffer = {{}, 0, 0}

Oder gibt es noch elegantere Möglichkeiten?

Antwort schreiben

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.