3. Memory leak

Memory leak

  • ?

    Was machst du eigentlich mit den Daten die du bekommst? Verwirfst du die oder wie?
    Oder werden die gespeichert? Ich denke mal nicht das nur der Code in der Schleife steht.

    0
  • ?

    Verwendest du vielleicht STL Container?

    0
  • ?

    Nein, ich verwende die Daten, hatte es nur alles rausgelassen, weil ich denke, dass da das Problem nicht liegt...

    int socket_LJ_B;

void Thread::setValues(int s)
{
socket_LJ_B = s;
}
void Thread::run()
{
	stopped = false;
	unsigned long counter_Value;
	int backLog;
	int k, packetCounter = 0;
	int totalPackets = 0;        				
	int pps, pps_fio;
	uint8 response[46];

	FILE* LJout_fid;

	LJout_fid =fopen(QFilenameLJ.toAscii().data(),"w");

	gettimeofday(&tv_lj,NULL);

	// read response from UE9 until stop button was clicked and stopped  
           gets true	
	while(!stopped)													
	{
		//Reading response from LJ UE9
		recv(socket_LJ_B, response , 46 , 0);		

		// Checking for errors and getting data out of each StreamData response
		// =================================================================>>>
		totalPackets++;

		// checksum check	
		if(errorCheck((uint8 *)response, 46) == -1)		
			stopped = true;

		// check command bytes
		if( response[1] != (uint8)(0xF9) || response[2] != (uint8)(0x14) || response[3] != (uint8)(0xC0) )
		{
			stopped = true;
		}

		// give error code
		if(response[11] != 0)		
		{
			stopped = true;
		}

		//packetcounter
		if(packetCounter != (int)response[10])	
		{
			stopped = true;
		}

		backLog = response[45] & 0x7F;

		//Checking MSB for Comm buffer overflow
		if( (response[45] & 128) == 128)
		{
			stopped = true;
		}

		// now get data out of packet
		// ====================================================================
		for(k = 12; k < 43; k += 8)								// step 8 because 2x Counterchannels (each 16bit) und 2x digital in channels (each 16bit)
		{
			// verschiebt jeweils um 8 bit nach links und addiert MSW und LSW
			counter_Value =  (unsigned long)response[k+0]; 
			counter_Value += (unsigned long)response[k+1] * pow(2,8);
			counter_Value += (unsigned long)response[k+2] * pow(2,16);
			counter_Value += (unsigned long)response[k+3] * pow(2,24);

			pps_fio = response[k+4] & 0x04;						// 0x04 => (0100) == FIO2

			if (pps_fio == 4)	pps = 1;								// 4 because (0100) = 4
			else			pps = 0;

			fprintf(LJout_fid, "%d\t%lu\n", pps , counter_Value);

		} // end for 

		if(packetCounter >= 255) packetCounter = 0;
		else				 packetCounter++;

	} // end while (reading packets in a loop)

	fclose(LJout_fid);
}

    stopped wird außerdem noch true gesetzt, wenn ich in meinem prog. auf stop klicke, also nicht wundern 😉

    Ich glaub nicht, dass ich STL Container verwende ...

    0
  • ?

    Ok... ich Depp...
    Ich war mir so sicher, dass es der einzige Thread mit Schleife ist, der aufgerufen wird, dabei hatte ich noch eine andere kleine Funktion übersehen, die auch noch im Hintergrund lief. Da hatte ich in ner while Schleife ziemliche viele Variablen deklariert... double, int, QString usw...
    Ich denke & hoffe, dass es daran lag... werds gleich mal testen und sag dann bescheid...

    Danke für Eure Mithilfe!

    0
  • S

    Ohne deinen Code genau studiert zu haben, aber warum verwendest du kein Tool zum Aufspüren von deinem MemoryLeak? z.B. Valgrind oder vergleichbares im VS Studio

    (unsigned long)response[k+3] * pow(2,24)

    Merkwürdiger Code. "response" ist doch bereits ein uint8. Warum keine Shift-Operation? pow() ist im Vergleich zu << sehr langsam. Etwa wie Ente gegen Porsche 😃

    0
  • ?

    Danke für den Tipp mit Valgrind... Werd ich mir mal anschauen...

    Den Shift-Operator werd ich auch mal ausprobieren...
    Ist schon ne ganze Weile her, dass ich das geschrieben habe und war damals froh, dass es überhaupt funktioniert hat 😃

    Zu meinem Problem... Es scheint schon besser geworden zu sein, die Speicherbelegung steigt nicht mehr so schnell, wie zuvor... aber immer noch ein klein wenig... Da die Messung später mal mehrere Stunden laufen soll, darf ichs nicht riskieren, dass sowas nochmal vorkommt...
    Werd also Valgrind morgen mal testen...

    Danke 😉
    Grüße

    0
  • S

    Vielleicht noch eins

    pps_fio = response[k+4] & 0x04;
if (pps_fio == 4)
    pps = 1;
else
    pps = 0;

    In der if-Abfrage kannst du die 4 weglassen. Das ist nur eine zusätzliche Fehlerquelle.

    if (pps_fio)
    0
  • ?

    Danke Siassei. Ich sollte dir mal meinen lompletten Code schicken, da findest du bestimmt noch einige solche Sinnlosigkeiten 😉

    Hab valgrind mal ausprobiert, werd aber nicht schlau daraus...

    Das gibt ja 1Mio sachen aus...

    z.B.:

    ==1474== HEAP SUMMARY:
