Adjust scope of map info list variable

The milist, map info list is allocated for each process and freed
by get_trib_mapinfo(), but the pointer life cycle is not well
shown. Adjust variable scope for each process, and add comment
about its free in get_trib_mapinfo().

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

free entire geminfo list

The geminfo is list, so it should free all list element. Add
clear_geminfo() function to free all list element and use it.

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

packaging: Update version to 0.2.0 to show drm render gpu node support

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

Add imported to geminfo

When imported of geminfo is 1, then it means the gem is not
allocated from the gpu driver, instead, it is imported from
other driver as dma-buf fd. In render drm driver case, it is
usually imported from kms drm driver.

The information will be used later.

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

Remove mmap size of drm render node as like drm node

If drm render is used as gpu, its mmap size needs to be removed
from total rss because it is already in 3d graphic memory.
Remove mmap size of drm render node.

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

Add gpu_gem_info sysfs node of drm render to 3d graphic memory

Some soc has its gpu driver in drm subsystem. In the case, gem
buffer of the gpu drm driver is 3d graphic memory, so add the
render node gpu_gem_info to 3d graphic memory.

NOTE2: the gpu_gem_info requires kernel porting for gem_info for
each process as like below link:
  https://git.tizen.org/cgit/platform/kernel/linux-rpi/commit/?h=tizen&id=dc9aad9594b2c8a18b15f6315b2d3ff1ff3dbe50

NOTE2: currently, sum of gpu_gem_info pss for each process is used
but the meory can be imported from drm kms node gem buffer. In the
case, TOTAL size result can be larger than real memory usage.

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

Divide functions to open and to parse gem_info sysfs node

To support other gem_info sysfs node, divide open function and
parse function for gem_info.

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

get mmap size of drm gem with all drm card node

There can be more than one drm card device. For example, in rpi4,
card0 is gpu and card1 is display controller node. Get mmap size
of drm gem with all drm card node.

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

Use same dri node for gem_info and gem_name

For rpi4, it uses gem_info from /sys/kernel/debug/dri/1/ but
gem_name is read from /sys/kernel/debug/dri/0, so total
gem size is always 0. Fix to use same dri sysfs path for
both gem_info and gem_name.

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

Typo correction

Typo correction line number 1430 in memps.c file
verbos -> verbose

Change-Id: Iaf58a97aa0456fd1f6328a725c11f438c0075962
Signed-off-by: Yunhee Seo <yuni.seo@samsung.com>

Add gem_info path in dri card1

In some target, display dri card can be in 1 instead of 0. Add
gem_info path in dri card1 if card0 is failed.

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

Use proper printing format for struct statfs

Use cross-platform printing for combined cases.

Change-Id: Ic2789b42c567e950813a404fc72e677ced1ef001
Signed-off-by: Sungguk Na <sungguk.na@samsung.com>

Use new zram node first

For zram memory usage, memps uses deprecated old node first, and
if fails, then try newer zram stat node. Even it works fine, but
kernel with old node marked as deprecated, it gives kernel log
about deprecation. Use new zaram stat node first.

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

Consider 64-bit address

The result for 'memps <pid>' shows address of maps only with
32-bit address. So on 64-bit, the printing style is messy.
Fix the result style considering 64-bit address.

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

fix length of pss for '-s <pid>'

The pss is printed with 18 length space for '-s <pid>' but it is
too long. Fix the length of pss as 8 like other options.

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

Read pid_max from kernel

From systemd v243, 'kernel.pid_max' is bumped to 4194304 on 64-bit
system. This change reads pid_max from proc and uses it for pid
validation.

Change-Id: Ief105ef09c7d89bc443878c6d66478f9c5ccc61d
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

fix printing style of title and data with fixed length

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

Update usage with using same whitespace and adding -r option

There were tabs in usage print, so usage shows different indent.
Use same whitespace for all the usage options. Also add missing
'-r' option explaination showing rss of all processes.

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

