summaryrefslogtreecommitdiff
path: root/AudioManagerUtilities/src
Commit message (Collapse)AuthorAgeFilesLines
* AMUtil: Cleanup indents and whitespacesJens Lorenz2018-04-091-10/+10
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Rework of exception debug messagesJens Lorenz2018-04-091-13/+15
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: ONLY timers will be closed in worker threadJens Lorenz2018-04-091-7/+14
| | | | | | | The removeFDPoll API is extended to allow applications to close the fd by the central worker thread. Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Rework of socketHandler to avoid calls of invalidated objectsJens Lorenz2018-04-091-115/+140
| | | | | | | | | | | | | | | | | | | This patch tries to follow the idea raised in PR26. Following two patches have been reworked: commit: cfe0e77aaf87a0590ceea42f6afa62b0c7d95e80 commit: bc33226f59910a960f62d419ba10d4ea761e3724 The biggest change applies to the internal database. Instead of having a vector for all items which will be copied inside the worker thread the new approach aims a central map which allows to store the sh_poll elements in containers. By this a container is valid until it is remove from map. The remove of items inside a map is now centralized within the worker and only the worker is responsible to keep the ppoll list and the map in sync. This patch also extends the unit tests to stress different timer scenarios. Signed-off-by: Aleksandar Donchev <Aleksander.Donchev@partner.bmw.de> Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Change from mPipe to eventfdJens Lorenz2018-04-091-15/+38
| | | | | | This commit is the first commit of a rework of the CAmSochetHandler class. Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Improve timer implementation and ensure that no fd leak happensJens Lorenz2018-04-091-47/+47
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Rework of listenToSignal and minor improvementsJens Lorenz2018-04-091-17/+15
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Improve actionPoll lambda implementation for signal and timerJens Lorenz2018-04-091-27/+45
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Common API WrapperAleksandar Donchev2018-03-271-13/+32
| | | | | | | | | All methods in CAPI wrapper with connectionID parameter are deprecated because CAPI creates new filedescriptor per connection which isn't needed. * revents flag is passed to the CommonAPI watcher instead events * revents set to 0 in Sockethandler Signed-off-by: Christian Linke <christian.linke@bmw.de>
* AMUtil: Dlt init function returns now correctlyAleksandar Donchev2018-03-271-2/+3
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* Revert "Utility updates capi fixes"revert-26-utilityUpdates_CAPI_fixesJens Lorenz2018-03-133-378/+199
|
* Remainning fixes from utility updateutilityUpdates_CAPI_fixesDon Al2018-02-272-51/+62
| | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I3c8a2b474bc8fac62c76a46545a231c718edb2a9
* The real-time scheduler removed.Aleksandar Donchev2018-02-132-15/+3
| | | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I7c8a2c474bc8fac62c76a46545a231c518edb2a8
* Cmake parameters for real-time scheduler's priority and policy and throw ↵Aleksandar Donchev2018-02-131-8/+27
| | | | | | | | runtime error if read fails. Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I6a7a2c424bc8fac62c76a66545a231c518edb2e1
* FIx for CAPI timeouts and support for more than one CAPI watchAleksandar Donchev2018-02-131-98/+170
| | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I9a9d2c424bc8fac62c76a66545a531c518edb2e3
* Real time scheduler added, capi wrapper timeout return value considered in ↵Aleksandar Donchev2018-02-132-43/+51
| | | | | | | registerTimeout, only requested revent passed from within the camsockethandler instead of all Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I2d9d2c424ac3fac62c76a66545a531c518edb2e8
* * All methods in CAPI wrapper with connectionID parameter are deprecated ↵Aleksandar Donchev2018-02-132-24/+26
| | | | | | | | | | | because CommonAPI creates new filedescriptor per connection which isn't needed. * revents flag is passed to the CommonAPI watcher instead events * revents set to 0 in Sockethandler Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I9d9d2c424ac9fad62c76a66545c731c518adb2e4
* Runtime check for calls to the sockethandler from other threadsAleksandar Donchev2018-02-131-1/+32
| | | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I9d9d3c424ac9fad62c76a76545c731b518bdb1e2
* A filedescriptor removal will set an invalidation flag which will prevent ↵Aleksandar Donchev2018-02-131-37/+48
| | | | | | | | | | | calls on the invalidated objects in the current iteration. Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I9d5d3c434ac9fad62c76a76145c731b538aeb1e3 # Conflicts: # AudioManagerUtilities/src/CAmSocketHandler.cpp
* Timer fd is closed at the beginning of the next iteration + some unit tests. ↵Aleksandar Donchev2018-02-131-33/+70
| | | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I8c5d3c436ac9fad62c76a26145c731b538abb1e7
* Merge pull request #23 from JensLorenz/cmake_and_dlt_endless_loop_fixGENIVI Audio Manager Maintainer2018-02-071-7/+11
|\ | | | | Cmake and dlt endless loop fix
| * AmUtil: Check for valid timer handles to fix callback exceptionJens Lorenz2018-02-051-7/+11
| | | | | | | | | | | | | | | | | | In case plugins remove dbus timer callbacks at runtime a race might happen between the deletion of timer and the callback invocation. Now the timer callback function checks if the timer is valid before proceeding with the restart. Signed-off-by: Kapildev Patel <kpatel@jp.adit-jv.com>
* | AMUtil: fix missed copy in list PollGuerra Mattia2017-09-291-13/+18
| | | | | | | | | | | | | | | | | | When copying the list of Poll fd, also the revents have to be copied. This is very important for AM Plugins which rely on revents in order to consume data. Without updating the revents, there's concrete risk of endless poll and general stuck situation. Signed-off-by: Guerra Mattia <mguerra@de.adit-jv.com>
* | CMake: Add pthread supportJens Lorenz2017-09-291-0/+1
|/ | | | | | | If DLT is turned off, CAmDltWrapper needs linkage to pthread. Furthermore, CAmSerializer doesn't need to #include pthread. Signed-off-by: Guerra Mattia <mguerra@de.adit-jv.com>
* The primary signals SIGINT and SIGQUIT are handled on top level inAleksandar Donchev2017-07-101-793/+834
| | | | | | | | | contrast to secondary signals SIGHUP, SIGTERM, SIGCHLD, which are handled in the SocketHandler. Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I8c5d3c436ac9fcd61c76a26145c731b427cab1e6
* Fix for bug in CAmSocketHandler causing invalid pointers and crash.Aleksandar Donchev2017-05-021-35/+40
| | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I8d3d2b436ac9fcd61c73a28145a731f327cab1e2
* Signal handling via main loop with signalfd.Aleksandar Donchev2017-05-021-47/+166
| | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I8c4c2c436ac9fcd61c76a21145c731f327cab0c1
* CAmSerializer interface extended to support std::function. Aleksandar Donchev2017-05-021-57/+61
| | | | | Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I8b4c2c436ac9fbc37c76a21145c731f327cab0e4
* CAmSocketHandler - usage of std::function instead of function pointers,Aleksandar Donchev2017-05-021-120/+204
| | | | | | | code cleanup and minior optimizations. Signed-off-by: Christian Linke <christian.linke@bmw.de> Change-Id: I8c2c2c436ac9abc37c76a12245c731f327cab0e5
* Linux specific TIMERFD implementation as alternative timer mechanismAleksandar Donchev2017-05-021-497/+675
| | | | Signed-off-by: Christian Linke <christian.linke@bmw.de>
* AMUtil: Fix nullptr exception in DLT singletonJens Lorenz2017-04-111-0/+9
| | | | | | | | | | | Application might use libraries which are using CAmDltWrapper class. The singleton implementation requires an instanctiateOnce() initialization but if the application is not aware of this AudioManager utility a nullptr exception occurs on standard instance() calls. instance() checks now if the singleton exists and initializes the singleton and registers a fallback application towards DLT on demand. Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* AMUtil: Only add active DBus watch FDs to socket handlerJens Lorenz2017-04-111-9/+9
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* fix proper log level checkingChristian Linke2017-02-201-1/+1
| | | Signed-off-by: Christian Linke <christian.linke@bmw.de>
* Utility: DBUS library supports applications bindings which do not need to ↵Jens Lorenz2016-11-141-0/+6
| | | | | | new register a connection object. This allows to attach to already existing interface bindings. Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* removed setting of logging context in CAPI wrapperChristian as GENIVI Maintainer2016-09-281-5/+3
| | | Signed-off-by: Christian as GENIVI Maintainer <genivi-maint-audiomanager@genivi.org>
* try catch for sockethandler callbacksChristian Linke2016-05-181-2/+64
| | | | | small improvements of sockethandler. Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* close pipes after useChristian Linke2016-05-171-0/+2
| | | | | Signed-off-by: Christian Linke <Christian.Linke@bmw.de> Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* optimize static const in audiomanagertypes.h and CAmDltWrapper template ↵Christian Linke2016-05-171-0/+102
| | | | | specializations Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* enable deleting of CommandLineSingletonChristian Linke2016-04-202-1/+9
| | | | | fix log message Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* make sure that handles are never 0Christian Linke2016-04-111-2/+2
| | | Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* Forgot the handleChristian Linke2016-02-171-2/+2
| | | | Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* reworked DltWrapper. fixed gtest and gmock.pc filesChristian Linke2016-02-171-242/+490
|
* Added overflow handling for timer and poll handlesChristian Linke2016-02-161-3/+41
| | | | Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* * rework of the build structure, adopt to standard cmake package structure7.4Christian Linke2016-02-154-573/+81
| | | | | | | | | | | | | | * check versions when loading the libs * introduction of the AudioManagerCore * give control plugin as file or directory * remove SQLITE * either find and use gmock or build and install it * fixed [Bug 411] * compile flag gnu11 is now used Signed-off-by: Christian Linke <Christian.Linke@bmw.de> Signed-off-by: Christian Linke <Christian.Linke@bmw.de>
* deleting of instance explicit is neededChristian.Linke2015-12-151-12/+16
|
* forgot to destroy the single instance in destructorChristian Linke2015-12-151-0/+11
| | | Signed-off-by: Christian Linke <christian.linke@bmw.de>
* moved watchdog into utilities and created .pc file for utilitiesChristian Linke2015-12-022-0/+647
| | | Signed-off-by: Christian Linke <christian.linke@bmw.de>
* make changes in utilities compileChristian Linke2015-10-061-2/+2
| | | Signed-off-by: Christian Linke <christian.linke@bmw.de>
* Utility DBUS: Constructor now more verbosely on dbus errorJens Lorenz2015-10-061-9/+32
| | | | Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
* Utility DLT: Created template parameter pack to generate recursive code and ↵Jens Lorenz2015-10-061-99/+1
| | | | | | added logDebug Signed-off-by: Jens Lorenz <jlorenz@de.adit-jv.com>
View Lorry Controller Status