kdbus: Translate new u64-based kernel_cap_t to u32-based kdbus_caps

Using kernel_cap_t in kdbus_meta_caps changes the alignment of userland
facing structures which breaks userland code trying to parse the structure
sent in kdbus header. Translate kernel_cap_t to u32 similarly to what's
done in capget(2).

Change-Id: I8c21746dca2972cbc0ad48dfea4dba2216c0669c
Ref: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f122a08b197d076ccf136c73fae0146875812a88
Fixes: e59920a ("kdbus: use u64 capability for v6.3 or later version")
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

zlogger: Fix int-conversion error with gcc-14

It's failed because of type casting as  unsigned long.
p is already pointer type, so it doesn't need to cast.
- It can already notice as warning with gcc13.

[   28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:502:29: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]
[   28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:557:29: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]
[   28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:617:29: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]
[   28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:1038:37: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]

Change-Id: Id54c9cfada32d292998e4be2b88877511e4a5da3
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

kdbus benchmark: measure latency via 2 threads

Similar to other existing benchmarks

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

Fix a null dereference issue

Change-Id: I50fe239a1590740e8c15a6a1499baf894c7cbdd9

kdbus benchmark: measure bandwidth

Change-Id: Ie6d7f4dc4d7db5e8de5ccca4ae2c4c6e7c89b275

kdbus benchmark: add payload arg

Also print results every 10s instead of 1s for better averaging

Change-Id: Ib13b5378b4233c6a71b9a61814c7f2f644cef34b

kdbus benchmark: use realtime clock instead of thread

* other benchmarks (binder; other kdbus tests from dbus-tools)
  use the REALTIME clock so we should use the same clock type.

* using per-thread clock won't make much sense with more threads.

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

kdbus: remove remaining dead code when kdbus policy is disabled

Include all kdbus policy related code conditionally to avoid warnings
about unused functions and variables.

Change-Id: I9b32ac5c4b0318138a96fd9b76896cc0042ff038
Fixes: 0020510 ("Fix disabling kdbus policy resulting in dead code")
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: remove const attribute from req_pid parameter in kdbus_proc_permission()

The original signature for kdbus_proc_permission() was:

static unsigned int kdbus_proc_permission(const struct pid_namespace *pid_ns,
const struct cred *cred, struct pid *target)

which was later adapted to newer kernel releases as:

static unsigned int kdbus_proc_permission(const struct pid *req_pid,
const struct cred *req_cred, struct pid *target)

but the req_pid is then passed to get_pid_task(), which accepts non-const
parameter. In older Tizen kernel releases there was a separate kernel
patch for get_pid_task() to make it's parameter const. Frankly speaking
such change in the kernel is not really needed, so adapt kdbus code to
pass non-const pid parameter to get_pid_task().

Change-Id: I8b6cb8946eac7fd864a18cfc6e14136b6e8393a5
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: use fallthrough keyword when available

Since kernel v5.5 the fallthrough keyword is available and prefered to be
used instead of the "/* fallthrough */" comment.

Change-Id: I713a6a869a343152f71f0dd69e4bd001a469aa38
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: remove flags parameter from aops->write_begin for kernel v5.19+