remove possible  unsigned integer overflow for return value

Static analysis tool warns about possible overflowed return value.
During calculation, check the possible overflow for return value.

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

Reset errno when cmd file is not exists

 Sometime, there is a timing issue.
 If the process is terminated before calling readdir() and
 the directory of the process is removed, the file fails to open.
 If the file fails to open, errno is set, and the logic is terminated
 without reading other process information.

Change-Id: I332e076ce035bd6d66175d6886ce223aa964513a
Signed-off-by: Seungha Son <seungha.son@samsung.com>

Add PSS column in result table

Change-Id: I3b078d8e0b0cc1053b9489946b21cf128f2d7b6c
Signed-off-by: Seungha Son <seungha.son@samsung.com>

Get properly zram's used size.

To get zram_used_total propely which is deprecated in up-to-date linux
kernels, zram size will be retrieved by two-paths: first try with
deprecated node for backward-compatibility and if fails then use the
new node.

Change-Id: If9a8ea131b9fbed8165dbbf5d743cbed40bf01ef
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>

adjust print style for -s option

Adjust thee whitespace style for "memps -s <pid>".

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

Adjust -Werror option

Change-Id: Ia90be5c9b9b637f632c9c4b0dc9f81ec5be47dc4
Signed-off-by: Seungha Son <seungha.son@samsung.com>

Bound maximum index for accessing array

To avoid too large index to access buffer array, bound maximum
index.

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

Use proper printing format for struct statfs

If __USE_FILE_OFFSET64 is defined, struct statfs has 64-bit
elements even in 32-bit arch. Use proper printing format as
the definition.

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

Use enough arrary size for snprintf()

Use enough arrary size for snprintf() to avoid format-truncation.

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

Always check return value of fgets()

Fix to check return value of fgets().

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

Fix wrong if condition

Fix wrong if condition having both "&&" and "||" without proper
parentheses.

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

Clear freed pointer to null

To avoid possible use-after-free, clear frred pointer to null.

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

Fix not to use uninitialized variable

From get_mem_info(), buf is used without initialization in error
path. Fix not to use uninitialized variable, buf.

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

[PATCH] Support 3D memory usage value for Mali midgard GPU Driver

The usage of 3D memory is differ from GPU series
In case of Mali utgard, it comes from smaps of process
In case of Mali midgard, it comes form specific debug node

Change-Id: Iaef54537d107f49763ef8f8508c6787a73851a81
Signed-off-by: Minyoung, Song <minyoung.song@samsung.com>

Fix Coverity issues

22581 Checked_return
36976 Resource_leak

Change-Id: I983248d1a48d8029f4a8cca2d2eb0997af0410de
Signed-off-by: Sungguk Na <sungguk.na@samsung.com>

fix build warning

Change-Id: I75d689f1fd1bcb5881eb0409a43819f37b3ceda2

Consider oom_score_adj in case of non-root permission

When memps is executed without root permission,
it can't read oom_score_adj value.
The latest kernel doesn't support reading oom_score_adj value without root.
In case of that, memps should read oom_score value.

Change-Id: Ieb24bfcf7ef065f1230dd7c13698cd52264abe91
Signed-off-by: ByungSoo Kim <bs1770.kim@samsung.com>

Adapt ASLR to memps

Change-Id: I1d0210ec580c1b6e483b70e8eba8ab382e517be2
Signed-off-by: Sungguk Na <sungguk.na@samsung.com>

Fix freeing null pointer 'glist'

WGID : 248006
glist pointer can have null value when memps fails to open gem_info debugfs
node in load_geminfo(). This patch adds null check to prevent freeing
null pointer.

Change-Id: I0a7143ef5405fadfb2e39131ddc5db1f58bea135
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix coding style issues detected by rule checker

Change-Id: I0b2f6237f733dc300c7fb8c0f223fc08d8cacd07
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Use debugfs node for accessing GEM information

