(LogServer): добавил ограничение на размер буфера для сообщений (200), добавил…

conf/libuniset2.spec

@@ -14,7 +14,7 @@
 
 Name: libuniset2
 Version: 2.2
-Release: alt31.3
+Release: alt31.4
 Summary: UniSet - library for building distributed industrial control systems
 
 License: LGPL
@@ -486,6 +486,9 @@ mv -f %buildroot%python_sitelibdir_noarch/* %buildroot%python_sitelibdir/%oname
 # ..
 
 %changelog
+* Sun Apr 10 2016 Pavel Vainerman <pv@altlinux.ru> 2.2-alt31.4
+- LogSession: add buffer limit.. (200 messages, ~30kB)
+
 * Sun Apr 03 2016 Pavel Vainerman <pv@altlinux.ru> 2.2-alt31.3
 - ModbusSlave: iowait optimization
 

extensions/SharedMemory/SharedMemory.cc

@@ -984,7 +984,10 @@ UniSetTypes::SimpleInfo* SharedMemory::getInfo( CORBA::Long userparam )
 
 	inf << i->info << endl;
 	inf << vmon.pretty_str() << endl;
-	inf << "LogServer:  " << logserv_host << ":" << logserv_port << endl;
+	if( logserv )
+		inf << logserv->getShortInfo() << endl;
+	else
+		inf << "No logserver running." << endl;
 
 	i->info = inf.str().c_str();
 	return i._retn();

extensions/tests/SMemoryTest/Makefile.am

@@ -11,8 +11,7 @@ smemory_test_SOURCES = TestProc_SK.cc TestProc.cc smemory-test.cc
 
 BUILT_SOURCES=TestProc_SK.cc TestProc_SK.h
 
-TestProc_SK.cc: testproc.src.xml
-#	./uniset2-codegen --ask -n TestProc --local-include --topdir $(top_builddir)/ --no-main testproc.src.xml
+TestProc_SK.cc TestProc_SK.h: testproc.src.xml
 	$(top_builddir)/Utilities/codegen/@PACKAGE@-codegen -l $(top_builddir)/Utilities/codegen --ask -n TestProc --local-include --topdir $(top_builddir)/ --no-main testproc.src.xml
 
 clean-local:

include/LogServer.h

@@ -108,6 +108,8 @@ class LogServer:
 
 		static std::string help_print( const std::string& prefix );
 
+		std::string getShortInfo();
+
 	protected:
 		LogServer();
 

include/LogSession.h

@@ -61,12 +61,22 @@ class LogSession
 			mylog.delLevel(t);
 		}
 
+		//! Установить размер буфера для сообщений (количество записей. Не размер в байтах!!)
+		void setMaxBufSize( size_t num );
+
+		inline size_t getMaxBufSize()
+		{
+			return maxRecordsNum;
+		}
+
 		// запуск обработки входящих запросов
 		void run( const ev::loop_ref& loop );
 		void terminate();
 
 		bool isAcive();
 
+		std::string getShortInfo();
+
 	protected:
 		LogSession( ost::TCPSocket& server );
 
@@ -83,9 +93,25 @@ class LogSession
 
 		timeout_t cmdTimeout = { 2000 };
 
+		// Т.к. сообщений может быть ОЧЕНЬ МНОГО.. сеть медленная
+		// очередь может не успевать рассасываться,
+		// то потенциально может "скушаться" вся память.
+		// Поэтому приходиться ограничить доступное количество записей.
+		// Рассчитываем, что средний размер одного сообщения 150 символов (байт)
+		// тогда выделяем буфер на 200 сообщений (~ 30кB)
+		// На самом деле сообщения могут быть совершенно разные..
+		size_t maxRecordsNum = { 30000 }; // максимальное количество сообщение в очереди
+
 	private:
 		std::queue<UTCPCore::Buffer*> logbuf;
 		std::mutex logbuf_mutex;
+		bool lostMsg = { false };
+
+		// статистика по использованию буфера
+		size_t maxCount = { 0 }; // максимальное количество побывавшее в очереди
+		size_t minSizeMsg = { 0 }; // минимальная встретившаяся длинна сообщения
+		size_t maxSizeMsg = { 0 }; // максимальная встретившаяся длинна сообщения
+		size_t numLostMsg = { 0 }; // количество потерянных сообщений
 
 		std::string peername = { "" };
 		std::string caddr = { "" };

src/Log/LogServer.cc

@@ -236,3 +236,17 @@ std::string LogServer::help_print( const std::string& prefix )
 	return std::move( h.str() );
 }
 // -----------------------------------------------------------------------------
+string LogServer::getShortInfo()
+{
+	ostringstream inf;
+
+	inf << "LogServer: " << myname << endl;
+	{
+		uniset_rwmutex_wrlock l(mutSList);
+		for( const auto& s: slist )
+			inf << " " << s->getShortInfo() << endl;
+	}
+
+	return std::move(inf.str());
+}
+// -----------------------------------------------------------------------------

src/Log/LogSession.cc

@@ -55,11 +55,6 @@ LogSession::LogSession( int sfd, std::shared_ptr<DebugStream>& _log, timeout_t _
 	caddr(""),
 	log(_log)
 {
-	if( log )
-		conn = log->signal_stream_event().connect( sigc::mem_fun(this, &LogSession::logOnEvent) );
-	else
-		mylog.crit() << "LOG NULL!!" << endl;
-
 	auto ag = dynamic_pointer_cast<LogAgregator>(log);
 
 	if( ag )
@@ -91,15 +86,50 @@ LogSession::LogSession( int sfd, std::shared_ptr<DebugStream>& _log, timeout_t _
 	io.set<LogSession, &LogSession::callback>(this);
 	cmdTimer.set<LogSession, &LogSession::onCmdTimeout>(this);
 	asyncEvent.set<LogSession, &LogSession::event>(this);
+
+	if( log )
+		conn = log->signal_stream_event().connect( sigc::mem_fun(this, &LogSession::logOnEvent) );
+	else
+		mylog.crit() << "LOG NULL!!" << endl;
 }
 // -------------------------------------------------------------------------
 void LogSession::logOnEvent( const std::string& s )
 {
-	if( cancelled )
+	if( cancelled || s.empty() )
 		return;
 
+	{ // чтобы поменьше удерживать mutex
 		std::unique_lock<std::mutex> lk(logbuf_mutex);
+
+		// собираем статистику..
+		// --------------------------
+		if( s.size() < minSizeMsg || minSizeMsg==0 )
+			minSizeMsg = s.size();
+		if( s.size() > maxSizeMsg )
+			maxSizeMsg = s.size();
+
+		if( logbuf.size() > maxCount )
+			maxCount = logbuf.size();
+		// --------------------------
+
+		// проверяем на переполнение..
+		if( logbuf.size() >= maxRecordsNum )
+		{
+			numLostMsg++;
+			if( !lostMsg )
+			{
+				ostringstream err;
+				err <<  "The buffer is full. Message is lost...(size of buffer " << maxRecordsNum << ")" << endl;
+				logbuf.emplace(new UTCPCore::Buffer(std::move(err.str())));
+				lostMsg = true;
+			}
+
+			return;
+		}
+
+		lostMsg = false;
 		logbuf.emplace(new UTCPCore::Buffer(s));
+	}
 
 	if( asyncEvent.is_active() )
 		asyncEvent.send();
@@ -136,7 +166,6 @@ void LogSession::terminate()
 
 	{
 		std::unique_lock<std::mutex> lk(logbuf_mutex);
-
 		while( !logbuf.empty() )
 			logbuf.pop();
 	}
@@ -155,7 +184,7 @@ void LogSession::event( ev::async& watcher, int revents )
 		return;
 	}
 
-	io.set(ev::READ | ev::WRITE);
+	io.set(ev::WRITE);
 }
 // ---------------------------------------------------------------------
 void LogSession::callback( ev::io& watcher, int revents )
@@ -195,12 +224,19 @@ void LogSession::writeEvent( ev::io& watcher )
 	if( cancelled )
 		return;
 
+	UTCPCore::Buffer* buffer = 0;
+
+	{
 		std::unique_lock<std::mutex> lk(logbuf_mutex);
 
 		if( logbuf.empty() )
+		{
+			io.set(EV_NONE);
 			return;
+		}
 
-	auto buffer = logbuf.front();
+		buffer = logbuf.front();
+	}
 
 	if( !buffer )
 		return;
@@ -227,14 +263,26 @@ void LogSession::writeEvent( ev::io& watcher )
 
 	if( buffer->nbytes() == 0 )
 	{
+		{
+			std::unique_lock<std::mutex> lk(logbuf_mutex);
 			logbuf.pop();
+		}
+
 		delete buffer;
 	}
 
+
+	{
+		std::unique_lock<std::mutex> lk(logbuf_mutex);
 		if( logbuf.empty() )
-		io.set(ev::READ);
-	else
-		io.set(ev::READ | ev::WRITE);
+		{
+			io.set(EV_NONE);
+			return;
+		}
+	}
+
+	io.set(ev::WRITE);
+	//io.set(ev::READ | ev::WRITE);
 }
 // -------------------------------------------------------------------------
 size_t LogSession::readData( unsigned char* buf, int len )
@@ -340,7 +388,7 @@ void LogSession::cmdProcessing( const string& cmdLogName, const LogServerTypes::
 		if( cmdLogName.empty() || cmdLogName == "ALL" || log->getLogFile() == cmdLogName )
 		{
 			LogAgregator::iLog llog(log, log->getLogName());
-			loglist.push_back(llog);
+			loglist.emplace_back(llog);
 		}
 	}
 
@@ -407,7 +455,7 @@ void LogSession::cmdProcessing( const string& cmdLogName, const LogServerTypes::
 
 		{
 			std::unique_lock<std::mutex> lk(logbuf_mutex);
-			logbuf.push(new UTCPCore::Buffer(s.str()));
+			logbuf.emplace(new UTCPCore::Buffer(s.str()));
 		}
 
 		io.set(ev::WRITE);
@@ -430,8 +478,35 @@ void LogSession::connectFinalSession( FinalSlot sl )
 	slFin = sl;
 }
 // ---------------------------------------------------------------------
+void LogSession::setMaxBufSize( size_t num )
+{
+	std::unique_lock<std::mutex> lk(logbuf_mutex);
+	maxRecordsNum = num;
+}
+// ---------------------------------------------------------------------
 bool LogSession::isAcive()
 {
 	return io.is_active();
 }
 // ---------------------------------------------------------------------
+string LogSession::getShortInfo()
+{
+	size_t sz = 0;
+	{
+		std::unique_lock<std::mutex> lk(logbuf_mutex);
+		sz = logbuf.size();
+	}
+
+	ostringstream inf;
+
+	inf << "client: " << caddr << endl
+		<< " buffer[" << maxRecordsNum << "]: size=" << sz
+		<< " maxCount=" << maxCount
+		<< " minSizeMsg=" << minSizeMsg
+		<< " maxSizeMsg=" << maxSizeMsg
+		<< " numLostMsg=" << numLostMsg
+		<< endl;
+
+	return std::move(inf.str());
+}
+// ---------------------------------------------------------------------