- Jul 13, 2021
-
-
George Liu authored
Update meson.build and add wrapfiles so that all dependencies are handled as subprojects. This allows builds completely outside of bitbake or an SDK. Tested: built phosphor-debug-collector repo successfully using `meson -Dtests build`. Signed-off-by:George Liu <liuxiwei@inspur.com> Change-Id: I51bae90f4fac113270e259b7acac3630b5ec6778
-
George Liu authored
- Add ramoops file monitor and support to enable ramoops data collection for kernel dump. - Systemd will collect ramoops data and write it into /var/lib/systemd/pstore when a system kernel oops/panics. - Today, need to grab everything in that directory, put it in a dump and then delete everything in that directory. Tested: - Simulate a kernel panic by creating similar data in the expected location. `dmesg > /var/lib/systemd/pstore/dmesg1.txt` `dmesg > /var/lib/systemd/pstore/dmesg2.txt` `dmesg > /var/lib/systemd/pstore/dmesg3.txt` `dmesg > /var/lib/systemd/pstore/dmesg4.txt` tar -tvf obmcdump_1_248.tar.xz: drwxr-xr-x 0/0 0 1970-01-01 00:01:32 obmcdump_1_248/ -rw-r--r-- 0/0 21239 1970-01-01 00:01:32 obmcdump_1_248/dmesg1.txt -rw-r--r-- 0/0 21239 1970-01-01 00:01:32 obmcdump_1_248/dmesg2.txt -rw-r--r-- 0/0 21239 1970-01-01 00:01:32 obmcdump_1_248/dmesg3.txt -rw-r--r-- 0/0 21239 1970-01-01 00:01:32 obmcdump_1_248/dmesg4.txt -rw-r--r-- 0/0 162 1970-01-01 00:01:32 obmcdump_1_248/dreport.log -rw-r--r-- 0/0 294 1970-01-01 00:01:31 obmcdump_1_248/os-release -rw-r--r-- 0/0 278 1970-01-01 00:01:31 obmcdump_1_248/summary.log Signed-off-by:
-
- Jun 25, 2021
-
-
Tim Lee authored
Symptom: "Verify Maximum BMC Dump Creation" test item got failed sometimes. This test item expect that dump space is enough to create a new BMC dump file. But, dump manager doesn't think dump space is enough to create according to current size of dump directory. Root cause: dump manager is using std::filesystem::file_size to calculate total size of dump log files in dump directory. Then total size is divided by 1024 convert to KB. However, test item is using linux command "du -s" to calculate total size of dump log files in dump directory. The calculation result is different between dump manager and test item then cause test got failed sometimes. Solution: calculate each dump file with std::ceil() and convert size to KB then calculate total dump files size in dump directory. That's can reduce size difference of total dump files between dump manager and test item then avoid this test got failed. Tested: Run 3~4 times robot Verify_Maximum_BMC_Dump_Creation redfish/managers/test_bmc_dumps.robot Verify Maximum BMC Dump Creation :: Create maximum BMC dump and ve... | PASS | 1 critical test, 1 passed, 0 failed 1 test total, 1 passed, 0 failed Signed-off-by:Tim Lee <timlee660101@gmail.com> Change-Id: Ic1a6d941d12516678ff21789355941653d319858
-
- May 17, 2021
-
-
Bruce Mitchell authored
Discord has more users. IRC traffic has ground to a halt. Signed-off-by:Bruce Mitchell <bruce.mitchell@linux.vnet.ibm.com> Change-Id: I4f8b81652d16dc5df3d6c28cd1bf7969080f06b5
-
- May 10, 2021
-
-
Patrick Williams authored
Change I17a8d7479556596a3cf252b3f4eae9c8df547189 will change how sdbus++ generates names which start with an acronym. Prepare for this by keying off the SDBUSPP_NEW_CAMELCASE define to use the new format. Changes: vSPString -> vspString Signed-off-by:Patrick Williams <patrick@stwcx.xyz> Change-Id: Idec24c3e2d58abca39fe236051f28567b626e260
-
- Apr 14, 2021
-
-
Jayanth Othayoth authored
Update to the latest OpenBMC clang-format. Signed-off-by:Jayanth Othayoth <ojayanth@in.ibm.com> Change-Id: I54010d3c756af4c85786285644038d449730612b
-
Jayanth Othayoth authored
Use the real filesystem library, and drop support for building with experimental under c++14. Signed-off-by:
-
- Apr 09, 2021
-
-
Chirag Sharma authored
Changes: -Added time. -Added bmc serial number -Added dump entry Result: Tested the dump header using busctl command and dreport script. Able to add dump header on top of the <obmcdump>.tar.xz file. In hexdump o/p can see the time, serial number and entry details. Separated out the header from the output file using dd command: dd bs=628 skip=1 if=<obmc dump>.tar.xz of=openbmc.tar.xz The new output file doesn't have the header now and can be unzipped. Post unzipping the files in the dump can be seen. Signed-off-by:Chirag Sharma <chirshar@in.ibm.com> Change-Id: Ia8af50cebb09350d93a631531495081c24667f26
-
- Mar 26, 2021
-
-
PriyangaRamasamy authored
Collect bad VPD as a part of user intiated BMC dump. In openpower-vpd-parser we collect IBM vpd using a feature flag "ibm-parser". So the vpd we collect are specific to ibm-systems. Hence placing the badvpd plugin in tools/dreport.d/ibm.d rather than in tools/dreport.d/plugins.d. The badvpd plugin will get installed via phosphor-debug-collector recipe file from meta-ibm layer. The symlink creation to the right type of plugin directory(say - pl_user.d/pl_elog.d/pl_checkstop.d/pl_core.d) will also be done from the meta-ibm layer in phosphor-debug-collector recipe file. Tested on simics: root@rainier:~# mkdir /tmp/bad-vpd root@rainier:~# dreport -v Wed Mar 3 04:43:18 UTC 2021 Name: obmcdump_00000000_1614746598.tar.xz Wed Mar 3 04:43:18 UTC 2021 Epochtime: 1614746598 Wed Mar 3 04:43:18 UTC 2021 ID: 00000000 Wed Mar 3 04:43:18 UTC 2021 Type: user Wed Mar 3 04:43:18 UTC 2021 INFO: Collected disk usage Wed Mar 3 04:43:18 UTC 2021 INFO: Collected failed services Wed Mar 3 04:43:19 UTC 2021 INFO: Collected Journal pretty log Wed Mar 3 04:43:19 UTC 2021 INFO: Copied Memory info /proc/meminfo Wed Mar 3 04:43:19 UTC 2021 INFO: Collected top Wed Mar 3 04:43:19 UTC 2021 INFO: Copied Bad VPD /tmp/bad-vpd Wed Mar 3 04:43:19 UTC 2021 INFO: Copied OS release info /etc/os-release Wed Mar 3 04:43:19 UTC 2021 INFO: No elog entries Wed Mar 3 04:43:19 UTC 2021 INFO: Collected inventory Wed Mar 3 04:43:19 UTC 2021 INFO: Copied OBMC console log /var/log/obmc-console.log Wed Mar 3 04:43:19 UTC 2021 INFO: Collected ip addr Wed Mar 3 04:43:19 UTC 2021 INFO: Collected ip link Wed Mar 3 04:43:19 UTC 2021 INFO: Collected PRI-Flash: U-Boot environment variables Wed Mar 3 04:43:19 UTC 2021 INFO: Copied GUARD Records /var/lib/phosphor-software-manager/pnor/prsv/GUARD Wed Mar 3 04:43:19 UTC 2021 ERROR: /etc/alt_fw_env.config does not exist Wed Mar 3 04:43:19 UTC 2021 INFO: Copied CPU info /proc/cpuinfo Wed Mar 3 04:43:19 UTC 2021 INFO: Collected uptime Wed Mar 3 04:43:19 UTC 2021 INFO: Collected BMC State Wed Mar 3 04:43:19 UTC 2021 INFO: Collected Chassis State Wed Mar 3 04:43:19 UTC 2021 INFO: Collected hostnamectl Wed Mar 3 04:43:19 UTC 2021 INFO: Collected Host State Adding Dump Header :/usr/share/dreport.d/include.d/gendumpheader Wed Mar 3 04:44:08 UTC 2021 Report is available in /tmp Wed Mar 3 04:44:08 UTC 2021 Successfully completed Check the dump tar file generated in /tmp directory. After skipping 628 bytes of dump header in tar, untar it. The files in /tmp/bad-vpd directory will get collected. Signed-off-by:PriyangaRamasamy <priyanga24@in.ibm.com> Change-Id: I47f6dde077f9852cc3061cc8c03819c12749337c
-
- Mar 25, 2021
-
-
Dhruvaraj Subhashchandran authored
Problem: The wrong identifier type was passed to the host dump transport for deleting the dump, so an exception is thrown. Fix was needed in two places 1) Dump identifier number was old in resource dump manager 2) PLDM_FILE_TYPE_RESOURCE_DUMP_PARAMS was given as dump identifier instead of PLDM_FILE_TYPE_RESOURCE_DUMP. Test: - Created a new dump - Invalidated the dump Result: Dump is deleted from the host. Signed-off-by:Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com> Change-Id: If68f415439d8322f47fa6a6bc7a9b230e9f67fb1
-
- Feb 23, 2021
-
-
PriyangaRamasamy authored
Due to recent changes made in arm-sdk, the FORTIFY_SOURCE security flag enables optimization level 2. The repo has a default debug level of optimization which conflicts with FORTIFY_SOURCE flag. In this commit the default build type is set to debugoptimized and also made sure to disable the FORTIFY_SOURCE when we need to build without any optimization. Signed-off-by:
-
- Feb 16, 2021
-
-
Dhruvaraj Subhashchandran authored
Add support for deleting resource dumps in host memory. Change requestDelete to accept dump type. Update system dump delete to pass dump type. Signed-off-by:
-
- Feb 12, 2021
-
-
Dhruvaraj Subhashchandran authored
The emit_object_added was added in the base entry class so all information was not present in the InterfaceAdded. In the below example the VSP String and password are null To fix this moved emit_object_added to derived classes so InterfaceAdded signal will come only after the creation of complete object. Without fix: Sender=:1.41 Path=/xyz/openbmc_project/dump Interface=org.freedesktop.DBus.ObjectManager Member=InterfacesAdded UniqueName=:1.41 MESSAGE "oa{sa{sv}}" { OBJECT_PATH "/xyz/openbmc_project/dump/resource/entry/1"; ARRAY "{sa{sv}}" { DICT_ENTRY "sa{sv}" { STRING "org.freedesktop.DBus.Peer"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "org.freedesktop.DBus.Introspectable"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "org.freedesktop.DBus.Properties"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Time.EpochTime"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "Elapsed"; VARIANT "t" { UINT64 0; }; }; }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Object.Delete"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Dump.Entry"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "Size"; VARIANT "t" { UINT64 0; }; }; DICT_ENTRY "sv" { STRING "Offloaded"; VARIANT "b" { BOOLEAN false; }; }; DICT_ENTRY "sv" { STRING "OffloadUri"; VARIANT "s" { STRING ""; }; }; }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Common.Progress"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "Status"; VARIANT "s" { STRING "xyz.openbmc_project.Common.Progress.OperationStatus.InProgress"; }; }; DICT_ENTRY "sv" { STRING "StartTime"; VARIANT "t" { UINT64 1612968832; }; }; DICT_ENTRY "sv" { STRING "CompletedTime"; VARIANT "t" { UINT64 0; }; }; }; }; DICT_ENTRY "sa{sv}" { STRING "com.ibm.Dump.Entry.Resource"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "SourceDumpId"; VARIANT "u" { UINT32 0; }; }; DICT_ENTRY "sv" { STRING "VSPString"; VARIANT "s" { STRING ""; }; }; DICT_ENTRY "sv" { STRING "Password"; VARIANT "s" { STRING ""; }; }; }; }; }; }; With Fix: Type=signal Endian=l Flags=1 Version=1 Cookie=68 Sender=:1.286 Path=/xyz/openbmc_project/dump Interface=org.freedesktop.DBus.ObjectManager Member=InterfacesAdded UniqueName=:1.286 MESSAGE "oa{sa{sv}}" { OBJECT_PATH "/xyz/openbmc_project/dump/resource/entry/4"; ARRAY "{sa{sv}}" { DICT_ENTRY "sa{sv}" { STRING "org.freedesktop.DBus.Peer"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "org.freedesktop.DBus.Introspectable"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "org.freedesktop.DBus.Properties"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Time.EpochTime"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "Elapsed"; VARIANT "t" { UINT64 0; }; }; }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Object.Delete"; ARRAY "{sv}" { }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Dump.Entry"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "Size"; VARIANT "t" { UINT64 0; }; }; DICT_ENTRY "sv" { STRING "Offloaded"; VARIANT "b" { BOOLEAN false; }; }; DICT_ENTRY "sv" { STRING "OffloadUri"; VARIANT "s" { STRING ""; }; }; }; }; DICT_ENTRY "sa{sv}" { STRING "xyz.openbmc_project.Common.Progress"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "Status"; VARIANT "s" { STRING "xyz.openbmc_project.Common.Progress.OperationStatus.InProgress"; }; }; DICT_ENTRY "sv" { STRING "StartTime"; VARIANT "t" { UINT64 1612972013; }; }; DICT_ENTRY "sv" { STRING "CompletedTime"; VARIANT "t" { UINT64 0; }; }; }; }; DICT_ENTRY "sa{sv}" { STRING "com.ibm.Dump.Entry.Resource"; ARRAY "{sv}" { DICT_ENTRY "sv" { STRING "SourceDumpId"; VARIANT "u" { UINT32 4294967295; }; }; DICT_ENTRY "sv" { STRING "VSPString"; VARIANT "s" { STRING "vsp"; }; }; DICT_ENTRY "sv" { STRING "Password"; VARIANT "s" { STRING "pass0"; }; }; }; }; }; }; Signed-off-by:
-
- Feb 02, 2021
-
-
Chirag Sharma authored
Changes: Renaming the ubifs-workaround flag to jffs-workaround. Default behaviour is IN_CREATE, if jffs-workaround is enabled will monitor for IN_CLOSE_WRITE. Test: Tested the fix on romulus i.e. jffs2, ext4 and ubifs. The core file is getting transferred to dump folder. Also did full build for jffs and ext4 fs. Change-Id: I964f5923ad73926db214526aa6c62fca9ac90ba8 Signed-off-by:
-
- Jan 19, 2021
-
-
Andrew Geissler authored
The MPIPL function was moved from the diagnostic target to the crash target. The diagnostic target is now a part of the crash target so it will still be called as a part of this process. The following commit has more info on why this change was made: https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/39711 Signed-off-by:
Andrew Geissler <geissonator@yahoo.com> Change-Id: Ie1598a06f5d14edd6edde1e95d63becbd1fdba96
-
Ramesh Iyyar authored
Added missing header file inclusion to fix compilation error when using "host-transport" option alone to get default pldm behaviour to do required host dump action. Error 1: 'uint32_t' was not declared in this scope Fixed by using `#include <stdint.h>` Error 2: 'runtime_error' is not a member of 'std' Fixed by using `#include <stdexcept>` TestedBy: - meson builddir -Dhost-transport=pldm ninja -C builddir Signed-off-by:Ramesh Iyyar <rameshi1@in.ibm.com> Change-Id: I136defcf5f45a6f15be41e11e792091e103ce2c3
-
- Jan 18, 2021
-
-
Dhruvaraj Subhashchandran authored
Change the namespace of OpenPOWER dumps to openpower instead of phosphor. Signed-off-by: -
Dhruvaraj Subhashchandran authored
To build use option -Dhost-transport=pldm and -Dopenpower-dumps-extension=enabled Add support for resource dump operations like creation and offloading. Tests:(Dump manager tests) Created dump entry. Faked notify to check whether progress is turning to completed. Signed-off-by: -
Dhruvaraj Subhashchandran authored
When the host is not up the system dump cannot be initiated or offloaded. Adding a check whether the host is up before starting the creation of the host dump. Added a new utility to check whether host is running, this can be called to check whether host is running before initiating actions like initiate, offload, delete on host dumps. Automatic clearing off host dump entries during a system power off will be handled with #ibm-openbmc/issues/2860 Signed-off-by:
-
- Jan 12, 2021
-
-
Dhruvaraj Subhashchandran authored
Each type of dump is hosted in its own service so DumpType is no more needed to differentiate between dumps in Notify. So removing from implementation to match the interface. Changes are tested with creation of a system dump. Signed-off-by:
-
- Dec 18, 2020
-
-
Dhruvaraj Subhashchandran authored
Updating create dump implementation to match the change in the interface to accept the additional parameters. Testing: Created BMC dump Created system dump Dbus changes associated with this: https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-dbus-interfaces/+/37355 Signed-off-by:
-
- Dec 16, 2020
-
-
Ramesh Iyyar authored
To delete host system dump using pldm oem command "FileAck" with dump file type and dump id which is present as source dump id (which is got from pldm after created the host system dump) in system dump entry dbus object and also, to use pldm oem command added support to indicate delete host system dump support is not available for other host transport consumer and for other oem vendors who are using pldm as host transport to get host dump. Also, changed existing commited error i.e internal failure into not allowed for host system dump offload to avoid bmc dump creation in dump path. Signed-off-by: -
Ramesh Iyyar authored
Used "CustomFd" class instead closeFD() to avoid call in requried place for close fd. Signed-off-by: -
Ramesh Iyyar authored
Currently used host mctp end point id is ibm specific and also the used effecter id (0x05) is for phyp and "SetNumericEffecterValue" pldm command is common so, changing existing pldm offload implementation into oem based and added default pldm offload interface as well. Signed-off-by: -
Ramesh Iyyar authored
Currently dump manager using "host-dump-offload-transport" to mention the transport protocol name to offload host dumps but, same transport can use to other actions i.e create and delete. So, Changing build option name into "host-transport" to make generic. Also, transport code sources files are organized based on "offload" name so, changed same as "host-transport". Signed-off-by: -
Dhruvaraj Subhashchandran authored
A new attribute status is added which indicates the status of the dump creation. The user requested dump entries will be created with InProgress and updated to Completed once the dump creation is completed Test: Request user initiated BMC and system dump and make sure the progress is changing and the dump details are updated correctly Signed-off-by: -
Dhruvaraj Subhashchandran authored
Return the object path to a dump entry for the user requested dump. A dump entry will be created when the user request for the dump and that will be used for tracking the progress. The dump details like size etc will be empty and the complete details will be filled once the dump creation is completed. Executed current BMC and system dump test cases on a test build - Create BMC dump - List All dumps - Offload BMC dump - Delete BMC dump - Create manual system dump. - Attempt to offload dump. Signed-off-by:
-
- Dec 14, 2020
-
-
Ravi Teja authored
This commit enables bmc dump offload through unix socket On dump offload request, dump manager setup unix socket and writes data on unix socket, on other end bmcweb connects to this socket and get this data offloaded to client. Tested By: Ran BMC Dump offload and verified content Signed-off-by:Ravi Teja <raviteja28031990@gmail.com> Change-Id: I9ce558778a73cf33b3f65a002490d3ad01fe75f3
-
- Oct 06, 2020
-
-
Dhruvaraj Subhashchandran authored
Make all OpenPower dumps as optional feature and should be enabled in the configuration. System dump is the currently implemented example of OpenPower dump. BMC dump will be enabled by default. To enable openpower-dumps-extension=enabled Test: Tested all existing dump operations using automated testing. Signed-off-by:
-
- Sep 29, 2020
-
-
Chirag Sharma authored
-Adding dump header script: -Changes are related to IBM specific systems only, no impact on the existing dump package format without dump header. In package function only for IBM systems the generated header will be added to the dump, for remaining systems the existing dump format remains same. -Adding dump header which will be added on top of the the <obmc file>.tar.xz, only on IBM systems Note: If user wants to extract the dump, they need to remove the header from the dump using dd command. Example dd bs=628 skip=1 if=<input_dump_file> of=<output_file> Result: Tested the dump header using busctl command and dreport script. Able to add dump header on top of the <obmcdump>.tar.xz file. Separated out the header from the output file using dd command: dd bs=628 skip=1 if=<obmc dump>.tar.xz of=openbmc.tar.xz The new output file doesn't have the header now and can be unzipped. Post unzipping the files in the dump can be seen. hexdump ooutput of the dump header:- 00000000 46 49 4c 45 20 20 20 20 00 40 00 00 00 00 00 00 |FILE .@......| 00000010 00 00 00 00 00 01 00 0f 42 4d 50 44 55 4d 50 2e |........BMPDUMP.| 00000020 53 49 4d 50 31 30 52 2e 30 30 30 30 30 30 30 36 |SIMP10R.00000006| 00000030 2e 32 30 32 30 30 39 32 39 31 31 31 30 34 33 00 |.20200929111043.| 00000040 53 45 43 54 49 4f 4e 20 00 30 00 00 00 00 00 00 |SECTION .0......| 00000050 00 00 00 01 00 02 00 00 00 00 00 00 00 02 ad dc |................| 00000060 42 4d 43 44 55 4d 50 00 00 00 00 00 00 00 00 00 |BMCDUMP.........| 00000070 42 4d 43 20 44 55 4d 50 00 00 00 00 00 00 00 00 |BMC DUMP........| 00000080 00 00 00 00 02 10 02 00 00 00 00 00 00 02 ad dc |................| 00000090 39 31 30 35 2d 32 32 41 00 00 00 00 00 00 00 00 |9105-22A........| 000000a0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| 000000b0 53 65 72 76 65 72 2d 39 31 30 35 2d 32 32 41 2d |Server-9105-22A-| 000000c0 53 4e 2d 53 49 4d 50 31 30 52 00 00 00 00 00 00 |SN-SIMP10R......| 000000d0 00 53 49 4d 50 31 30 52 00 00 00 00 00 70 00 00 |.SIMP10R.....p..| 000000e0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * 00000270 01 01 00 00 fd 37 7a 58 5a 00 00 04 e6 d6 b4 46 |.....7zXZ......F| Signed-off-by:
-
- Sep 25, 2020
-
-
Dhruvaraj Subhashchandran authored
Add method to set the diag mode target to start the creation of user requested system dump. Signed-off-by: -
Dhruvaraj Subhashchandran authored
Currently all types of dumps exist in the same path and under the single dump manager. When there are multiple dumps to be created separate path is needed for creating and managing the dump. this commit is splitting the dump manager into multiple objects without adding any new functionality. There will be only one dump manager process but it will contain seperate dump manager objects for system and BMC dumps as per current scope. Tested the existing dump functions with the build - created bmc dump - created system dump using notify - deleted dump entry - offloaded bmc dump Signed-off-by: -
Dhruvaraj Subhashchandran authored
Signed-off-by:
-
- Sep 11, 2020
-
-
Chirag Sharma authored
Change: -Serialize method does not use any of the watch functionalities, so defining set in serialization header as using watch header creates unnecessary dependency. Test: Added 3 test cases related to dump serialization. Ran the UTs on docker environment. Result: 1/1 debug_inif_test OK 1.48s Ok: 1 Expected Fail: 0 Fail: 0 Unexpected Pass: 0 Skipped: 0 Timeout: 0 Signed-off-by:
-
- Aug 21, 2020
-
-
Ramesh Iyyar authored
Removing autotools build infrastructure because, phosphor-debug-collector will support meson build tool so, maintaining two build tool is unnecessary so, removed. Signed-off-by:
-
- Aug 13, 2020
-
-
Ramesh Iyyar authored
In this commit enabled meson build infrastructure which is taking less time than autotools to build. Changes: - Same default value used for all build time configure option. - All external library checks are added into meson as well. - In meson, no need to mention list of files which are not required to install because, by default nothing will be installed. - Auto generated files are added into custom target and that will trigger when some target dependent with that. - In meson, enabled to treat warning as error so, modified few sources which are producing un-used variables. - Fixes made by removing those function parameters identifier alone. By using meson, can able to see below built improvement time between meson and autotools. meson: - time sh -c 'meson builddir -Dhost-dump-offload-transport=pldm -Dubifs-workaround=enabled; ninja -C builddir/' real 0m12.244s user 0m57.575s sys 0m7.793s autotools: - time sh -c 'autoreconf -i; ./configure ${CONFIGURE_FLAGS} --enable-ubifs-workaround --with-host-dump-offload-transport=pldm; make' real 1m16.539s user 1m2.738s sys 0m9.645s TestedBy: - meson builddir ninja -C builddir - meson builddir -Dubifs-workaround=enabled ninja -C builddir - meson builddir -Dubifs-workaround=disabled ninja -C builddir - meson builddir -Dhost-dump-offload-transport=pldm ninja -C builddir - meson builddir -Dhost-dump-offload-transport=pldm \ -Dubifs-workaround=enabled ninja -C builddir Note: Need to update openbmc phosphor-debug-collector recipe to use meson instead autotools, this will be updated once this patch got merged and also autotools build infrastructure will be removed as well. Change-Id: Iadf2d3542dc2556377e7b2f91f01b04d5f8d7218 Signed-off-by:
-
- Jul 24, 2020
-
-
Artem Senichev authored
The latest version of the Hostlogger supports multi-host mode, each host is served by a separate service instance. The service instance now uses SIG_USR1 signal to flush collected messages to a file. Tested: Run dreport inside the BMC console, the result dump archive contains host console logs. Change-Id: I2ba926dad79b362d0f4abac50619d7969a4f28f7 Signed-off-by:
Artem Senichev <a.senichev@yadro.com> Tested-by:
-
- Jul 15, 2020
-
-
Chirag Sharma authored
In the dreport script there is a condition which checks for '/tmp' only. When the user passes directory as '/tmp/' this check fails. As a result the overall operation fails. This commit fixes this by treating '/tmp/' as a valid directory. Signed-off-by:
-
- Jun 30, 2020
-
-
Dhruvaraj Subhashchandran authored
Add the support for reading the dump and writing to the path provided. The supported for writing the BMC dump is added. Tested using offloading a BMC dump to a remote system using NBD. Change-Id: I5250b41ed162905548a7d522b9dc86533d0a9a83 Signed-off-by:
-
- Jun 26, 2020
-
-
Alexander Filippov authored
Unblocking the `SIGCHLD` signal before the child process completes prevents the `callback()` from being ever called. This commit makes the blocking `SIGCHLD` once for the whole main process life by moving the corresponding code block to the main(). Tested: `callback()` now called when the child process completes. Change-Id: I553c683e5dfcbc0b33c72aa2d7b394f689772966 Signed-off-by:Alexander Filippov <a.filippov@yadro.com>
-