drm_proc.c was removed in linux v3.11

Change-Id: I0ff9e92dea03c654ddc0c871f9bd4388ca5afc7b
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Prevent buffer overflow with sscanf

By allocating string buffer, prevent buffer overlow issue with sscanf.

Change-Id: I5655f1b7049dc457b525dfbc2d351b67616b3f88
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Replace deprecated readdir_r() with readdir()

Change-Id: I88e7060aecfd0cc982fd995f66a30f643b4fd292
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix smaps parsing logic

Memps fails to parse smaps and gets crash when kernel shows anon name
information in /proc/smaps, because kernel adds additional line to
print the information.
Instead of using fixed smaps line count, this patch checks every line
to find VMA address line.

Change-Id: I5eaad65804f9cae67bfc62b6625b4b7da0eb27aa
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix typos

Change-Id: Ibbfd8c2418959d35b890dd53a838d7994ea60333
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Surpress Svace warning

And also change type of rss variable to unsigned long

Change-Id: I2de5440c7ac8eaefc5acafb84244194e435bad14
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix fixed value of smaps line count

Vma line was missed, so we need to add 1 on each hardcoded value.

Change-Id: I735bc0aaa312e034655534a78b456a35c5ce9fdb
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix build warnings

Change-Id: Icf8062b8f35b788a9d58ed2a7d207ef80a98bdaa
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix smaps parsing on 64bit kernel

In previous patch, there is a issue that memps truncates pathname
when it includes blank space.

Change-Id: I72770bc84c4f11b114c2dcdaad69314bacb113e4
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Fix smaps parsing issue on 64bit kernel

Signed-off-by: Minyoung, Song <minyoung.song@samsung.com>
Change-Id: Ib1b439805dab0475fce70f1fa41906691fd5790a

Refactoring smaps parsing logic

To be compatible with various kernel version

Change-Id: Ie494cebc8581612271acb6eb584437563be1017a
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Modified kernel version check to support 4.x kernel smaps

Change-Id: I0878424145df4920ef09d4f4b659808ddc26c874
Signed-off-by: sungguk <sungguk.na@samsung.com>

Bug fix for guaranteeing NULL-termination of strncpy

Change-Id: I8ef500d9007f0b151fcc5558cc5faab84dbb9d7c
Signed-off-by: Minyoung, Song <minyoung.song@samsung.com>

Modified licensing header

Signed-off-by: Minyoung, Song <minyoung.song@samsung.com>
Change-Id: I0388c60d0e30aa2fb5a47101a52f268d5f56ccbd

Fixed gem_info parsing logic to find and sum from all rows of gem_info

Signed-off-by: Minyoung, Song <minyoung.song@samsung.com>
Change-Id: Ia4af00cfa8542352d6bc2f9a8b096ccb8e99ce82

Fix svace issues

Change-Id: I9182ba8d42006a5388dacad9144d9cb2b6a0b2b1
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Merged latest memps code providing swap and cgroup usage

Change-Id: I44624fa22a9bc9f9703c55d919cf1fdf5b9cca42
Signed-off-by: Minyoung, Song <minyoung.song@samsung.com>

Fix Svace issues

15723 - DIVISION_BY_ZERO
25769 - DIVISION_BY_ZERO
25770 - DIVISION_BY_ZERO

Change-Id: Icfed0528b6891af56f51c4c7864652ae01801ef8
Signed-off-by: Hyeongsik Min <hyeongsik.min@samsung.com>

Initial code commit

* memps is a tool which provides information about the memory usage of processes (RSS, PSS, memory map of process, clean and dirty pages), tmps and graphics memory usage
* Added source files and CmakeLists file
* Added packaging and license files

Change-Id: I5dc2357ff229ad876ec17919a2e9df51488b89e7
Signed-off-by: Prajwal A N <an.prajwal@samsung.com>

Initial empty repository