diff --git a/interfaces/kits/updaterkits/updaterkits.cpp b/interfaces/kits/updaterkits/updaterkits.cpp index fcb68064bef9885321a9ac1786a27989268479f1..81f0085e879a3c037c5796c6df60ab39b004b9cd 100755 --- a/interfaces/kits/updaterkits/updaterkits.cpp +++ b/interfaces/kits/updaterkits/updaterkits.cpp @@ -39,7 +39,7 @@ static bool WriteToMiscAndRebootToUpdater(const struct UpdateMessage &updateMsg) } #ifndef UPDATER_UT WriteUpdaterMiscMsg(updateMsg); - DoReboot("updater"); + DoReboot("updater:reboot to updater to trigger update"); while (true) { pause(); } @@ -114,7 +114,7 @@ static bool WriteToMiscAndResultFileRebootToUpdater(const struct UpdateMessage & // Flag after the misc in written std::string writeMiscAfter = "0x80000008"; WriteUpdaterResultFile(pkgPath, writeMiscAfter); - DoReboot("updater"); + DoReboot("updater:reboot to updater to trigger update"); while (true) { pause(); } diff --git a/services/flashd/daemon/system_depend.cpp b/services/flashd/daemon/system_depend.cpp index c23550978c76d910b95d521e5e5d2b4129eb4c01..c8d702ec1b2d296530235a9cfffc66ef29388123 100644 --- a/services/flashd/daemon/system_depend.cpp +++ b/services/flashd/daemon/system_depend.cpp @@ -54,7 +54,7 @@ namespace SystemDepend { { FLASHD_LOGE("Flashd RebootDevice cmd is %s", cmd.c_str()); Updater::PostUpdater(true); - Updater::Utils::UpdaterDoReboot(""); + Updater::Utils::UpdaterDoReboot("", "Updater flashd reboot command"); return true; } } diff --git a/services/hwfault_retry/hwfault_retry.cpp b/services/hwfault_retry/hwfault_retry.cpp index 01325c3aa850ceabc89b809a1b88404038b3b236..433af9c55a181b4c2a24643010f1366d3b6457eb 100644 --- a/services/hwfault_retry/hwfault_retry.cpp +++ b/services/hwfault_retry/hwfault_retry.cpp @@ -79,7 +79,7 @@ void HwFaultRetry::RebootRetry() PostUpdater(false); sync(); #ifndef UPDATER_UT - DoReboot("updater"); + DoReboot("updater:Updater fault retry"); while (true) { pause(); } diff --git a/services/updater_binary/update_partitions.cpp b/services/updater_binary/update_partitions.cpp index 09f8bf1a9207561490942b288c692602b6a61fd2..0eff03f32c424c8fc116e3517ca62d65c78abc77 100644 --- a/services/updater_binary/update_partitions.cpp +++ b/services/updater_binary/update_partitions.cpp @@ -140,7 +140,7 @@ int UpdatePartitions::DoNewPartitions(PartitonList &newPartList) } else if (ret > 1) { LOG(INFO) << "do_partitions success reboot"; #ifndef UPDATER_UT - Utils::UpdaterDoReboot("updater"); + Utils::UpdaterDoReboot("updater", "Updater finish do new partitions"); #endif } return ret; diff --git a/services/updater_main.cpp b/services/updater_main.cpp index 8de2eba9513a1d531d284d011c04ece50115f1b5..4e5bfc460b367dd1448054403fd9071808f1371c 100644 --- a/services/updater_main.cpp +++ b/services/updater_main.cpp @@ -800,7 +800,7 @@ void RebootAfterUpdateSuccess(const UpdaterParams &upParams) if (IsNeedWipe() || upParams.sdExtMode == SDCARD_UPDATE_FROM_DEV || upParams.sdExtMode == SDCARD_UPDATE_FROM_DATA) { - Utils::UpdaterDoReboot("updater", "--user_wipe_data"); + Utils::UpdaterDoReboot("updater", "Updater wipe data after upgrade success", "--user_wipe_data"); return; } if (upParams.factoryResetMode == "factory_wipe_data") { @@ -809,7 +809,8 @@ void RebootAfterUpdateSuccess(const UpdaterParams &upParams) Utils::UsSleep(120 * DISPLAY_TIME); // 120 : 120s } upParams.forceUpdate || upParams.factoryResetMode == "factory_wipe_data" ? - Utils::DoShutdown() : Utils::UpdaterDoReboot(""); + Utils::DoShutdown("Updater update success go shut down") : + Utils::UpdaterDoReboot("", "Updater update success"); } int UpdaterMain(int argc, char **argv) @@ -836,7 +837,7 @@ int UpdaterMain(int argc, char **argv) if (upParams.forceReboot) { Utils::UsSleep(5 * DISPLAY_TIME); // 5 : 5s PostUpdater(true); - Utils::UpdaterDoReboot(""); + Utils::UpdaterDoReboot("", "Updater night update fail"); return 0; } } else if (mode == SDCARD_UPDATE) { diff --git a/services/updater_ui.cpp b/services/updater_ui.cpp index 90162af42f87b5c1a75e4715beacd5a8d30f3862..0b8f06f7dd09e9da29d6cc2ff7904d8937b3ce7a 100644 --- a/services/updater_ui.cpp +++ b/services/updater_ui.cpp @@ -68,7 +68,7 @@ DEFINE_ASYN_CALLBACK(OnRebootEvt) LOG(INFO) << "On Label Reboot"; GraphicEngine::GetInstance().StopEngine(); PostUpdater(false); - Utils::UpdaterDoReboot(""); + Utils::UpdaterDoReboot("", "Updater reboot btn event"); } DEFINE_SYNC_CALLBACK(OnLabelResetEvt) @@ -98,7 +98,7 @@ DEFINE_ASYN_CALLBACK(OnLabelSDCardEvt) return; } PostUpdater(true); - Utils::UpdaterDoReboot(""); + Utils::UpdaterDoReboot("", "Updater sdcard update success reboot"); } DEFINE_ASYN_CALLBACK(OnLabelSDCardNoDelayEvt) @@ -121,7 +121,7 @@ DEFINE_ASYN_CALLBACK(OnLabelSDCardNoDelayEvt) GetFacade().ShowSuccessPage(); Utils::UsSleep(SUCCESS_DELAY); PostUpdater(true); - Utils::UpdaterDoReboot(""); + Utils::UpdaterDoReboot("", "Updater sdcard update success reboot"); } DEFINE_ASYN_CALLBACK(OnLabelSDUpdateResEvt) @@ -151,7 +151,7 @@ DEFINE_ASYN_CALLBACK(OnLabelSDUpdateResEvt) GetFacade().ShowSuccessPage(); Utils::UsSleep(SUCCESS_DELAY); PostUpdater(true); - Utils::UpdaterDoReboot(""); + Utils::UpdaterDoReboot("", "Updater sdcard update success reboot"); } DEFINE_SYNC_CALLBACK(OnLabelCancelEvt) @@ -206,7 +206,7 @@ DEFINE_ASYN_CALLBACK(OnConfirmRstEvt) GetFacade().ShowSuccessPage(); PostUpdater(true); Utils::UsSleep(SUCCESS_DELAY); - Utils::UpdaterDoReboot(""); + Utils::UpdaterDoReboot("", "Updater factory reset success"); } } @@ -214,7 +214,7 @@ DEFINE_ASYN_CALLBACK(OnMenuShutdownEvt) { LOG(DEBUG) << "shutdown"; GraphicEngine::GetInstance().StopEngine(); - Utils::DoShutdown(); + Utils::DoShutdown("Updater shutdown btn event"); } DEFINE_ASYN_CALLBACK(OnMenuClearCacheEvt) diff --git a/utils/include/utils.h b/utils/include/utils.h index 961948203fa2663d1b9c12281c7fe90ba324b4d8..f97f8eda45c08577f923ab176a19e2a428a780bd 100644 --- a/utils/include/utils.h +++ b/utils/include/utils.h @@ -64,8 +64,9 @@ int32_t DeleteFile(const std::string& filename); std::vector SplitString(const std::string &str, const std::string del = " \t"); std::string Trim(const std::string &str); std::string ConvertSha256Hex(const uint8_t* shaDigest, size_t length); -void UpdaterDoReboot(const std::string& rebootTarget, const std::string& extData = ""); -void DoShutdown(); +void UpdaterDoReboot(const std::string& rebootTarget, const std::string &rebootReason, + const std::string& extData = ""); +void DoShutdown(const std::string &shutdownReason); std::string GetCertName(); bool WriteFully(int fd, const uint8_t *data, size_t size); bool ReadFully(int fd, void* data, size_t size); diff --git a/utils/updater_reboot.cpp b/utils/updater_reboot.cpp index 16311183c833c3ac9c43ec7dec1ba7d0e164e468..026ada98e2e5fd1c39b7570bc3ab8425d82342b9 100644 --- a/utils/updater_reboot.cpp +++ b/utils/updater_reboot.cpp @@ -27,15 +27,16 @@ int main(int argc, char **argv) } if (argc == 1) { Updater::LOG(Updater::INFO) << "Updater::Utils::UpdaterDoReboot nullptr"; - Updater::Utils::UpdaterDoReboot(""); + Updater::Utils::UpdaterDoReboot("", "Updater_reboot default reboot"); } else { std::string updaterStr = "updater"; if (!memcmp(argv[1], updaterStr.c_str(), updaterStr.length())) { - Updater::Utils::UpdaterDoReboot(argv[1]); + Updater::Utils::UpdaterDoReboot(argv[1], "Updater_reboot command reboot to updater"); } else if (strcmp(argv[1], "flashd") == 0) { - Updater::Utils::UpdaterDoReboot(argv[1]); + Updater::Utils::UpdaterDoReboot(argv[1], "Updater_reboot command reboot to flashd"); } else if (strncmp(argv[1], "flashd:", strlen("flashd:")) == 0) { - Updater::Utils::UpdaterDoReboot("flashd", argv[1] + strlen("flashd:")); + Updater::Utils::UpdaterDoReboot("flashd", "Updater_reboot command reboot to flashd", + argv[1] + strlen("flashd:")); } else { Updater::LOG(Updater::INFO) << "param must be updater/flashd!"; } diff --git a/utils/utils.cpp b/utils/utils.cpp index 7337f1e9b66b0c721989a6f319d4f631673bb47e..eb145b1e827dbe5233badd0b0d58038697f57c92 100644 --- a/utils/utils.cpp +++ b/utils/utils.cpp @@ -224,9 +224,10 @@ bool SetRebootMisc(const std::string& rebootTarget, const std::string &extData, return true; } -void UpdaterDoReboot(const std::string& rebootTarget, const std::string &extData) +void UpdaterDoReboot(const std::string& rebootTarget, const std::string &rebootReason, const std::string &extData) { LOG(INFO) << ", rebootTarget: " << rebootTarget; + LOG(INFO) << ", rebootReason: " << rebootReason; LoadFstab(); struct UpdateMessage msg = {}; if (rebootTarget.empty()) { @@ -246,7 +247,7 @@ void UpdaterDoReboot(const std::string& rebootTarget, const std::string &extData } sync(); #ifndef UPDATER_UT - DoReboot(rebootTarget.c_str()); + DoRebootExt(rebootTarget.c_str(), rebootReason.c_str()); while (true) { pause(); } @@ -255,7 +256,7 @@ void UpdaterDoReboot(const std::string& rebootTarget, const std::string &extData #endif } -void DoShutdown() +void DoShutdown(const std::string &shutdownReason) { UpdateMessage msg = {}; if (!WriteUpdaterMiscMsg(msg)) { @@ -263,7 +264,7 @@ void DoShutdown() return; } sync(); - DoReboot("shutdown"); + DoRebootExt("shutdown", shutdownReason.c_str()); } std::string GetCertName()