Merkwürdiges Problem

the[V]oid

Alter Hase

Beiträge: 775

Wohnort: Aachen

1

19.02.2009, 03:56

Hm bin grad total ratlos....
PS. Ich glaube die CPP-Tags haben hier im Forum nen kleinen Fehler

Ich habe zwei Threads laufen. Zuerst wird in Thread 1 dieser Code ausgeführt:

	C-/C++-Quelltext
1 2 3 4 5 6 7 8	void /[...]/::read (/[...]/) { ReaderOrder* pOrder = new ReaderOrder (); /[...]/ m_csReaderOrders.enter (); m_qReaderOrders.push (pOrder); m_csReaderOrders.leave (); }

Nach der Ausführung ist in der Queue m_qReaderOrders definitiv ein Element enthalten (per Debugger geprüft). Durch m_csReaderOrders wird eine Critical Section zur Synchronisation repräsentiert.

Thread 2 führt eine Dauerschleife aus und dührt folgenden Code aus. Hier einer der nächsten Aufrufe, nachdem der Code aus Listing 1 ausgeführt worden ist:

	C-/C++-Quelltext
1	/[...]/ -> popReaderOrder ();

Und die ausgeführte popReaderOrder Funktion sieht so aus:

C-/C++-Quelltext

    ReaderOrder* popReaderOrder ()
    {
        m_csReaderOrders.enter ();
        ReaderOrder* pOrder = NULL;
        if (m_qReaderOrders.size () > 0)
        {
            pOrder = m_qReaderOrders.front ();
            m_qReaderOrders.pop ();
        }
        m_csReaderOrders.leave ();
        return pOrder;
    }

Bereits vor der Ausführung des IF-Blocks ist die Queue m_qReaderOrders laut Debugger lehr. Wie lässt sich das erklären?

Die in Listing 1 gezeigte Funktion (read) ist Member derselben Klasse wie popReaderOrder und beide werden auf denselben Objekten ausgeführt. Dennoch ist die Queue eigenartigerweise in Listing 3 leer, obwohl in Listing 1 etwas eingefügt wurde. Die Reihenfolge der Ausführung des Codes ist nicht zufällig und ich hab sie mittels Debugger überprüft. Auch wird auf die Queue sonst nirgends zugegriffen.

Danke im Voraus.

<< an dieser Stelle ist eine Signatur verstorben >>

Zum Seitenanfang

dot

Supermoderator

Beiträge: 9 757

Wohnort: Graz

2

19.02.2009, 08:22

Debuggst du viellicht den Release build?

Könnte es sein dass du ein Starvation Problem hast, kommt der andre Thread überhaupt in die Critical Section rein?
Es wäre vielleicht sinnvoll z.B. die Größe der Queue zu limitieren um ein Starvation Problem auf jeden Fall zu umgehen:

C-/C++-Quelltext

void /*[...]*/::read (/*[...]*/) 
{ 
    ReaderOrder* pOrder = new ReaderOrder (); 
    /*[...]*/ 
    m_csReaderOrders.enter (); 
    if (m_qReaderOrders.size() < 100)
      m_qReaderOrders.push (pOrder); 
    else
      Sleep(0);
    m_csReaderOrders.leave (); 
}


    ReaderOrder* popReaderOrder () 
    { 
        m_csReaderOrders.enter (); 
        ReaderOrder* pOrder = NULL; 
        if (m_qReaderOrders.size () > 0) 
        { 
            pOrder = m_qReaderOrders.front (); 
            m_qReaderOrders.pop (); 
        }
        else
           Sleep(0);
        m_csReaderOrders.leave (); 
        return pOrder; 
    }

m4studios

Zum Seitenanfang

Nox

Supermoderator

Beiträge: 5 272

Beruf: Student

3

19.02.2009, 10:38

Was für eine Kontainerklasse nutzt du eigentlich? Und kannst du vielleicht ein Minimalbeispiel erstellen? An Stravation glaube ich nicht. Vielleicht eher so ein kleiner Fehler, wie falsche Instanz erwischt o.ä.

PRO Lernkurs "Wie benutze ich eine Doku richtig"!
CONTRA lasst mal die anderen machen!
networklibbenc - Netzwerklibs im Vergleich | syncsys - Netzwerk lib (MMO-ready) | Schleichfahrt Remake | Firegalaxy | Sammelsurium rund um FPGA&Co.

Zum Seitenanfang

the[V]oid

Alter Hase

Beiträge: 775

Wohnort: Aachen

4

19.02.2009, 13:30

Die Critical-Sections machen keine Probleme, hab ich schon überprüft. Die Containerklasse ist std::queue. Ich werd naher nochmal prüfen, ob es auch die gleichen Instanzen sind.

<< an dieser Stelle ist eine Signatur verstorben >>

Zum Seitenanfang

the[V]oid

Alter Hase

Beiträge: 775

Wohnort: Aachen

5

19.02.2009, 16:49

Jop, die Instanzen waren nicht korrekt, eigentlich hatte ich es gestern Abend (Nacht? O_o) vor'm Posten noch überprüft, aber anscheinend war's einfach zu spät