Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Rework code related to options -s/-e

Timestamps entered as a number of seconds since the Epoch are now used
as they are and no longer converted to a broken time representation, so
that even month and year are taken into account when current record time
is compared to those timestamps.

The record which is exactly at the ending time is now displayed.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Update manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Strengthen tests made on args given to options -s/-e (#350)

Make additional tests on the argument following options -s and -s used
with sar and sadf to make sure it is actually a timestamp before using
it as such.
Previously, in the examples below, xfile (5 chars) and my_xfile (8
chars) were considered as timestamps in parse_timestamp() function,
although they actually are data files names.

sadf -s xfile
or
sadf -s my_xfile

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: Replace time_start/time_end with start_time/end_time

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Replace time_start/time_end with start_time/end_time

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update sar and sadf usage message

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update sar and sadf manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'NatePW-sadf_epoch_time'

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Remove unneeded break after return in switch case

Replace potentially dangerous localtime function

Compare UTC time properly format agnostic

Add epoch type to start and end times

sar: Make sure timestamps are always displayed in local time

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar: Fix help message

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: XML: A_PWR_BAT: Fix XSD document

Variation of battery capacity may be negative. So update XSD document to
accept negative values.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: PCP: Fix pmiID number for three USB metrics

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Generate batteries statistics in PCP format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Add f_count_new() function

This function is aimed at creating a list of battery names that will
later be used to create PCP instances.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Use colors to display negative values

Two thresholds are defined: -10.0 and -5.0.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Change field name from "%cap/min" to "cap/min"

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar: A_PWR_BAT: Change arrow type for status "Not charging"

Use East Arrow symbol (right arrow) instead of South Arrow for status
"Not charging".

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: JSON: A_PWR_BAT: Add missing check

Make sure that status code is not greater than upper limit before using.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in RAW format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Move some definitions from common.h to rd_stats.h

proc and sys files which are used only by sadc are moved to rd_stats.h
file.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in SVG format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in JSON format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in XML format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Update DTD and XSD documents

Update XML documents to deal with batteries statistics.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in db/ppc formats

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Change column name from "sts" to "status"

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

common.c: Display sign with cprintf_f() function

Update cprintf_f() function so that the sign (+ or -) can be displayed
if requested.
This is intended for values used for batteries capacity variation.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Makefile: Fix dependencies

Files pcp_def_metrics.c and pcp_stats.c depend on sa.h which depends
itself on common.h, rd_stats.h and rd_sensors.h.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Display battery statistics with sar

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Add f_count and f_read functions

Add functions used to count number of batteries installed and to read
batteries statistics.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Add new option and basic definitions for sar and sadc

Add new activity A_PWR_BAT.
It will enable the user to monitor battery capacity, variation and
status.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Cosmetic changes (enumerate values in header files when applicable)

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: RAW: Fix extra space when no TZ printed

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'NatePW-correct_dbpcc_timestamp'

Merge branch 'correct_dbpcc_timestamp' of https://github.com/NatePW/sysstat into NatePW-correct_dbpcc_timestamp

sadc: Add another overflow check

Make sure resulting value won't overflow UINT_MAX before allocating
memory.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Try to avoid displaying healthy metrics values in "red" (#177)

Percentage values are now displayed in specific colors when they are
greater or lower than a predefined limit depending on the metric's
meaning.
This means that, e.g. CPU %idle value will no longer be printed in red
if the CPU is actually idle.

Specific colors are now used when:
* values are greater than 90% (or lower than 10%),
* values are in the range 75% to 90% (or 25% to 10%).

This is probably still not perfect but should eliminate many questions.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Fix extra space when no TZ printed

Merge pull request #346 from Kwstubbs/Kwstubbs/add-codeql-workflow

Add CodeQL workflow

Add CodeQL workflow

Starting sysstat 12.7.2

configure script updated.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sysstat-12.7.1

sysstat version 12.7.1 final packaging.
Changelog added.
spec and lsm files updated.

This version adds some small new features, like option -H to mpstat to
handle vCPU physical hotplug, or true time mode support to sadf's SVG
output.
It also fixes several bugs, one of which has been the object of a
security advisory (CVE-2022-39377).

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update NLS translations

Sync with the Translation Project.
Add new Georgian translation.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Make sure values to be compared are unsigned integers

It seems safer to make sure that input values are unsigned int before
casting them to unsigned long long and making the comparison.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests again

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add more overflow checks

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge pull request from GHSA-q8r6-g56f-9w7x

Fix size_t overflow in sa_common.c (GHSL-2022-074)

Merge pull request #342 from shaleen-bathla/master

mpstat: return void from rw_mpstat_loop

mpstat: return void from rw_mpstat_loop

Previous commits removed the usage of return value
from rw_mpstat_loop()
Hence change return value from int to void.

Signed-off-by: Shaleen Bathla <shaleen.bathla@oracle.com>

Fix size_t overflow in sa_common.c (GHSL-2022-074)

allocate_structures function located in sa_common.c insufficiently
checks bounds before arithmetic multiplication allowing for an
overflow in the size allocated for the buffer representing system
activities.

This patch checks that the post-multiplied value is not greater than
UINT_MAX.

Signed-off-by: Sebastien <seb@fedora-2.home>

Update non regression tests

...and add a new non regression test for mpstat -H.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

mpstat: Fix missing sample after physically hotplugging vCPU

When a vCPU is physically hotplugged, mpstat with its option -H skips
one line of statistics:

$ ./mpstat -H -P ALL 5 5
Linux 5.4.17-2136.310.7.1.el8uek.x86_64 (localhost.localdomain) 10/10/2022 x86_64 (3 CPU)

10:11:22 AM CPU %usr %nice %sys %iowait %irq %soft %steal %guest %gnice %idle
10:11:27 AM all 0.07 0.00 0.07 0.00 0.07 0.00 0.00 0.00 0.00 99.80
10:11:27 AM 0 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00
10:11:27 AM 1 0.20 0.00 0.20 0.00 0.20 0.00 0.00 0.00 0.00 99.40
10:11:27 AM 2 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00

10:11:27 AM CPU %usr %nice %sys %iowait %irq %soft %steal %guest %gnice %idle
10:11:32 AM all 0.00 0.00 0.07 0.00 0.07 0.00 0.00 0.00 0.00 99.87
10:11:32 AM 0 0.00 0.00 0.00 0.00 0.20 0.00 0.00 0.00 0.00 99.80
10:11:32 AM 1 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00
10:11:32 AM 2 0.00 0.00 0.20 0.00 0.00 0.00 0.00 0.00 0.00 99.80
[ 1154.848226] CPU3 has been hot-added
[ 1154.862315] smpboot: Booting Node 0 Processor 3 APIC 0x3
[ 1154.863956] kvm-clock: cpu 3, msr adc010c1, secondary cpu clock
[ 1154.884436] KVM setup async PF for cpu 3
[ 1154.886055] kvm-stealtime: cpu 3, msr 13b9b2040
[ 1154.887657] Will online and init hotplugged CPU: 3

10:11:32 AM CPU %usr %nice %sys %iowait %irq %soft %steal %guest %gnice %idle
<<<<<<< No sample for 10:11:37 >>>>>>>
10:11:42 AM all 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00
10:11:42 AM 0 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00
10:11:42 AM 1 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00
10:11:42 AM 2 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00
10:11:42 AM 3 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 0.00 100.00

This patch makes sure that we get a line of stats for every interval of
time, even when a vCPU is physically hotplugged.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'shaleen-bathla-master'

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

mpstat : add -H option to handle vCPU physical hotplug

For virtualized environments, physical CPU hotplug can occur.
OR to be more precise vCPU physical hotplug can occur.

Currently, mpstat does not detect physical hotplug when running with an interval.
Example : mpstat -P ALL 5
When -H option is given, display stats of physically hotplugged vCPU also.
Example : mpstat -H -P ALL 5

This is done only when -H option is provided to remain legacy compatible.

Also updated the man page describing the new option.

Signed-off-by: Shaleen Bathla <shaleen.bathla@oracle.com>

mpstat : Add reset parameter to salloc_mp_struct()

To reset stats structs memory, add parameter to salloc_mp_struct().

This will be used in the next commit for introducing -H option for
handling vCPU physical hotplugs.

Use realloc calls which will help in resizing the structs also.
if reset is TRUE, memory will be cleared else not cleared

Signed-off-by: Shaleen Bathla <shaleen.bathla@oracle.com>

configure: Update configure file to deal with newer autoconf version

Rename configure.in file to configure.ac.
Run autoupdate to upgrade configure file.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

configure.in: Look for sys/wait.h include file

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

pidstat: pidstat should return the exit code of the child process (#339)

When a process is monitored with option -e, pidstat should return the
exit code of this process and a default value of 0.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

PCP: Fix timestamps written to PCP archive file

We should not consider true time / local time or UTC modes.
The timezone in which data will be displayed should be selected using
options -z / -Z with pmdumplog command.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge get_locatime() and get_gmtime() functions

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update DTD and XSD documents

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: Display timezone for all output formats

The timezone is now displayed even in local time (-T) and in true time
mode (-t).

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar: Fix typo in manual page

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

SVG: Fix timestamps displayed in true time mode

Hours, minutes and seconds are saved as 8-bit values in structure
record_header. This is not enough to avoid a possible overflow when
calculating X-axis graduations.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf_misc.c: Remove *tzname[] declaration

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: Print time zone instead of UTC in true time mode

This patch expands the scope of patch 217ee12 for all formats generated
by sadf.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'ObiWahn-master'

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

doc-fix: remove `-I int_list` from man-page and help.

Interrupts are can be specified via the `--int=int_list` option.

SVG: Print time zone instead of UTC in true time mode (#331)

sadf -g -t should not display "UTC" but the real time zone of the user
who created the datafile.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

README: Display old syntax if option --now not available (#334)

Display the commands that can be used when option --now is not supported
by systemd.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

README: consolidate systemctl commands

Turn off gcc's tree-slp-vectorize option

sadf crashes in non regression test nr. 00610 with a segmentation fault:

$ sadf -c tests/data-10.3.1 >/dev/null
file_magic: OK
HZ: Using current value: 100
file_header: OK
file_activity: A_CPU A_PCSW A_IRQ A_SWAP A_PAGE A_IO A_MEMORY A_HUGE A_KTABLES A_QUEUE A_DISK A_NET_DEV A_NET_EDEV A_NET_NFS A_NET_NFSD A_NET_SOCK A_NET_IP A_NET_EIP A_NET_ICMP A_NET_EICMP A_NET_TCP A_NET_ETCP A_NET_UDP A_NET_SOCK6 A_NET_IP6 A_NET_EIP6 A_NET_ICMP6 A_NET_EICMP6 A_NET_UDP6 A_PWR_CPU A_PWR_FAN A_PWR_TEMP A_PWR_USB A_FS OK
Statistics:
HRHCHnuunu...uuuunununu...uu......uu...n.n.n.n.Segmentation fault (core dumped)

It turns out that the problem is due to gcc optimizations made with
option -O2. Unsetting tree-slp-vectorize flag fixes that problem.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add "debug" target to iconfig script

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update sysstat(5) manual page

Explain new REPEAT_HEADER variable.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add REPEAT_HEADER variable to sysstat configuration file (#332)

This variable will make it easier to tell sa2 script to insert header
lines in the reports.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add "local" and "xlocal" targets to iconfig script

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update sar manual page

Explain new environment variable (S_REPEAT_HEADER).

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar: Add new environment variable S_REPEAT_HEADER (#332)

Add new environment variable S_REPEAT_HEADER.
This variable will contain the number of lines after which a header has
to be displayed by sar when stdout is not a terminal.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add support for option -t with sadf's SVG output (#331)

When sadf was used to display SVG graphs, it was not possible to display
the timestamps (on the X axis) in the same locale as that of the file
creator. This was because option -t was not supported by sadf in SVG
output format. This patch fixes that.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sa_conv.c: Use sizeof() instead of a constant

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: Make structures' size from older versions immutable

The size of structures used by older sysstat versions should not depend
on the size of current sysstat version.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'bmwiedemann-master'

Declare sadc dependency on libsyscom.a

Without this patch, make -j1 would try to link sadc
and fail to find libsyscom.a

Fixes #329

Signed-off-by: Bernhard M. Wiedemann <bwiedemann@suse.de>