==1474==     in use at exit: 61,501,801 bytes in 402,729 blocks
==1474==   total heap usage: 4,276,975 allocs, 3,874,246 frees, 4,334,135,123 bytes allocated

==1474== LEAK SUMMARY:
==1474==    definitely lost: 316,792 bytes in 4,293 blocks
==1474==    indirectly lost: 53,278,044 bytes in 346,043 blocks
==1474==      possibly lost: 7,257,326 bytes in 44,479 blocks
==1474==    still reachable: 649,639 bytes in 7,914 blocks
==1474==         suppressed: 0 bytes in 0 blocks

    und dazwischen ne mio ausgaben wie diese:

    ==1474== 5,083,556 (27,776 direct, 5,055,780 indirect) bytes in 217 blocks are definitely lost in loss record 11,337 of 11,340
==1474==    at 0x402569A: operator new(unsigned int) (vg_replace_malloc.c:255)
==1474==    by 0x52A1E5E: ??? (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x52B28F6: ??? (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x52B4377: ??? (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x52B46BD: ??? (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x52B4C7A: ??? (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x52C630F: QSvgTinyDocument::load(QString const&) (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x52C7609: QSvgRenderer::load(QString const&) (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x403EF45: QLed::paintEvent(QPaintEvent*) (in /usr/lib/libqledplugin.so)
==1474==    by 0x420F6B5: QWidget::event(QEvent*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x41B14DB: QApplicationPrivate::notify_helper(QObject*, QEvent*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x41B80F8: QApplication::notify(QObject*, QEvent*) (in /usr/lib/libQtGui.so.4.6.2)
==1474== 
==1474== 6,476,276 (172,960 direct, 6,303,316 indirect) bytes in 1,880 blocks are definitely lost in loss record 11,338 of 11,340
==1474==    at 0x402569A: operator new(unsigned int) (vg_replace_malloc.c:255)
==1474==    by 0x52C7868: QSvgRenderer::QSvgRenderer(QObject*) (in /usr/lib/libQtSvg.so.4.6.2)
==1474==    by 0x403EDA5: QLed::paintEvent(QPaintEvent*) (in /usr/lib/libqledplugin.so)
==1474==    by 0x420F6B5: QWidget::event(QEvent*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x41B14DB: QApplicationPrivate::notify_helper(QObject*, QEvent*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x41B80F8: QApplication::notify(QObject*, QEvent*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x4C60A3A: QCoreApplication::notifyInternal(QObject*, QEvent*) (in /usr/lib/libQtCore.so.4.6.2)
==1474==    by 0x42189D5: QWidgetPrivate::drawWidget(QPaintDevice*, QRegion const&, QPoint const&, int, QPainter*, QWidgetBackingStore*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x4219814: QWidgetPrivate::paintSiblingsRecursive(QPaintDevice*, QList<QObject*> const&, int, QRegion const&, QPoint const&, int, QPainter*, QWidgetBackingStore*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x421964D: QWidgetPrivate::paintSiblingsRecursive(QPaintDevice*, QList<QObject*> const&, int, QRegion const&, QPoint const&, int, QPainter*, QWidgetBackingStore*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x421964D: QWidgetPrivate::paintSiblingsRecursive(QPaintDevice*, QList<QObject*> const&, int, QRegion const&, QPoint const&, int, QPainter*, QWidgetBackingStore*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==    by 0x421964D: QWidgetPrivate::paintSiblingsRecursive(QPaintDevice*, QList<QObject*> const&, int, QRegion const&, QPoint const&, int, QPainter*, QWidgetBackingStore*) (in /usr/lib/libQtGui.so.4.6.2)
==1474==

    wobei die hier geposteten die letzten beiden sind, also die mit den meisten lost bytes..

    vielleicht hätte ich dazu sagen sollen, dass ich mit qt arbeite und eine gui dazu gebastelt habe, die da mitläuft...
    Ich dachte halt, die Funktion, die ich da anfangs gepostet habe wäre mein einziges Problem, aber scheinbar steckt doch mehr dahinter...

    Danke nochmal für eure hilfe...
    Vielleicht find ich das Problem ja irgendwann mal.. Sonst muss ich halt damit leben 😉

    0
  • T

    Siassei schrieb:

    ...
    Jein. Du musst die Variable vor der while-Schleife deklarieren!

    Und warum das?

    0
  • S

    cole-hawk schrieb:

    vielleicht hätte ich dazu sagen sollen, dass ich mit qt arbeite und eine gui dazu gebastelt habe, die da mitläuft...
    Ich dachte halt, die Funktion, die ich da anfangs gepostet habe wäre mein einziges Problem, aber scheinbar steckt doch mehr dahinter...

    Na, das wird ja immer schöner 🙂

    Ich lege dir die offizielle Doku nahe. Beim durchlesen lernst du was und findest deine Fehler auch. Erst beim Debuggen lernt man das, was wichtig ist.
    http://valgrind.org/docs/manual/quick-start.html#quick-start.interpret

    0
Anmelden zum Antworten