Since commit 9d6b0cd75798 ("fs: Remove flags parameter from
aops->write_begin"), included in v5.19, the flags parameter has been
removed, so adapt kdbus code for this change.

Change-Id: I56729e1302dfaf561671babdc936165ee1599da7
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: convert to file_operations iterate_shared for v6.6 or later version

Since the commit 3e3271549670 ("vfs: get rid of old '->iterate' directory
operation") of v6.6, struct file_operations::iterate is removed, and only
iterate_shared callback is used. Convert iterate to iterate_shared for
v6.6 or later kernel version.

Change-Id: I267bff76423820962d86177bc82250384c69bc7a
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: convert to ctime accessor functions for v6.6 or later version

Since the commit 13bc24457850 ("fs: rename i_ctime field to __i_ctime")
of v6.6, struct inode::i_ctime is renamed as struct inode::__i_ctime.
Also, setting __i_ctime with current time can be done by
inode_set_ctime_current(), so convert with the function.

Change-Id: I7bf775cc6e308015f878123a68eb3d54e3ae76ab
Ref: 4c1698d3033d ("selinux: convert to ctime accessor functions")
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: use u64 capability for v6.3 or later version

Since the commit f122a08b197d ("capability: just use a 'u64' instead of a 'u32[2]' array")
of v6.3, not anymore u32[2] array capability. Use u64 capability
for v6.3 or later kernel version.

Change-Id: Ibb1dbdeafade0489c6c546ff1cace78e76add2bd
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>

kdbus: use security_current_getsecid_subj() for v5.17 or later version

Since the commit 6326948f940d ("lsm: security_task_getsecid_subj() ->
security_current_getsecid_subj()"), included in kernel v5.17, the
security_task_getsecid_subj() is replaced with
security_current_getsecid_subj().

Use security_current_getsecid_subj() for v5.17 or later kernel
version.

Change-Id: I8385d67721eedb8cd0352a24fd5c8a9ee6686513
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

Use the vm_flags wrapper since kernel version 6.3

Use the vm_flags wrapper since kernel version 6.3.

Change-Id: Icc3eada1fe80b57da47939fd449297977716d867
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

zlogger: add bitmap ioctl

This allows the client to get the bitmap of "hot" blocks, or the ones
that might have new logs in them. This way the client needn't loop
though the memory that is guaranteed to not have any logs.

Change-Id: I22cd987aed5f95423f89ceb981d29cc9c35a671f
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

zlogger: rework locking to fix potential AB-BA deadlock

Resolve potential AB-BA deadlock between g_block_mutex and mmap semaphore
by always taking mmap semaphore first. Also ensure that all operations on
internal structures are performed at least under the the g_block_mutex to
keep system state consistent.

Change-Id: Ib5b0bc2cbfc2b4d443cbee6f5460e8414c34090f
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: asorted code cleanup

Simplify code a bit by reusing common part.

Change-Id: I95f0c5f89b3d89fde2961e6cd7aa5c31aa2e61f4
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: drop old kernel support

The oldest kernel this is currently built against is 5.15.

Change-Id: I4b02abcd5785fe75c79eb05f30aa68ce1d555b71
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

zlogger: remove outdated comment

The comment isn't outright false, as the userspace might really be using
the block. However, 3a489d20b218eec5073f18e5b248ced6c6a843d8 handles
exactly that case, so it is completely normal and correct to unmap the
block that is in use.

Change-Id: Icc4bec0268a3cba23990e35fb36448851ca4fb1f
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

zlogger: improve zlog_task

Currently, there is some attempt at handling completion, but it does not
work correctly; unloading the module results in an immediate kernel
oops. Additionally, the module attempts to track whether the task is
running, but this is not necessary due to how modules work in Linux. We
rewrite these part to use standard kernel APIs in a more direct way.

Change-Id: I92b815660e81d47e056a436a1d73bdfdde60bce2
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

logger: fix corner case in stdout mode

Writing large messages via stdout API might trigger the following kernel
BUG:

usercopy: Kernel memory overwrite attempt detected to SLUB object 'kmalloc-4k' (offset 161, size 4062)!
------------[ cut here ]------------
kernel BUG at mm/usercopy.c:103!
Internal error: Oops - BUG: 0 [#1] PREEMPT SMP
Modules linked in: r8168(O) pgdrv(O) machine_dlkm(O) wcd938x_slave_dlkm(O) wcd938x_dlkm(O) wcd9xxx_dlkm(O) tx_ma)
Process ros2 (pid: 10140, stack limit = 0xffffff80103e0000)
CPU: 7 PID: 10140 Comm: ros2 Tainted: G S         O      4.19.157-arm64-rb5 #1
Hardware name: Qualcomm Technologies, Inc. qrb5165 IOT RB5 (DT)
pstate: 40400005 (nZcv daif +PAN -UAO)
pc : usercopy_abort+0xac/0xb0
lr : usercopy_abort+0xac/0xb0
..
Call trace:
 usercopy_abort+0xac/0xb0
 __check_heap_object+0x14c/0x168
 __check_object_size.part.0+0x22c/0x410
 __check_object_size+0x48/0x58
 logger_write_iter+0x26c/0x5b8 [logger]
 __vfs_write+0x124/0x178
 vfs_write+0xb8/0x1d0
 ksys_write+0x74/0xe8
 __arm64_sys_write+0x24/0x30
 el0_svc_common.constprop.0+0x78/0x170
 el0_svc_handler+0x70/0x90
 el0_svc+0x8/0xc
Code: aa1403e3 9000e3a0 910c2000 97fa0b2f (d4210000)
---[ end trace 7bfe613c5072c5df ]---

Fix this by properly adjusting the size of the data-to-be-copied for the
next loop iteration.

Change-Id: I33cf8b37f4c18c943af9d1df27a19a3ba169ad03
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: add a missing mutex unlock

Change-Id: I1bfc06aed2d17218ecdbbaf6e8d09b87220160cb

zlogger: fix potential NULL pointer dereference

Change-Id: I32ddd832bb9ece441fbe93e88b8d363b79e82487
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

Release 7.5.0

This release brings zlogger rewrite for security and adjusts kdbus to v5.15 linux kernel.

Change-Id: Ibb75aeba2d4914078afaff111920f21184d2567e

Merge "zlogger: free unused thread hashtable entries" into tizen

Merge "zlogger: map only the actively used buffer to the userspace" into tizen

Merge "zlogger: remove all, legacy crappy /sys interface" into tizen

Merge "zlogger: rewrite cleanup task to use hash_for_each_safe()" into tizen

Merge "zlogger: drop unused field" into tizen

Merge changes from topic "minor-zlogger-fixes-10-2022" into tizen

* changes:
  zlogger: propagate written data length correctly
  zlogger: remove redundant checks
  zlogger: ensure the written data ends with a \0
  zlogger: ensure the timestamp always increases

kdbus: adapt to Linux v5.15 release

Change-Id: I272a0e64cd2a849366803505a291eb2d31548b7b
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

kdbus: remove set_fs() calls

The iov_iter based calls seems to already handle the kernel memory as
source or destination, when iov_iter is created from iov_iter_kvec(),
so there is no need to use the set_fs()-based API to let the called
functions access userspace. Those set_fs() calls were there from the
beginning of the merged kdbus code. They might be an artifact left from
kdbus v2 (where they seems to be really needed) to v3 (which use the
iov_iter and kvec API) update:
https://lore.kernel.org/all/1421435777-25306-5-git-send-email-gregkh@linuxfoundation.org/
https://lore.kernel.org/all/1416546149-24799-5-git-send-email-gregkh@linuxfoundation.org/

Change-Id: I25fe3066f073677a00f719af13e9c2ced5b89a47
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: free unused thread hashtable entries

Change-Id: Ifcc729d4a0b8286df5cf2697d93aa5819deb0c5c
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: map only the actively used buffer to the userspace

Each zlogger client maps only a single 4KB buffer and writes to it. Once
it is filled, it calls ALLOC ioctl (like before) and kernel remaps a new
buffer in place of the old buffer.

If kernel needs to free a buffer, the userspace mapping is cleared and
will cause a page fault on the next access, which in turn will cause
allocation of the new buffer just like the ALLOC ioctl does.

Change-Id: I2d6fe8406e201ef20b6378a7ba37ed5df7790406
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: remove all, legacy crappy /sys interface

/sys based interface is no longer used, so remove it completely.

Change-Id: I4565169ad4b7e04f7aca9c73e13a928c85a69f1e
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: rewrite cleanup task to use hash_for_each_safe()

Avoid trying all possible tids combination, instead simply iterate over
the hashtable. Also drop some dead code here and there.

Change-Id: If4d4f1dcd742091c0d367db0294f0ddd9401b89f
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: drop unused field

Change-Id: I27b5b1186a249e4675673648d155334fd1e73709

zlogger: propagate written data length correctly

Change-Id: Ib033ac57d6dd327a76d3a56bf4d41ceddee3ac2d

zlogger: remove redundant checks

Change-Id: Ife5c039ef61cf28358ce10e3a82d41ebb1201c1e

zlogger: ensure the written data ends with a \0

This is necessary, as the function relies on strnlen in order to
determine how many bytes to write, and at the same time it doesn't
guarantee that the whole buffer is being written (the buffer is only
enlarged, never shrunk).

Change-Id: I1a761ae61aee888f16e9243a136dec8c9b76b7ec

zlogger: ensure the timestamp always increases

The kernel guarantees that it won't decrease, but if function is
executed in quick repetition it might happen that we get the same value
twice in a row. This is unacceptable, since the timestamp is the only
thing we rely on in order to sort the messages.

Change-Id: I64e0255acede0af44a8510705f469e55cde6ca30

Release 7.0.10

Change-Id: Ifa8f3e61c2c8cf85c8a3ebb8e3009de7524139c7

zlogger: correct calculation of buffer map/offset

Fix calculation of buffer map & offset for the non-2KB buffer case.

Change-Id: Icb53076c5c23493ac8489d5393a53a9d7fe01c0a
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

zlogger: add clear ioctl

Change-Id: I44590f2cb68d6cb26c13ad56b88d1c202abf2ae2
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

Release 7.0.9

Change-Id: I542d30848974f5eb8c8eb810907f3042cd435cc4

Assorted fixes from product division

This commit brings following changes from product divisions aligning
codebases together:

 - introduce dbus match cache (performance optimization),
 - raise max number of possible matches,
 - minor fixes.

Change-Id: I6a58e52d48b9ac8a2eb8cb333e7d20b4e821c08c
Signed-off-by: Karol Lewandowski <k.lewandowsk@samsung.com>
[ Describe in more detail what this change brings ]
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

Fix an uninitialized var

Change-Id: I8dc8f571a2723e86308373f394b9c9ab9133f3c8

zlogger: switch from kzalloc to __get_free_pages

Since we want to mmap the memory in userspace, it's important for the
memory to be consisting of full pages. Even though this has been true in
our tests, kzalloc does not guarantee this and there is a danger of this
changing in the future kernel releases. Instead, we can use
__get_free_pages, which explicitly returns a number of contiguous pages.

Change-Id: I4db57e37fbbcd17716718ed81ef2a6b79e4b1afc

Zlogger module ioctl rework

Changed ioctl command type.
Added compat zlogger_compat_ioctl.
Switch based selector against if in ioctl.
Removed unnecessary  __init, __exit attributes.

Test program zlog_stdout should show error if ioctl and write call failed.

Change-Id: I5994899de42c09997cced28a7a159e1b4a55bf38
Signed-off-by: Arkadiusz Nowak <a.nowak3@samsung.com>

zlogger: add __user to read and write operations

Change-Id: I93dbf060a7d107a5e71f2ed02403a1356c69751f

zlogger: improve initialization

The original code used an incorrect order of initialization. It's
important to first initialize the module variables, and only afterwards
initialize the device. Otherwise, userspace might access the device
before it is ready.

Additionally, some missing failure handling is added.

Change-Id: I8c907b92c5c45568e44fdbb0e645e0b593fb898c
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

zlogger: fix various style issues

Change-Id: I42f514cac7555ecd6a4a190e9e25d2aa9e008378
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

zlogger: move userspace-shared definitions to UAPI

This commit also changes types to the UAPI-friendly ones, and names not
to litter the global namespace too much. Additionally, pid_t occurrences
are changed into __s32, as we don't want to have pid_t in kernel ABI.

Change-Id: I7658f0874e07bfb63384a401efe8f3427c7d0af3
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

Include zero-copy logging kernel module

This is basically borrowed from the Tizen TV, where the kernel module
was called vlog. Hence the attribution in the MODULE_AUTHOR field. In
our version, the module is called zlogger instead. There is also a
number of changes from the original:

- we support direct writing to the module (similar to the stdout
  redirection support in the logger module), which includes the write
  operation, various ioctl commands and the test vlog_stdout program,
- the thread block table (g_threads) works slightly differently. It is
  no longer a table, but a hashtable. This also removes a limitation on
  the max TID (previously such a limitation has been needed, as the
  table was indexed using the TID),
- the thread block table is no longer mmapable. Instead, the userspace
  is supposed to cache the received block number using thread-local
  storage, and check if it still has the block by verifying the TID
  field in the block header, which is reset when the block is lost.

Change-Id: Ia04f1add0a4904bfd92b156022c704bc62495931

kdbus: message: fix parameters passed to kdbus_pool_slice_copy_kvec()

Change-Id: Ia3a0af72110fc837e6c62c421c4de6f6d7c30c90

tests: refactor logger tests

Move testing scenarios to separate functions and enable looped operation
for detection of deadlocks, race conditions and other rearly observable
phenomena.

Change-Id: I213e5d10d9f541411c9301d147132ed9fd4af57d
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Fix build errors

-Wno-error=unused-result is applied to tests/logger since it's a test module.

[  160s] logger.c: In function 'dump_logger':
[  160s] logger.c:33:27: error: initialization of 'struct logger_entry *'
from incompatible pointer type 'char *' [-Werror=incompatible-pointer-types]
[  160s]    33 |  struct logger_entry *e = buf;
...
[  160s] logger.c: In function 'tstart':
[  160s] logger.c:24:2: error: ignoring return value of 'write', declared with
attribute warn_unused_result [-Werror=unused-result]
[  160s]    24 |  write(*fd, "child thread msg #1\nchild thread msg #2", 39);
[  160s]       |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
...

Change-Id: I1496c7591031e3c61c3b3646712d641bdbdbdfe9
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

kdbus: Add KDBUS_SUPER_MAGIC define

The define has been originally provided by kernel's uapi headers, allowing
userspace programs to detect kdbus file system type being in use.

This commit moves the define to this repository as in Tizen it is the source
of all kdbus interfaces and APIs.

Change-Id: I058b72a76e5a8a49160095f1e8b132d07b2ab921
Signed-off-by: Karol Lewandowski <k.lewandowsk@samsung.com>

Add -Wno-error=shadow build option for kdbus tests

In some different build environment (e.g., product divisions),
several strict compilation options can be added like -Werror=shadow.
Shadowing is intentional and not a bug in tests so that we avoid this build error
by adding -Wno-error=shadow

[  105s] kdbus-test.h:42:20: error: declaration of '_ASSERT_RETURN_VAL_val0_' shadows
a previous local [-Werror=shadow]
[  105s]    42 |  __auto_type const _ASSERT_RETURN_VAL_val0_ = (val0);\

Change-Id: Ia4f8a746dce0b39c896daf2e4cd1c87f62a8dadd
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

"Fix" var shadowing warnings in macros

Introduce some horrible spaghetti code to "fix" var shadowing
in recursive macros which accept code referencing vars declared
inside the macro as arguments.

Change-Id: I426f060630010a0f8a5f1eaa642e2eef6100a26c

Fix shadowing in recursive ASSERT macros

Change-Id: I509e0e481ba147fa5a29a0e1e83e6458dea41d5e

Fix some variable shadowing.

Change-Id: I1a6c0f0b51be5a7aa69455afe98ebdb21f5e0b3e

Fix further var shadowing

This macro is just used in one place so doesn't need counters.

Change-Id: I19991d02a9fa4915c87dc4803b0695778d9dad9e

Cast signed to unsigned explicitly

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

Fix disabling kdbus policy resulting in dead code

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

Remove unnecessary local variable in kdbus tests

[  230s] kdbus-test.c: In function 'main':
[  230s] kdbus-test.c:972:31: error: declaration of 't' shadows a previous local [-Werror=shadow]
[  230s]   972 |      const struct kdbus_test *t = &tests[i];
[  230s]       |                               ^
[  230s] kdbus-test.c:892:6: note: shadowed declaration is here
[  230s]   892 |  int t, ret = 0;
[  230s]       |      ^
[  230s] cc1: all warnings being treated as errors

Change-Id: I688feaa00630509363718f2d871adec3c2b951c7
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Use memcpy for unknown destination buffer size

Since we're using the source length instead
of destination size, use memcpy instead.

This is because the compiler realizes that this
should've been implemented via strcpy (without N)
and issues a warning, but then Tizen tools for
static analysis complain about regular strcpy.

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

Fix a type signedness mismatch

For size < 4, `char` was still cast to `unsigned` because of the earlier
operands, which also involved int promotion (so behaved the same as size == 4).

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

Fix a useless shadowing var

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

logger: fix log->mutex + misc_mtx ab-ba deadlock

When a forked process reopens a file for stdio logging during the first write(2)
after a fork(2), it holds log->mutex when it calls filp_open() which
in turn calls misc_open(). misc_open() takes misc_mtx.

If at the same time (after acquiring log->mutex and before misc_open()
is called) another process (e.g. dlogutil) opens the same logger
device for reading (open(2) -> misc_open() -> logger_open() ->
mutex_lock(log->mutex)), a race condition and a AB-BA deadlock occurs. To
avoid it log->mutex is released before calling make_new_file() and
reaquired after. It is safe to do so, because the log structure isn't
accessed.

Change-Id: Ibaab2947638997dca82c0e47146f77ce0f1bee57
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

logger: fix type mismatch on arm64

Change-Id: Iba4d8d53ed2c9244e6b4f2c87e626d001f42c629
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Release 7.0.6

Change-Id: Ie6042c435d4414f11a796abb69392dbb7146efed
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

packaging: move logger-test out of unnececssary directory

Change-Id: Ib194ab911aa111f1c0f224c17df911d982aa9974
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

tests: kdbus: fix memory leak

Change-Id: I90606b1364825ed163f63d7597444aa6e7a8a6cd
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

tests: kdbus: fix memory leak

Change-Id: I7c061c3a55b661db3a31dd2fd4ded60e4b37df06
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

kdbus: Replace strcpy() with strncpy()

Change-Id: Ie59bae03c5573e6a640998a6ae39f0753a98be95
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

kdbus: Replace strcat() with strncat()

Change-Id: Ie8c68c7a48bcb359a2c8e649a96f7d538ed26ad8
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

logger: make variable names match

Change-Id: Ibe70c2c0cba0a722259bae4cbe40dd865ac63448
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

logger: configure log buffer sizes

Add module parameters main_buffer_size, events_buffer_size,
radio_buffer_size and system_buffer_size to configure respective buffer
sizes. If a parameter is set to zero respective buffer is not created
whatsoever.

Change-Id: Ibaf5fe152837ded0f31f924e81ea112d9d03b63e
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

logger: configure buffer size separately

Add separate configuration for each buffer size.

Change-Id: If7134868996a20a8fd81a6bbfbe101bad6d21824
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

logger: make buffer size configurable during compilation

Change-Id: I5c50b4281cb1790248a39821c76f3ee874c76851
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

logger: fix RPi4 timestamp issue

Change-Id: Iaceffde5c5fb566a1481c2c8b48776e60057d811
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

Import and build logger test

Change-Id: Ibe00dad3ebd319875748c682a614fd517ebc8a4c
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Build and package kdbus tests

Change-Id: I797e79acf31e4e2e88f71430d7d570069850e819
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Import kdbus test

Change-Id: I02d685dbb73055daebd5bed9640937ac05e33e35
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Rename the headers package

Rename the output rpm containing logger.h and kdbus.h from
linux-tizen-modules-source-headers to linux-tizen-modules-headers since
the output rpm contains only headers and no other source code.

Change-Id: I9d471a074e48bdaaa382c06e935e0c4551516511
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Move and export uapi headers

Move logger.h and kdbus.h uapi headers to a separate directory
and export them to /usr/include/linux and
/usr/src/linux-tizen-modules-source.

Change-Id: Ie3fd39f44baf33552cc742f0c13228b1e87b9333
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Release 7.0.4

This release adjusts logger.ko to be able to work with older (<3.18)
kernels.

Change-Id: I8230de61a67096b0dcadbe12fc5d34f68112b93e

Replace copy_from_iter() with copy_from_user()

iov_iter structure and associated function have been introduced in
v3.18. Logger needs use copy_from_user() and iterate over IO vectores
manualy.

Change-Id: Ifa46c67da65195611adc50b1dc6f0be66daac517
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Import stdio handling code to logger_aio_write()

Copy the stdio handling code from logger_write_iter()

Change-Id: I94be88c5db2e7482106ddd537d763726a89aad5a
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Discard data from stdio

Accept and discard unstructured data coming from stdio.

This commit is mainly to make the later changes more readable.

Change-Id: I0428eef0b3f9517e14b8de282003436e5296b66d
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Import old logger_aio_write()

Copy logger_aio_write() and two other functions from Android logger
driver in v3.10.65. Use them when compiling against old kernels.

Change-Id: I1e2cb304106e48ffad80f03dc7c9ae899cd228f4
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

kdbus: domain: drop overwritten assignment

Change-Id: Idf43dadcbee8156b1099f30dfd8ebc40c7c5bb45

packaging: generate headers package

Change-Id: Icfd89649aeb8d723a7f70499cad16d361b83ee68
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

kdbus: use kdbus.h header from this repository

… as opposed to the header from the kernel headers package.

Change-Id: I25c86328cc16aba1099214802acb661291e31258
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

Bump version due to kdbus inclusion

Other packages wanting kdbus might depend on particular version.

Change-Id: Iab701418f25314b5035d8ece0bfd95159b5daba6

Import kdbus

For the record, these files come from the
git://git.tizen.org/platform/kernel/linux-rpi.git repository, from the
be5781a07d48f8fd62633310fea57963aea1595d commit.

Change-Id: Ia3be026cf4a97fce32bbc98dc4151d75d670b186
Co-authored-by: Karol Lewandowski <k.lewandowsk@samsung.com>
Signed-off-by: Mateusz Majewski <m.majewski2@samsung.com>

Keep only source package

This commit drops all public-tizen customizations and keep only source
package generation.

Public Tizen linux-tizen-modules will use this package to build code
and will add its specific configuration.

Change-Id: I633f869530d6b0a34c27ff4d0d137bc7eaac262f