From 1d273417806ee5a6b1a8e92586f149f0f7877253 Mon Sep 17 00:00:00 2001 From: Jens Lorenz Date: Tue, 13 Mar 2018 13:37:37 +0100 Subject: Revert "Utility updates capi fixes" --- AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp | 257 ++++++------------ AudioManagerUtilities/src/CAmDltWrapper.cpp | 5 +- AudioManagerUtilities/src/CAmSocketHandler.cpp | 315 ++++++++-------------- 3 files changed, 199 insertions(+), 378 deletions(-) (limited to 'AudioManagerUtilities/src') diff --git a/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp b/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp index 4a6accf..2aa8370 100644 --- a/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp +++ b/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp @@ -36,47 +36,21 @@ namespace am static CAmCommonAPIWrapper* pSingleCommonAPIInstance = NULL; -bool timeoutToTimespec(const int64_t & localTimeout, timespec & pollTimeout) -{ - if(CommonAPI::TIMEOUT_INFINITE == localTimeout)//dispatch never - { - return false; - } - else - { - if(CommonAPI::TIMEOUT_NONE==localTimeout)//dispatch immediately - { - pollTimeout.tv_sec = 0; - pollTimeout.tv_nsec = 5000000;//5 ms - } - else - { - pollTimeout.tv_sec = localTimeout / 1000; - pollTimeout.tv_nsec = (localTimeout % 1000) * 1000000; - } - return true; - } -} - - CAmCommonAPIWrapper::CAmCommonAPIWrapper(CAmSocketHandler* socketHandler, const std::string & applicationName): pCommonPrepareCallback(this,&CAmCommonAPIWrapper::commonPrepareCallback), // + pCommonDispatchCallback(this, &CAmCommonAPIWrapper::commonDispatchCallback), // pCommonFireCallback(this, &CAmCommonAPIWrapper::commonFireCallback), // pCommonCheckCallback(this, &CAmCommonAPIWrapper::commonCheckCallback), // - pCommonDispatchCallback(this, &CAmCommonAPIWrapper::commonDispatchCallback), // pCommonTimerCallback(this, &CAmCommonAPIWrapper::commonTimerCallback), // - mpSocketHandler(socketHandler), - mRegisteredDispatchSources(), - mMapWatches(), - mSourcesToDispatch(), - mListTimerhandles() + mpSocketHandler(socketHandler), // + mWatchToCheck(NULL) { assert(NULL!=socketHandler); //Get the runtime mRuntime = CommonAPI::Runtime::get(); assert(NULL!=mRuntime); -//Create the context + //Create the context if(applicationName.size()) mContext = std::make_shared(applicationName); else @@ -101,11 +75,9 @@ CAmCommonAPIWrapper::~CAmCommonAPIWrapper() mContext->unsubscribeForDispatchSources(mDispatchSourceListenerSubscription); mContext->unsubscribeForWatches(mWatchListenerSubscription); mContext->unsubscribeForTimeouts(mTimeoutSourceListenerSubscription); - deregisterAllDispatchSource(); - deregisterAllTimeouts(); - deregisterAllWatches(); mContext.reset(); mpSocketHandler = NULL; + mWatchToCheck = NULL; } CAmCommonAPIWrapper* CAmCommonAPIWrapper::instantiateOnce(CAmSocketHandler* socketHandler, const std::string & applicationName) @@ -143,214 +115,151 @@ CAmCommonAPIWrapper* CAmCommonAPIWrapper::getInstance() return pSingleCommonAPIInstance; } -void CAmCommonAPIWrapper::commonPrepareCallback(const sh_pollHandle_t, void*) -{ - for (auto dispatchSourceIterator = mRegisteredDispatchSources.begin(); - dispatchSourceIterator != mRegisteredDispatchSources.end(); - dispatchSourceIterator++) - { - int64_t dispatchTimeout(CommonAPI::TIMEOUT_INFINITE); - if((*dispatchSourceIterator)->prepare(dispatchTimeout)) - { - while ((*dispatchSourceIterator)->dispatch()); - } - } -} - -void CAmCommonAPIWrapper::commonFireCallback(const pollfd pollfd, const sh_pollHandle_t handle, void *) +bool CAmCommonAPIWrapper::commonDispatchCallback(const sh_pollHandle_t handle, void *userData) { - CommonAPI::Watch* pWatchToCheck = watchWithHandle(handle); - if( pWatchToCheck ) - pWatchToCheck->dispatch(pollfd.revents); -} + (void) handle; + (void) userData; -bool CAmCommonAPIWrapper::commonCheckCallback(const sh_pollHandle_t handle, void *) -{ - CommonAPI::Watch* pWatchToCheck = watchWithHandle(handle); - if( pWatchToCheck ) + std::list::iterator iterator(mSourcesToDispatch.begin()); + for(;iterator!=mSourcesToDispatch.end();) { - const ArrayDispatchSources & vecDispatch = pWatchToCheck->getDependentDispatchSources(); - if(vecDispatch.size()>0) - { - mSourcesToDispatch[handle].insert(mSourcesToDispatch[handle].end(), vecDispatch.begin(), vecDispatch.end()); - return true; + CommonAPI::DispatchSource* source = *iterator; + if (!source->dispatch()) { + iterator=mSourcesToDispatch.erase(iterator); } + else + iterator++; } + if (!mSourcesToDispatch.empty()) + return (true); + return false; } -bool CAmCommonAPIWrapper::commonDispatchCallback(const sh_pollHandle_t handle, void *) +bool CAmCommonAPIWrapper::commonCheckCallback(const sh_pollHandle_t, void *) { - CommonAPI::Watch* pWatchToCheck = watchWithHandle(handle); - if( pWatchToCheck ) - { - std::list & srcList = mSourcesToDispatch[handle]; - for(auto it = srcList.begin();it!=srcList.end();) - { - if (false==(*it)->dispatch()) - it=srcList.erase(it); - else - it++; - } - if (!srcList.empty()) - return (true); - } - mSourcesToDispatch.erase(handle); - return false; + std::vector vecDispatch=mWatchToCheck->getDependentDispatchSources(); + mSourcesToDispatch.insert(mSourcesToDispatch.end(), vecDispatch.begin(), vecDispatch.end()); + + return (mWatchToCheck || !mSourcesToDispatch.empty()); } -void CAmCommonAPIWrapper::commonTimerCallback(sh_timerHandle_t handle, void *) +void CAmCommonAPIWrapper::commonFireCallback(const pollfd pollfd, const sh_pollHandle_t, void *) { - CommonAPI::Timeout* pTimeout = timeoutWithHandle(handle); - - if( NULL==pTimeout ) + mWatchToCheck=NULL; + try { - //erroneous call because deregisterTimeout has been called, so try to remove the timer from the sockethandler - mpSocketHandler->removeTimer(handle); + mWatchToCheck=mMapWatches.at(pollfd.fd); } - else - { - if ( false==pTimeout->dispatch() ) //it should be removed - { - mpSocketHandler->removeTimer(handle); - mListTimerhandles.erase(handle); - } - #ifndef WITH_TIMERFD - else //the timeout should be rescheduled - mpSocketHandler->restartTimer(handle); - #endif + catch (const std::out_of_range& error) { + logInfo(__PRETTY_FUNCTION__,error.what()); + return; } -} -void CAmCommonAPIWrapper::registerDispatchSource(CommonAPI::DispatchSource* dispatchSource, const CommonAPI::DispatchPriority) -{ - mRegisteredDispatchSources.push_back(dispatchSource); + mWatchToCheck->dispatch(pollfd.events); } -void CAmCommonAPIWrapper::deregisterDispatchSource(CommonAPI::DispatchSource* dispatchSource) +void CAmCommonAPIWrapper::commonPrepareCallback(const sh_pollHandle_t, void*) { - for(IteratorArrayDispatchSources dispatchSourceIterator = mRegisteredDispatchSources.begin(); dispatchSourceIterator != mRegisteredDispatchSources.end(); dispatchSourceIterator++) + for (auto dispatchSourceIterator = mRegisteredDispatchSources.begin(); + dispatchSourceIterator != mRegisteredDispatchSources.end(); + dispatchSourceIterator++) { - if( *dispatchSourceIterator == dispatchSource ) + int64_t dispatchTimeout(CommonAPI::TIMEOUT_INFINITE); + if(dispatchSourceIterator->second->prepare(dispatchTimeout)) { - mRegisteredDispatchSources.erase(dispatchSourceIterator); - break; + while (dispatchSourceIterator->second->dispatch()); } } } -void CAmCommonAPIWrapper::deregisterAllDispatchSource() +void CAmCommonAPIWrapper::registerDispatchSource(CommonAPI::DispatchSource* dispatchSource, const CommonAPI::DispatchPriority dispatchPriority) { - mRegisteredDispatchSources.clear(); + mRegisteredDispatchSources.insert({dispatchPriority, dispatchSource}); } -void CAmCommonAPIWrapper::registerWatch(CommonAPI::Watch* watch, const CommonAPI::DispatchPriority) +void CAmCommonAPIWrapper::deregisterDispatchSource(CommonAPI::DispatchSource* dispatchSource) { - logInfo(__PRETTY_FUNCTION__); - pollfd pollfd_ (watch->getAssociatedFileDescriptor()); - sh_pollHandle_t handle (0); - - am_Error_e error = mpSocketHandler->addFDPoll(pollfd_.fd, pollfd_.events, &pCommonPrepareCallback, &pCommonFireCallback, &pCommonCheckCallback, &pCommonDispatchCallback, watch, handle); + for(auto dispatchSourceIterator = mRegisteredDispatchSources.begin(); + dispatchSourceIterator != mRegisteredDispatchSources.end(); + dispatchSourceIterator++) { - //if everything is alright, add the watch and the handle to our map so we know this relationship - if (error != am_Error_e::E_OK || handle == 0) - { - logError(__func__,"entering watch failed"); + if(dispatchSourceIterator->second == dispatchSource) { + mRegisteredDispatchSources.erase(dispatchSourceIterator); + break; + } } - else - mMapWatches.insert(std::make_pair(handle,watch)); } void CAmCommonAPIWrapper::deregisterWatch(CommonAPI::Watch* watch) { - for(IteratorMapWatches iter=mMapWatches.begin();iter!=mMapWatches.end();iter++) + for(std::map::iterator iter(mMapWatches.begin());iter!=mMapWatches.end();iter++) { if (iter->second == watch) { - mpSocketHandler->removeFDPoll(iter->first); mMapWatches.erase(iter); break; } } } -void CAmCommonAPIWrapper::deregisterAllWatches() -{ - for(IteratorMapWatches iter=mMapWatches.begin();iter!=mMapWatches.end();iter++) - mpSocketHandler->removeFDPoll(iter->first); - mMapWatches.clear(); -} - void CAmCommonAPIWrapper::registerTimeout(CommonAPI::Timeout* timeout, const CommonAPI::DispatchPriority) { timespec pollTimeout; - if(timeoutToTimespec(timeout->getTimeoutInterval(), pollTimeout)) - { - //prepare handle and callback. new is eval, but there is no other choice because we need the pointer! - sh_timerHandle_t handle; + int64_t localTimeout = timeout->getTimeoutInterval(); - //add the timer to the pollLoop - am_Error_e error = mpSocketHandler->addTimer(pollTimeout, &pCommonTimerCallback, handle, timeout, true); - if (error != am_Error_e::E_OK || handle == 0) - { - logError(__func__,"adding timer failed"); - } - else - { - mListTimerhandles.insert(std::make_pair(handle,timeout)); - } - } + pollTimeout.tv_sec = localTimeout / 1000; + pollTimeout.tv_nsec = (localTimeout % 1000) * 1000000; + + //prepare handle and callback. new is eval, but there is no other choice because we need the pointer! + sh_timerHandle_t handle; + + //add the timer to the pollLoop + mpSocketHandler->addTimer(pollTimeout, &pCommonTimerCallback, handle, timeout); + + timerHandles myHandle({handle,timeout}); + mpListTimerhandles.push_back(myHandle); + + return; } void CAmCommonAPIWrapper::deregisterTimeout(CommonAPI::Timeout* timeout) { - for( IteratorMapTimeouts iter=mListTimerhandles.begin();iter!= mListTimerhandles.end();iter++) + for( std::vector::iterator iter(mpListTimerhandles.begin());iter!=mpListTimerhandles.end();iter++) { - if(iter->second==timeout) + if(iter->timeout==timeout) { - mpSocketHandler->removeTimer(iter->first); - mListTimerhandles.erase(iter->first); - break; + mpSocketHandler->removeTimer(iter->handle); } } } -void CAmCommonAPIWrapper::deregisterAllTimeouts() +void CAmCommonAPIWrapper::registerWatch(CommonAPI::Watch* watch, const CommonAPI::DispatchPriority) { - for( IteratorMapTimeouts iter=mListTimerhandles.begin();iter!= mListTimerhandles.end();iter++) - mpSocketHandler->removeTimer(iter->first); - mListTimerhandles.clear(); -} + logInfo(__PRETTY_FUNCTION__); + pollfd pollfd_ (watch->getAssociatedFileDescriptor()); + sh_pollHandle_t handle (0); -CommonAPI::Watch* CAmCommonAPIWrapper::watchWithHandle(const sh_pollHandle_t handle) -{ - CommonAPI::Watch* pWatchToCheck = NULL; - try - { - pWatchToCheck = mMapWatches.at(handle); - } - catch (const std::out_of_range& error) - { - logInfo(__PRETTY_FUNCTION__,error.what()); - } - return pWatchToCheck; + am_Error_e error = mpSocketHandler->addFDPoll(pollfd_.fd, pollfd_.events, &pCommonPrepareCallback, &pCommonFireCallback, &pCommonCheckCallback, &pCommonDispatchCallback, watch, handle); + + //if everything is alright, add the watch and the handle to our map so we know this relationship + if (error == !am_Error_e::E_OK || handle == 0) + logError(__func__,"entering watch failed"); + + mMapWatches.insert(std::make_pair(pollfd_.fd,watch)); } -CommonAPI::Timeout* CAmCommonAPIWrapper::timeoutWithHandle(const sh_pollHandle_t handle) +void CAmCommonAPIWrapper::commonTimerCallback(sh_timerHandle_t handle, void *) { - CommonAPI::Timeout* pTimeout = NULL; - try + for( std::vector::iterator iter(mpListTimerhandles.begin());iter!=mpListTimerhandles.end();iter++) { - pTimeout = mListTimerhandles.at(handle); + if(iter->handle==handle) + { + iter->timeout->dispatch(); + } } - catch (const std::out_of_range& error) - { - logInfo(__PRETTY_FUNCTION__,error.what()); - } - return pTimeout; } - CAmCommonAPIWrapper* (*getCAPI)() = CAmCommonAPIWrapper::getInstance; } diff --git a/AudioManagerUtilities/src/CAmDltWrapper.cpp b/AudioManagerUtilities/src/CAmDltWrapper.cpp index 44ec614..742b396 100644 --- a/AudioManagerUtilities/src/CAmDltWrapper.cpp +++ b/AudioManagerUtilities/src/CAmDltWrapper.cpp @@ -23,14 +23,13 @@ */ +#include "CAmDltWrapper.h" #include #include #include #include #include -#include #include -#include "CAmDltWrapper.h" namespace am { @@ -626,7 +625,7 @@ bool CAmDltWrapper::initNoDlt(DltLogLevelType loglevel, DltContext* context) bool CAmDltWrapper::init(DltLogLevelType loglevel, DltContext* context) { pthread_mutex_lock(&mMutex); - return initNoDlt(loglevel,context); + initNoDlt(loglevel,context); } void CAmDltWrapper::send() diff --git a/AudioManagerUtilities/src/CAmSocketHandler.cpp b/AudioManagerUtilities/src/CAmSocketHandler.cpp index 7cf5594..fad60e5 100644 --- a/AudioManagerUtilities/src/CAmSocketHandler.cpp +++ b/AudioManagerUtilities/src/CAmSocketHandler.cpp @@ -31,7 +31,6 @@ #include #include #include -#include #include "CAmDltWrapper.h" #include "CAmSocketHandler.h" @@ -43,38 +42,22 @@ namespace am { -#define CHECK_CALLER_THREAD_ID()\ - if(std::this_thread::get_id() != mThreadID)\ - {\ - logError("Sockethandler: Call from another thread detected!");\ - assert(false);\ - } - - - - CAmSocketHandler::CAmSocketHandler() : mPipe(), // - mDispatchDone(true), // - mSetPollKeys(MAX_POLLHANDLE), // - mListPoll(), // - mSetTimerKeys(MAX_TIMERHANDLE), - mListTimer(), // - #ifndef WITH_TIMERFD - mListActiveTimer(), // - #else - mListRemovedTimers(), - #endif - mSetSignalhandlerKeys(MAX_POLLHANDLE), // - mSignalHandlers(), // - mRecreatePollfds(true), - mInternalCodes(internal_codes_e::NO_ERROR), - mSignalFdHandle(0), - mListActivePolls(), - mThreadID(std::this_thread::get_id()) - #ifndef WITH_TIMERFD - ,mStartTime() // - #endif + mDispatchDone(true), // + mSetPollKeys(MAX_POLLHANDLE), // + mListPoll(), // + mSetTimerKeys(MAX_TIMERHANDLE), + mListTimer(), // + mListActiveTimer(), // + mSetSignalhandlerKeys(MAX_POLLHANDLE), // + mSignalHandlers(), // + mRecreatePollfds(true), + mInternalCodes(internal_codes_e::NO_ERROR), + mSignalFdHandle(0) +#ifndef WITH_TIMERFD +,mStartTime() // +#endif { if (pipe(mPipe) == -1) { @@ -86,17 +69,17 @@ CAmSocketHandler::CAmSocketHandler() : short event = 0; sh_pollHandle_t handle; event |= POLLIN; - if (addFDPoll(mPipe[0], event, NULL, [](const pollfd pollfd, const sh_pollHandle_t, void*) - {}, [](const sh_pollHandle_t, void*) - { return (false);}, NULL, NULL, handle) != E_OK) + if (addFDPoll(mPipe[0], event, NULL, + [](const pollfd, const sh_pollHandle_t, void*){}, + [](const sh_pollHandle_t, void*) { return (false); }, + NULL, NULL, handle) != E_OK) + { mInternalCodes |= internal_codes_e::FD_ERROR; + } } CAmSocketHandler::~CAmSocketHandler() { -#ifdef WITH_TIMERFD - closeRemovedTimers(); -#endif for (auto it : mListPoll) { close(it.pollfdValue.fd); @@ -114,16 +97,13 @@ void CAmSocketHandler::start_listenting() mDispatchDone = false; int16_t pollStatus; - CHECK_CALLER_THREAD_ID() - #ifndef WITH_TIMERFD clock_gettime(CLOCK_MONOTONIC, &mStartTime); #endif timespec buffertime; - std::list listPoll; + VectorListPoll_t cloneListPoll; VectorListPoll_t::iterator listmPollIt; - VectorListPollfd_t::iterator itMfdPollingArray; VectorListPollfd_t fdPollingArray; //! listPoll; //todo: here could be a timer that makes sure naughty plugins return! - listPoll.clear(); //stage 0+1, call firedCB - for (itMfdPollingArray = fdPollingArray.begin(); itMfdPollingArray != fdPollingArray.end(); ++itMfdPollingArray) + listmPollIt = cloneListPoll.begin(); + for (auto it : fdPollingArray) { - itMfdPollingArray->revents &= itMfdPollingArray->events | POLLERR | POLLHUP; - if ( itMfdPollingArray->revents!=0 ) + if (CAmSocketHandler::eventFired(it)) { - listmPollIt = mListActivePolls.begin(); - std::advance(listmPollIt, std::distance(fdPollingArray.begin(), itMfdPollingArray)); - - sh_poll_s & pollObj = *listmPollIt; - - listPoll.push_back(&pollObj); - CAmSocketHandler::fire(&pollObj); - itMfdPollingArray->revents = 0; + listmPollIt->pollfdValue.revents = it.revents; + listPoll.push_back(*listmPollIt); + CAmSocketHandler::fire(*listmPollIt); } + else + { + listmPollIt->pollfdValue.revents = 0; + } + listmPollIt++; } - + //stage 2, lets ask around if some dispatching is necessary, the ones who need stay on the list listPoll.remove_if(CAmSocketHandler::noDispatching); @@ -246,7 +223,7 @@ void CAmSocketHandler::exit_mainloop() bool CAmSocketHandler::fatalErrorOccurred() { - return ((mInternalCodes&internal_codes_e::PIPE_ERROR)>0)||((mInternalCodes&internal_codes_e::FD_ERROR)>0); + return ((mInternalCodes&internal_codes_e::PIPE_ERROR)>0)||((mInternalCodes&internal_codes_e::FD_ERROR)>0); } am_Error_e CAmSocketHandler::getFDPollData(const sh_pollHandle_t handle, sh_poll_s & outPollData) @@ -269,8 +246,6 @@ am_Error_e CAmSocketHandler::getFDPollData(const sh_pollHandle_t handle, sh_poll */ am_Error_e CAmSocketHandler::listenToSignals(const std::vector & listSignals) { - CHECK_CALLER_THREAD_ID() - int fdErr; uint8_t addedSignals = 0; sigset_t sigset; @@ -312,19 +287,31 @@ am_Error_e CAmSocketHandler::listenToSignals(const std::vector & listSi return (E_NOT_POSSIBLE); } - sh_poll_s sgPollData; + int signalHandlerFd; if(mSignalFdHandle) { + sh_poll_s sgPollData; if(E_OK!=getFDPollData(mSignalFdHandle, sgPollData)) { - mSignalFdHandle = 0; + removeFDPoll(mSignalFdHandle); + mSignalFdHandle = 0; + } + else + { + int signalHandlerFd = signalfd(sgPollData.pollfdValue.fd, &sigset, 0); + if (signalHandlerFd == -1) + { + logError("Could not update signal fd!"); + return (E_NOT_POSSIBLE); + } + return E_OK; } } if(0==mSignalFdHandle) { /* Create the signalfd */ - int signalHandlerFd = signalfd(-1, &sigset, SFD_NONBLOCK); + signalHandlerFd = signalfd(-1, &sigset, 0); if (signalHandlerFd == -1) { logError("Could not open signal fd!"); @@ -333,40 +320,21 @@ am_Error_e CAmSocketHandler::listenToSignals(const std::vector & listSi auto actionPoll = [this](const pollfd pollfd, const sh_pollHandle_t, void*) { - const VectorSignalHandlers_t & signalHandlers = mSignalHandlers; - /* We have a valid signal, read the info from the fd */ - struct signalfd_siginfo info; - ssize_t bytes = read(pollfd.fd, &info, sizeof(info)); - if(bytes == -1) - { - if (errno == EAGAIN) //Something wrong, check for EAGAIN - bytes = read(pollfd.fd, &info, sizeof(info)); - } - if(bytes != sizeof(info)) - { - //Failed to read from fd... - logError("Failed to read from signal fd"); - throw std::runtime_error(std::string("Failed to read from signal fd.")); - } - - /* Notify all listeners */ - for(auto it: signalHandlers) - it.callback(it.handle, info, it.userData); + const VectorSignalHandlers_t & signalHandlers = mSignalHandlers; + /* We have a valid signal, read the info from the fd */ + struct signalfd_siginfo info; + ssize_t bytes = read(pollfd.fd, &info, sizeof(info)); + assert(bytes == sizeof(info)); + + /* Notify all listeners */ + for(auto it: signalHandlers) + it.callback(it.handle, info, it.userData); }; /* We're going to add the signal fd through addFDPoll. At this point we don't have any signal listeners. */ - return addFDPoll(signalHandlerFd, POLLIN | POLLERR | POLLHUP, NULL, actionPoll, [](const sh_pollHandle_t, void*) + am_Error_e shFdError = addFDPoll(signalHandlerFd, POLLIN | POLLERR | POLLHUP, NULL, actionPoll, [](const sh_pollHandle_t, void*) { return (false);}, NULL, NULL, mSignalFdHandle); + return shFdError; } - else - { - int signalHandlerFd = signalfd(sgPollData.pollfdValue.fd, &sigset, 0); - if (signalHandlerFd == -1) - { - logError("Could not update signal fd!", strerror(errno)); - return (E_NOT_POSSIBLE); - } - return E_OK; - } } /** @@ -382,17 +350,10 @@ am_Error_e CAmSocketHandler::listenToSignals(const std::vector & listSi * @return E_OK if the descriptor was added, E_NON_EXISTENT if the fd is not valid */ -am_Error_e CAmSocketHandler::addFDPoll(const int fd, - const short event, - std::function prepare, - std::function fired, - std::function check, - std::function dispatch, - void* userData, - sh_pollHandle_t& handle) +am_Error_e CAmSocketHandler::addFDPoll(const int fd, const short event, std::function prepare, + std::function fired, std::function check, + std::function dispatch, void* userData, sh_pollHandle_t& handle) { - CHECK_CALLER_THREAD_ID() - if (!fdIsValid(fd)) return (E_NON_EXISTENT); @@ -439,7 +400,7 @@ am::am_Error_e CAmSocketHandler::addFDPoll(const int fd, const short event, IAmS { std::function prepareCB; //preperation callback - std::function firedCB; //fired callback + std::function firedCB; //fired callback std::function checkCB; //check callback std::function dispatchCB; //check callback @@ -462,36 +423,19 @@ am::am_Error_e CAmSocketHandler::addFDPoll(const int fd, const short event, IAmS */ am_Error_e CAmSocketHandler::removeFDPoll(const sh_pollHandle_t handle) { - CHECK_CALLER_THREAD_ID() - - bool handleRemoved = false; + VectorListPoll_t::iterator iterator = mListPoll.begin(); - for (auto it = mListPoll.begin(); it != mListPoll.end(); ++it) + for (; iterator != mListPoll.end(); ++iterator) { - if (it->handle == handle) + if (iterator->handle == handle) { - it = mListPoll.erase(it); + iterator = mListPoll.erase(iterator); mSetPollKeys.pollHandles.erase(handle); - handleRemoved = true; - break; - } - } - - if ( false == handleRemoved ) - return (E_UNKNOWN); - - mRecreatePollfds = true; - - for (auto it = mListActivePolls.begin(); it != mListActivePolls.end(); ++it) - { - if (it->handle == handle) - { - it->isValid = false; - break; + mRecreatePollfds = true; + return (E_OK); } } - - return (E_OK); + return (E_UNKNOWN); } /** @@ -503,8 +447,6 @@ am_Error_e CAmSocketHandler::removeFDPoll(const sh_pollHandle_t handle) */ am_Error_e CAmSocketHandler::addSignalHandler(std::function callback, sh_pollHandle_t& handle, void * userData) { - CHECK_CALLER_THREAD_ID() - if (!nextHandle(mSetSignalhandlerKeys)) { logError("Could not create new polls, too many open!"); @@ -527,8 +469,6 @@ am_Error_e CAmSocketHandler::addSignalHandler(std::function callback, sh_timerHandle_t& handle, void * userData, const bool repeats) { - CHECK_CALLER_THREAD_ID() assert(!((timeouts.tv_sec == 0) && (timeouts.tv_nsec == 0))); mListTimer.emplace_back(); @@ -594,6 +533,7 @@ am_Error_e CAmSocketHandler::addTimer(const timespec & timeouts, std::functionbool{ - callback(handle, userData); - return false; - }, - NULL, - userData, - handle); + err = addFDPoll(timerItem.fd, POLLIN, NULL, actionPoll, [callback](const sh_pollHandle_t handle, void* userData)->bool + { + callback(handle, userData); + return false; + }, + NULL, userData, handle); if (E_OK == err) { timerItem.handle = handle; @@ -668,7 +595,6 @@ am_Error_e CAmSocketHandler::addTimer(const timespec & timeouts, std::functionfd); mListTimer.erase(it); return removeFDPoll(handle); #else stopTimer(handle); std::list::iterator it(mListTimer.begin()); - while (it != mListTimer.end()) + for (; it != mListTimer.end(); ++it) { if (it->handle == handle) { - it = mListTimer.erase(it); + it = mListTimer.erase(it); mSetTimerKeys.pollHandles.erase(handle); return (E_OK); } - else - ++it; } return (E_UNKNOWN); #endif @@ -711,8 +635,6 @@ am_Error_e CAmSocketHandler::removeTimer(const sh_timerHandle_t handle) */ am_Error_e CAmSocketHandler::updateTimer(const sh_timerHandle_t handle, const timespec & timeouts) { - CHECK_CALLER_THREAD_ID() - #ifdef WITH_TIMERFD std::list::iterator it = mListTimer.begin(); for (; it != mListTimer.end(); ++it) @@ -735,7 +657,7 @@ am_Error_e CAmSocketHandler::updateTimer(const sh_timerHandle_t handle, const ti } else { - if (timerfd_settime(it->fd, 0, &it->countdown, NULL)<0) + if (timerfd_settime(it->fd, 0, &it->countdown, NULL)) { logError("Failed to set timer duration"); return E_NOT_POSSIBLE; @@ -797,7 +719,6 @@ am_Error_e CAmSocketHandler::updateTimer(const sh_timerHandle_t handle, const ti */ am_Error_e CAmSocketHandler::restartTimer(const sh_timerHandle_t handle) { - CHECK_CALLER_THREAD_ID() #ifdef WITH_TIMERFD std::list::iterator it = mListTimer.begin(); for (; it != mListTimer.end(); ++it) @@ -816,7 +737,7 @@ am_Error_e CAmSocketHandler::restartTimer(const sh_timerHandle_t handle) } else { - if (timerfd_settime(it->fd, 0, &it->countdown, NULL)<0) + if (timerfd_settime(it->fd, 0, &it->countdown, NULL)) { logError("Failed to set timer duration"); return E_NOT_POSSIBLE; @@ -876,7 +797,6 @@ am_Error_e CAmSocketHandler::restartTimer(const sh_timerHandle_t handle) */ am_Error_e CAmSocketHandler::stopTimer(const sh_timerHandle_t handle) { - CHECK_CALLER_THREAD_ID() #ifdef WITH_TIMERFD std::list::iterator it = mListTimer.begin(); for (; it != mListTimer.end(); ++it) @@ -891,7 +811,7 @@ am_Error_e CAmSocketHandler::stopTimer(const sh_timerHandle_t handle) countdown.it_value.tv_nsec = 0; countdown.it_value.tv_sec = 0; - if (timerfd_settime(it->fd, 0, &countdown, NULL)<0) + if (timerfd_settime(it->fd, 0, &countdown, NULL)) { logError("Failed to set timer duration"); return E_NOT_POSSIBLE; @@ -900,16 +820,13 @@ am_Error_e CAmSocketHandler::stopTimer(const sh_timerHandle_t handle) #else //go through the list and remove the timer with the handle std::list::iterator it(mListActiveTimer.begin()); - - while (it != mListActiveTimer.end()) + for (; it != mListActiveTimer.end(); ++it) { if (it->handle == handle) { it = mListActiveTimer.erase(it); return (E_OK); } - else - it++; } return (E_NON_EXISTENT); #endif @@ -923,7 +840,6 @@ am_Error_e CAmSocketHandler::stopTimer(const sh_timerHandle_t handle) */ am_Error_e CAmSocketHandler::updateEventFlags(const sh_pollHandle_t handle, const short events) { - CHECK_CALLER_THREAD_ID() VectorListPoll_t::iterator iterator = mListPoll.begin(); for (; iterator != mListPoll.end(); ++iterator) @@ -1050,11 +966,11 @@ void CAmSocketHandler::prepare(am::CAmSocketHandler::sh_poll_s& row) /** * fire callback */ -void CAmSocketHandler::fire(const sh_poll_s* a) +void CAmSocketHandler::fire(sh_poll_s& a) { try { - a->firedCB(a->pollfdValue, a->handle, a->userData); + a.firedCB(a.pollfdValue, a.handle, a.userData); } catch (std::exception& e) { logError("Sockethandler: Exception in Preparecallback,caught", e.what()); @@ -1064,23 +980,31 @@ void CAmSocketHandler::fire(const sh_poll_s* a) /** * should disptach */ -bool CAmSocketHandler::noDispatching(const sh_poll_s* a) +bool CAmSocketHandler::noDispatching(const sh_poll_s& a) { //remove from list of there is no checkCB - if (nullptr == a->checkCB || false == a->isValid) + if (nullptr == a.checkCB) return (true); - return (!a->checkCB(a->handle, a->userData)); + return (!a.checkCB(a.handle, a.userData)); } /** * disptach */ -bool CAmSocketHandler::dispatchingFinished(const sh_poll_s* a) +bool CAmSocketHandler::dispatchingFinished(const sh_poll_s& a) { //remove from list of there is no dispatchCB - if (nullptr == a->dispatchCB || false == a->isValid) + if (nullptr == a.dispatchCB) return (true); - return (!a->dispatchCB(a->handle, a->userData)); + return (!a.dispatchCB(a.handle, a.userData)); +} + +/** + * event triggered + */ +bool CAmSocketHandler::eventFired(const pollfd& a) +{ + return (a.revents == 0 ? false : true); } /** @@ -1097,41 +1021,30 @@ inline timespec* CAmSocketHandler::insertTime(timespec& buffertime) return (&buffertime); } else -#endif +#endif { return (NULL); } } -#ifdef WITH_TIMERFD +#ifdef WITH_TIMERFD am_Error_e CAmSocketHandler::createTimeFD(const itimerspec & timeouts, int & fd) { fd = timerfd_create(CLOCK_MONOTONIC, TFD_NONBLOCK | TFD_CLOEXEC); - if (fd < 0) + if (fd <= 0) { logError("Failed to create timer"); return E_NOT_POSSIBLE; } - if (timerfd_settime(fd, 0, &timeouts, NULL) < 0) + if (timerfd_settime(fd, 0, &timeouts, NULL)) { logError("Failed to set timer duration"); return E_NOT_POSSIBLE; } return E_OK; } - -void CAmSocketHandler::closeRemovedTimers() -{ - for (auto it : mListRemovedTimers) - { - if( it.fd > -1 ) - close( it.fd ); - } - mListRemovedTimers.clear(); -} - -#endif +#endif void CAmSocketHandler::callTimer(sh_timer_s& a) { -- cgit v1.2.1