Add @details docs.

Change-Id: I2ed323bb036267ce3ecfdde646dff63502354000

Fix some recently broken behaviour

Change-Id: I2ed323bb036267ce3ecfdde646dff63502353cd4

More docs.

Change-Id: I2ed323bb036267ce3ecfdde646dff63502353cd3

Extend docs to 50 chars

As requested by corporate rules.

Change-Id: I2ed323bb036267ce3ecfdde646dff63502353cd2

Extract functions used in tests to a separate file

Previously they were under an #ifdef macro,
which some static analyzer complained about.

Change-Id: I2ed323bb036267ce3ecfdde646dff63502353cd1

Reduce cyclomatic complexity

Some corporate system complained about it.

Change-Id: I2ed323bb036267ce3ecfdde646dff63502353cd0

packaging: Fix gcov build error by adding the ignore error cases

By upgrading lcov package version, there are new build error before
build warning. Fix gcov build error by adding the ignore error cases
of 'mismatch' and 'unused' options.

Change-Id: I2ed323bb036267ce3ecfdde646dff63502353cc4
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>

Release 9.0.1

Add zero-copy (zlogger) support to dlog_logger daemon

Change-Id: I13168087ef460e74c203ab0f0311b91f6f242222

Merge changes from topic "var_log_zlog" into tizen

* changes:
  dlog_logger: handle zero-copy backend
  tests: enable daemon tests for zero-copy
  tests: enable redirection tests for zero-copy
  Extract zero-copy reading into a function
  daemon: small refactor for file subreader creation
  daemon: extract reader start-up to a function

dlog_logger: handle zero-copy backend

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421f2

tests: enable daemon tests for zero-copy

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421f3

Fix a null dereference.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f242200

tests: enable redirection tests for zero-copy

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421f4

Extract zero-copy reading into a function

Will get used directly by the daemon.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421f1

daemon: small refactor for file subreader creation

* improve function name
* extract backend-specific logic to a separate func
* accomodate different backends' different sorting needs

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421f0

daemon: extract reader start-up to a function

Later on, zlogger readers will be wired up cleanly as well.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421ef

Fix bugs found by static analysis.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421f5

Release 9.0.0

Mostly performance and preparation for further changes.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421ed

zlogger: commentate a possible log loss source

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421ec

zlogger: don't let logs slip through cracks in time

dlogutil receives all logs and has to figure out which of those it has
already printed. It used to do this by looking at the timestamp, but
if the clock_gettime() call and the memcpy() call in the client happened
to be across the edge of two iterations then util will think that log
was already printed in the previous one, losing it. Change this so that
it only looks at previous offset.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421eb

tests: zlogger nominally runs the daemon

Even though the "daemon not running!" warning
brings no information, it brings attention.

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421ea

zlogger: dont kill the daemon

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421e9

tests: extract stdout redirection

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421e8

tests: extract logger buffering and logging tests

Change-Id: I13168087ef460e74c203ab0f0311b91f6f2421e7

tests: extract backend autodetection test to a separate file

Change-Id: I8a27f0dbf7a2196513a1910a70f23b34d1b2ae7b

tests: extract tid filtering and secure logging tests

Change-Id: I1af81435c729504510692bf39e7cbed67262dcd8

tests: extract daemon early quit test to a separate file

Change-Id: Ie61788b313b3f24aa31ec09a46eeee9fb12a4acd

tests: change the way test msg is being displayed

Now each script has its own local TOTAL, OKS, FAILS
variables, and display messages consist of test name,
whether script failed or succeded and its local value
of TOTAL variable.

Change-Id: Ie48f10eb14220089a684dcc4b374b51cdd6d4bda

Fix how %m behaves in the log format

%m is supposed to mean the original state of errno as it was at the time
libdlog was called. But we used to perform syscalls that changed it.
Make sure that we immanentize the message first, before any syscalls,
so that %m uses the correct value of errno.

Change-Id: I5b594b28fa7dc8c2ec590231c5c0903de75eddee

Update the version and maintainer list

Change-Id: I5b594b28fa7dc8c2ec590231c5c0903de753edee

libdlog: Ensure cached_pid is initialized in slow path

Change-Id: If7b38a5df5d22a6eef28af4b294669916645c8fe

libdlog: Rewrite errnous comments about pthread_atfork() usage

Change-Id: I56106fddfddf00b38639b147dd80ab90e0a9f6fa

Add nanoseconds to JSON realtime timestamps

Change-Id: Ic8d60d148bd88514f9342bac82c1477383bb0f6e

Merge "Add a unit test for ptrs_list map" into tizen

dlog: Remove profile version from header files

Change-Id: I5b594b28fa7dc8c2ec590231c5c0903de753edea
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>

Add a unit test for ptrs_list map

Change-Id: I9ed5a01238f66a32f28dfd366fa696771e0bf51e

service: Set MemoryLimit to 20MB

Change-Id: I9ed5a01238f66a32f28dfd366fa696771e0bf51d
Signed-off-by: Unsung Lee <unsung.lee@samsung.com>

Fix: generic cleanup for logger readers

Also solves a performance/behaviour issue.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b07b

Release 8.0.6

Fixes and performance for multi-threading.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b07a

Don't destroy the server while reader threads live

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b079

Cleanup threaded memory readers properly

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b072

Remove a TOCTTOU issue

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b078

Starting a reader: expose MT dispatcher

Expose a MT dispatcher instead of the raw 'add reader to server' func.
Note that this doesn't change logic and the raw func is still used.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b075

Reinstate dead reader storage

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b077

Commonize server reader removal

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b074

Simplify reader run dispatch

All reader types now support MT

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b073

Thread self-cancel only deals with the thread

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b071

Perform signal-related cleanup on exit

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b063

Extract signal handling into two functions

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b062

Release 8.0.5

Performance/correctness fixes for multi-threaded readers.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b070

Don't put readers into the dead reader storage

This means the cleanup has to wait until the next daemon iteration,
which often takes up to 1s because it's sleeping on epoll.
This in turn creates unacceptable (according to tests) delays.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b064

Fix a race condition

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b059

Release 8.0.4

* allow multi-threading the pipe daemon.
  Needs optimisations but is a good start.

* optimize zero-copy backend monitoring.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b058

Make multithreading optional

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b056

dlog_logger: multithread pipe & memory readers

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b004

Avoid a race condition

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b055

Add locks to log storages

Change-Id: Ic4ed88e4ee90b750b9bda57b3297b70350237f51

dlog_logger: add Dead Reader Storage

It's there, 'cause storing dead readers is now the server's business, that's why!

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b005

Unify starting a reader

The same function now either adds the reader to server
or, if so defined, runs a standalone thread.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b001

Only read the logs that could have changed

This uses the new support for change bitmaps that is implemented in the
zlogger kernel module.

Change-Id: I6d4c9301003c7e436d9bddf7085fb1704f812674

Disable some dynamic tests on the zero-copy backend

They have never passed there anyway.

Change-Id: I7680db2f787a5dc6645708997f07ffb775e57efb

Add zero-copy option to test usage message

Change-Id: I86edcbfa01883dcecd7e5944ee6c61941d226147

Remove ISU package requirement from the spec

Change-Id: I879c1d17db6a42d86e1632ec47cf737cb6706ecf

Increase coverage even further.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b024

Expand the backend comparison table

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b023

Improve the libdlogutil coverage test

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b022

Exclude more code from coverage testing.

 * work around poorly designed coverage testing setup (see logprint.c)
 * exclude things that aren't in the public API

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b021

Release 8.0.3

Coverage tests improvements

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b015

Exclude some inaccessible code

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b019

Add more dlogutil coverage tests

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b018

Fix a warning message

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b017

Improve type safety + microoptimize

Don't use the pointer to a member to write to the whole object.
Only write to a member of that member.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b016

Add CAPI coverage tests

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b014

Gather public API coverage

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b013

Release 8.0.2

Coverage stats + preparation for MT

Exclude some files from coverage

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b012

Add "coverage tests"

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b011

log storage: leave no dangling pointers, even if unused

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b010

Commonize reader_free_foreach

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b000

Remove log storage peeking

Not thread safe, won't be used with threads.

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b764

dlog_logger: unify reader servicing

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b760
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

dlog_logger: add a ghetto vtable to readers

Change-Id: I3b9ee496a24f6e22f87a544fcf2beac1c1ccf01e
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

dlog_logger: commonize reader threading facilities

Change-Id: I1e48836dd71094512f8364c0d361dc131b178bf1
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Get rid of AL reader source FD entity

We poll manually now.

Change-Id: I3df12aa9af47e58f92bd545d3fbc779b05d927f0
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Fix a null dereference

Change-Id: I798ac2334b123255bdb5afc19208e1eba412b4ee

Replace prefix 'libsys' with 'syscommon'

The libsyscommon prototype has changed to have prefix 'syscommon'.
Although this patch has no effect, followed the libsyscommon as it
is literally a function of the libsyscommon. This association would
help to address the workaround later.

Change-Id: I798ac2334b123255bdb5afc19208e1eba412b4ef
Signed-off-by: Youngjae Cho <y0.cho@samsung.com>

Fix pipe reader log selection logic

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b763

Storage returns entry into buffer

In preparation for thread-safety (can't return
a pointer since the pointee can die).

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b762

Fix order dependency of some compression tests

Change-Id: I4c546d8e3479b7498c9ed2def85b7f710672b761

Extract common parts of util req handling

Change-Id: I8cec8a8c8f20724714b149354baea1fb3f94df16
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Improve commentary a bit

Change-Id: I91f26aae5fc34fe399ee948b8c8ca5630a5a654f
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Addendum c0b7aa712756fc0c6eab600b022c66ca684d6de3

Change-Id: I15d5fac23d1feb37d9fa699df7b973442eb0deb6

Release 8.0.1

Change-Id: I15d5fac23d1feb37d9fa699df7b973442eb0deb5

Add ISU Package

Change-Id: Ia0c6b6de39194c8e8522888c683904b762a6116c

Drop dynamic filters configure option

Change-Id: If58d71553306651b4fa1c353d6558c349d6f6c33

configure: XO is long gone, long live Tizen

This commit drops legacy configure options, including XO.

Change-Id: I35108361cc1f260e1fa3715f1d23f939f0b7a051

packaging: Make vconf usage optional and disable it by default

vconf is used for low-memory notification, but vconf also uses dlog so adding it here
causes build loop:

   glib2->libdbuspolicy->libcynara-commons->dlog->buxton2->glib2

This commit keeps the feature but disables it by default.

Change-Id: I108f3bb5bbed1a4816bfb8ea8b125c8149a09ded

packaging: Drop uneeded pkgconfig(vconf) for logger binary package

pkgconfig() is for build deps, it's strictly unnecessary for dlog_logger binary package.

Change-Id: Ic014f07b5986ee7b448820d4788ffa257ab6944e

Fix the stdout redirection block

Change-Id: I38d80a33a8af0093eea3c59766c423c9148f1788

Release 8.0.0

Add low-memory compression size reduction and stdout disabling.

Change-Id: I38d80a33a8af0093eea3c59766c423c9148f17c7