12565 2018-02-22 00:03:02 +0000 watchdog.service dead on startup with systemd 2018-04-17 08:18:46 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 5.99 Other Multiple RESOLVED FIXED Medium+ normal 2.5 M4 1 scott.branden maxin.john alex.kanavin bkylerussell meta.mr.watcher meta.watcher stephano --- --- Don't know oldest_to_newest 79536 0 scott.branden 2018-02-22 00:03:02 +0000 The watchdog.service appears inactive on startup even though it is enabled. It appears the [Install] directive is not correct for a yocto build. If I change: [Install] WantedBy=default.target to WantedBy=multi-user.target Then the watchdog service is properly activated. When running "systemctl stop watchdog" there are some error messages printed related to wd_keepalive. It seems some clean attempt is sitting here. https://patchwork.openembedded.org/patch/144580/ But, I think that still doesn't solve the wd_keepalive issues. 79559 1 scott.branden 2018-02-22 15:57:01 +0000 Also, wd_keepalive is not handled properly. After change above there are errors in the stop mechanism. try running "systemctl stop watchdog" and you will see errors as the wd_keepalive script has not been installed by the watchdog recipe. 79794 2 4233 maxin.john 2018-03-13 06:49:25 +0000 Created attachment 4233 fix Removed the inter-dependencies of watchdog and wd_keepalive services. 79795 3 maxin.john 2018-03-13 06:57:10 +0000 @Scott, can you please verify this fix and share your thoughts/comments ? 79805 4 scott.branden 2018-03-13 23:25:02 +0000 (In reply to comment #3) > @Scott, can you please verify this fix and share your thoughts/comments ? I will have to rollback my local changes and try it on my system in the next few days. 79859 5 scott.branden 2018-03-19 14:16:37 +0000 Hi Maxin, I looked at the changes. I don't think the claim that watchdog and watchdog-keepalive packages can't be installed together is what the original designers of these watchdog utilities intended. I "think" what is supposed to happen is both services are supposed to be installed in the rootfs. Then they are controlled by run_watchdog and wd_keepalive in the configuration file. Possibly, as well, on shutdown, wd_keepalive is supposed to be started if watchdog is stopped? That way watchdog is still kicked throughout shutdown. In practical situations this probably isn't needed. Also, why do you indicate the Upstream-status as: Inappropriate [oe specific]. I would think that the watchdog.service simply isn't being configured properly and such changes need to be upstreamed. At least that's my impression? If none of above is true and only one of run_watchdog or run_wd_keep_alive can be set then I guess what you've changed may be correct. But, I would think upstream owner's should be able to comment further? 79872 6 maxin.john 2018-03-20 06:10:25 +0000 Hi Scott, (In reply to comment #5) > Hi Maxin, > > I looked at the changes. > > I don't think the claim that watchdog and watchdog-keepalive packages can't > be installed together is what the original designers of these watchdog > utilities intended. That claim was introduced by this commit: " commit e051c702503614b45d68f4c4c10ed413b2ea5c66 Author: Fabio Berton <fabio.berton@ossystems.com.br> Date: Wed Jan 25 09:22:05 2017 -0200 watchdog: Add RCONFLICTS to watchdog-keepalive and split RDEPENDS " IMHO, this splitting makes the watchdog handling less complex (avoids two daemons that handle watchdog at the same time, probably by wrong configuration) > I "think" what is supposed to happen is both services are supposed to be > installed in the rootfs. Then they are controlled by run_watchdog and > wd_keepalive in the configuration file. It looks good. However, that could cause potential troubles with wrong configuration. > Possibly, as well, on shutdown, wd_keepalive is supposed to be started if > watchdog is stopped? That way watchdog is still kicked throughout shutdown. > In practical situations this probably isn't needed. I am not sure about the practical use of that either. > Also, why do you indicate the Upstream-status as: Inappropriate [oe > specific]. I would think that the watchdog.service simply isn't being > configured properly and such changes need to be upstreamed. At least that's > my impression? Since we separated it into watchdog, wd_keepalive packages along with configuration package, changes that we introduced here are specific to oe. > If none of above is true and only one of run_watchdog or run_wd_keep_alive > can be set then I guess what you've changed may be correct. Ok. > But, I would think upstream owner's should be able to comment further? Agree. If the upstream maintainer feels it is wrong to go ahead with splitting the packages, we can revert those changes and go ahead with a better/safer fix. 79873 7 maxin.john 2018-03-20 06:17:41 +0000 Adding Alex in CC 79877 8 alex.kanavin 2018-03-20 11:36:39 +0000 I know even less about watchdog than you do :) Is there some specific question you'd like me to look into? 80236 9 maxin.john 2018-04-13 04:55:23 +0000 Merged in oe-core: https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=c3c336f106d579450ad09eedae830162a817214c Sent a query to Fabio about the reason behind splitting the package. Scott, if you think that we need to revert those changes and fix it, please feel free to reopen this bug or open another bug. 80240 10 scott.branden 2018-04-13 10:07:19 +0000 current patch works my use cases. But, I don't think it was the original intent for the option of specifying the wd_keepalive is started when the normal watchdog service is stop. The wd_keepalive is a simpler implementation that can keep kicking the watchdog while everything is shutting down. In practical situations (watchdog timeout long enough) this isn't an issue during linux shutdown/power off, though. 80246 11 maxin.john 2018-04-13 13:26:22 +0000 Hi Scott, (In reply to comment #10) > current patch works my use cases. But, I don't think it was the original > intent for the option of specifying the wd_keepalive is started when the > normal watchdog service is stop. The wd_keepalive is a simpler > implementation that can keep kicking the watchdog while everything is > shutting down. > > In practical situations (watchdog timeout long enough) this isn't an issue > during linux shutdown/power off, though. Fabio mentioned that he agree with your views and also promised to work on a fix. Thanks ! 80278 12 bkylerussell 2018-04-17 08:18:46 +0000 Is there an open issue tracking the work to resolve the RCONFLICT between watchdog and watchdog-keepalive? The man page for wd_keepalive indicates it's intended to be used on system startup before the full watchdog service can start (the implication being both packages should be installed), and our system configuration uses it as such. 4233 2018-03-13 06:49:25 +0000 2018-03-13 06:49:25 +0000 fix 0001-watchdog-remove-interdependencies-of-watchdog-and-wd.patch text/plain 5094 maxin.john RnJvbSAzMDc3ZTBiOWNiZDQ2ZjdkNzk5NjFkNjFhNzg1YmZmNGRlYjNiNzE2IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiAiTWF4aW4gQi4gSm9obiIgPG1heGluLmpvaG5AaW50ZWwuY29t PgpEYXRlOiBUdWUsIDEzIE1hciAyMDE4IDE1OjMyOjM0ICswMjAwClN1YmplY3Q6IFtQQVRDSF0g d2F0Y2hkb2c6IHJlbW92ZSBpbnRlcmRlcGVuZGVuY2llcyBvZiB3YXRjaGRvZyBhbmQKIHdkX2tl ZXBhbGl2ZQoKU2luY2Ugd2F0Y2hkb2cgYW5kIHdhdGNoZG9nLWtlZXBhbGl2ZSBwYWNrYWdlcyBj YW4ndCBiZSBpbnN0YWxsZWQKdG9nZXRoZXIsIG1vdmUgd2Rfa2VlcGFsaXZlLnNlcnZpY2UgdG8g d2F0Y2hkb2cta2VlcGFsaXZlIHBhY2thZ2UuCgpSZW1vdmUgdGhlIGludGVyLWRlcGVuZGVuY2ll cyBvZiB3YXRjaGRvZyBhbmQgd2Rfa2VlcGFsaXZlCnNlcnZpY2VzIGFzIHdlbGwuCgpbWU9DVE8g IzEyNTY1XQoKU2lnbmVkLW9mZi1ieTogTWF4aW4gQi4gSm9obiA8bWF4aW4uam9obkBpbnRlbC5j b20+Ci0tLQogLi4ubW92ZS1pbnRlcmRlcGVuZGVuY2llcy1vZi13YXRjaGRvZy1hbmQtd2QucGF0 Y2ggfCA2OCArKysrKysrKysrKysrKysrKysrKysrCiBtZXRhL3JlY2lwZXMtZXh0ZW5kZWQvd2F0 Y2hkb2cvd2F0Y2hkb2dfNS4xNS5iYiAgICB8ICA2ICstCiAyIGZpbGVzIGNoYW5nZWQsIDczIGlu c2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkKIGNyZWF0ZSBtb2RlIDEwMDY0NCBtZXRhL3JlY2lw ZXMtZXh0ZW5kZWQvd2F0Y2hkb2cvd2F0Y2hkb2cvMDAwMS13YXRjaGRvZy1yZW1vdmUtaW50ZXJk ZXBlbmRlbmNpZXMtb2Ytd2F0Y2hkb2ctYW5kLXdkLnBhdGNoCgpkaWZmIC0tZ2l0IGEvbWV0YS9y ZWNpcGVzLWV4dGVuZGVkL3dhdGNoZG9nL3dhdGNoZG9nLzAwMDEtd2F0Y2hkb2ctcmVtb3ZlLWlu dGVyZGVwZW5kZW5jaWVzLW9mLXdhdGNoZG9nLWFuZC13ZC5wYXRjaCBiL21ldGEvcmVjaXBlcy1l eHRlbmRlZC93YXRjaGRvZy93YXRjaGRvZy8wMDAxLXdhdGNoZG9nLXJlbW92ZS1pbnRlcmRlcGVu ZGVuY2llcy1vZi13YXRjaGRvZy1hbmQtd2QucGF0Y2gKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5k ZXggMDAwMDAwMC4uMzM4ZTBjZAotLS0gL2Rldi9udWxsCisrKyBiL21ldGEvcmVjaXBlcy1leHRl bmRlZC93YXRjaGRvZy93YXRjaGRvZy8wMDAxLXdhdGNoZG9nLXJlbW92ZS1pbnRlcmRlcGVuZGVu Y2llcy1vZi13YXRjaGRvZy1hbmQtd2QucGF0Y2gKQEAgLTAsMCArMSw2OCBAQAorRnJvbSBjMWZl MTRmYTViZDE2ODI5MmNjNDY3MDAzNGJjNDhiOTU0ZTlkYWM3IE1vbiBTZXAgMTcgMDA6MDA6MDAg MjAwMQorRnJvbTogIk1heGluIEIuIEpvaG4iIDxtYXhpbi5qb2huQGludGVsLmNvbT4KK0RhdGU6 IFR1ZSwgMTMgTWFyIDIwMTggMTQ6NDk6NTUgKzAyMDAKK1N1YmplY3Q6IFtQQVRDSF0gd2F0Y2hk b2c6IHJlbW92ZSBpbnRlcmRlcGVuZGVuY2llcyBvZiB3YXRjaGRvZyBhbmQKKyB3ZF9rZWVwYWxp dmUgc2VydmljZXMKKworU2luY2Ugd2F0Y2hkb2cgYW5kIHdhdGNoZG9nLWtlZXBhbGl2ZSBwYWNr YWdlcyBjYW4ndCBiZSBpbnN0YWxsZWQKK3RvZ2V0aGVyLCByZW1vdmUgdGhlIGludGVyLWRlcGVu ZGVuY2llcyBvZiB3YXRjaGRvZyBhbmQgd2Rfa2VlcGFsaXZlCitzZXJ2aWNlcworCitVcHN0cmVh bS1TdGF0dXM6IEluYXBwcm9wcmlhdGUgW29lIHNwZWNpZmljXQorCitTaWduZWQtb2ZmLWJ5OiBN YXhpbiBCLiBKb2huIDxtYXhpbi5qb2huQGludGVsLmNvbT4KKy0tLQorIGRlYmlhbi93YXRjaGRv Zy5zZXJ2aWNlICAgICB8ICA5ICsrKy0tLS0tLQorIGRlYmlhbi93ZF9rZWVwYWxpdmUuc2Vydmlj ZSB8IDEwICsrKysrLS0tLS0KKyAyIGZpbGVzIGNoYW5nZWQsIDggaW5zZXJ0aW9ucygrKSwgMTEg ZGVsZXRpb25zKC0pCisKK2RpZmYgLS1naXQgYS9kZWJpYW4vd2F0Y2hkb2cuc2VydmljZSBiL2Rl Ymlhbi93YXRjaGRvZy5zZXJ2aWNlCitpbmRleCA3YTJmYzM2Li5mMzFkMWZlIDEwMDY0NAorLS0t IGEvZGViaWFuL3dhdGNoZG9nLnNlcnZpY2UKKysrKyBiL2RlYmlhbi93YXRjaGRvZy5zZXJ2aWNl CitAQCAtMSwxNiArMSwxMyBAQAorIFtVbml0XQorIERlc2NyaXB0aW9uPXdhdGNoZG9nIGRhZW1v bgorLUNvbmZsaWN0cz13ZF9rZWVwYWxpdmUuc2VydmljZQorIEFmdGVyPW11bHRpLXVzZXIudGFy Z2V0CistT25GYWlsdXJlPXdkX2tlZXBhbGl2ZS5zZXJ2aWNlCisgCisgW1NlcnZpY2VdCisgVHlw ZT1mb3JraW5nCisgRW52aXJvbm1lbnRGaWxlPS9ldGMvZGVmYXVsdC93YXRjaGRvZworIEV4ZWNT dGFydFByZT0vYmluL3NoIC1jICdbIC16ICIke3dhdGNoZG9nX21vZHVsZX0iIF0gfHwgWyAiJHt3 YXRjaGRvZ19tb2R1bGV9IiA9ICJub25lIiBdIHx8IC9zYmluL21vZHByb2JlICR3YXRjaGRvZ19t b2R1bGUnCistRXhlY1N0YXJ0PS9iaW4vc2ggLWMgJ1sgJHJ1bl93YXRjaGRvZyAhPSAxIF0gfHwg ZXhlYyAvdXNyL3NiaW4vd2F0Y2hkb2cgJHdhdGNoZG9nX29wdGlvbnMnCistRXhlY1N0b3BQb3N0 PS9iaW4vc2ggLWMgJ1sgJHJ1bl93ZF9rZWVwYWxpdmUgIT0gMSBdIHx8IGZhbHNlJworK0V4ZWNT dGFydD0vYmluL3NoIC1jICdbIHgkcnVuX3dhdGNoZG9nICE9IHgxIF0gfHwgZXhlYyAvdXNyL3Ni aW4vd2F0Y2hkb2cgJHdhdGNoZG9nX29wdGlvbnMnCisrUElERmlsZT0vdmFyL3J1bi93YXRjaGRv Zy5waWQKKyAKKyBbSW5zdGFsbF0KKy1XYW50ZWRCeT1kZWZhdWx0LnRhcmdldAorLQorK1dhbnRl ZEJ5PW11bHRpLXVzZXIudGFyZ2V0CitkaWZmIC0tZ2l0IGEvZGViaWFuL3dkX2tlZXBhbGl2ZS5z ZXJ2aWNlIGIvZGViaWFuL3dkX2tlZXBhbGl2ZS5zZXJ2aWNlCitpbmRleCA0NWIwMThlLi43Zjhi MWRjIDEwMDY0NAorLS0tIGEvZGViaWFuL3dkX2tlZXBhbGl2ZS5zZXJ2aWNlCisrKysgYi9kZWJp YW4vd2Rfa2VlcGFsaXZlLnNlcnZpY2UKK0BAIC0xLDEzICsxLDEzIEBACisgW1VuaXRdCisgRGVz Y3JpcHRpb249d2F0Y2hkb2cga2VlcGFsaXZlIGRhZW1vbgorLUJlZm9yZT13YXRjaGRvZy5zZXJ2 aWNlIHNodXRkb3duLnRhcmdldAorLUNvbmZsaWN0cz13YXRjaGRvZy5zZXJ2aWNlIHNodXRkb3du LnRhcmdldAorK0FmdGVyPW11bHRpLXVzZXIudGFyZ2V0CisgCisgW1NlcnZpY2VdCisgVHlwZT1m b3JraW5nCisgRW52aXJvbm1lbnRGaWxlPS9ldGMvZGVmYXVsdC93YXRjaGRvZworIEV4ZWNTdGFy dFByZT0vYmluL3NoIC1jICdbIC16ICIke3dhdGNoZG9nX21vZHVsZX0iIF0gfHwgWyAiJHt3YXRj aGRvZ19tb2R1bGV9IiA9ICJub25lIiBdIHx8IC9zYmluL21vZHByb2JlICR3YXRjaGRvZ19tb2R1 bGUnCistRXhlY1N0YXJ0UHJlPS0vYmluL3N5c3RlbWN0bCByZXNldC1mYWlsZWQgd2F0Y2hkb2cu c2VydmljZQorIEV4ZWNTdGFydD0vdXNyL3NiaW4vd2Rfa2VlcGFsaXZlICR3YXRjaGRvZ19vcHRp b25zCistRXhlY1N0YXJ0UG9zdD0vYmluL3NoIC1jICdsbiAtcyAvdmFyL3J1bi93ZF9rZWVwYWxp dmUucGlkIC9ydW4vc2VuZHNpZ3Mub21pdC5kL3dkX2tlZXBhbGl2ZS5waWQnCistRXhlY1N0b3BQ b3N0PS9iaW4vc2ggLWMgJ3JtIC1mIC9ydW4vc2VuZHNpZ3Mub21pdC5kL3dkX2tlZXBhbGl2ZS5w aWQnCisrUElERmlsZT0vdmFyL3J1bi93ZF9rZWVwYWxpdmUucGlkCisrCisrW0luc3RhbGxdCisr V2FudGVkQnk9bXVsdGktdXNlci50YXJnZXQKKy0tIAorMi40LjAKKwpkaWZmIC0tZ2l0IGEvbWV0 YS9yZWNpcGVzLWV4dGVuZGVkL3dhdGNoZG9nL3dhdGNoZG9nXzUuMTUuYmIgYi9tZXRhL3JlY2lw ZXMtZXh0ZW5kZWQvd2F0Y2hkb2cvd2F0Y2hkb2dfNS4xNS5iYgppbmRleCAzZDBiNzJlLi4zN2Iz N2FlIDEwMDY0NAotLS0gYS9tZXRhL3JlY2lwZXMtZXh0ZW5kZWQvd2F0Y2hkb2cvd2F0Y2hkb2df NS4xNS5iYgorKysgYi9tZXRhL3JlY2lwZXMtZXh0ZW5kZWQvd2F0Y2hkb2cvd2F0Y2hkb2dfNS4x NS5iYgpAQCAtMTAsNiArMTAsNyBAQCBMSUNfRklMRVNfQ0hLU1VNID0gImZpbGU6Ly9DT1BZSU5H O21kNT1lY2MwNTUxYmY1NGFkOTdmNmI1NDE3MjBmODRkNjU2OSIKIAogU1JDX1VSSSA9ICIke1NP VVJDRUZPUkdFX01JUlJPUn0vd2F0Y2hkb2cvd2F0Y2hkb2ctJHtQVn0udGFyLmd6IFwKICAgICAg ICAgICAgZmlsZTovLzAwMDEtSW5jbHVkZS1saW51eC1wYXJhbS5oLWZvci1FWEVDX1BBR0VTSVpF LWRlZmluaXRpb24ucGF0Y2ggXAorICAgICAgICAgICBmaWxlOi8vMDAwMS13YXRjaGRvZy1yZW1v dmUtaW50ZXJkZXBlbmRlbmNpZXMtb2Ytd2F0Y2hkb2ctYW5kLXdkLnBhdGNoIFwKICAgICAgICAg ICAgZmlsZTovL3dhdGNoZG9nLWluaXQucGF0Y2ggXAogICAgICAgICAgICBmaWxlOi8vd2F0Y2hk b2ctY29uZi5wYXRjaCBcCiAgICAgICAgICAgIGZpbGU6Ly93ZF9rZWVwYWxpdmUuaW5pdCBcCkBA IC0zNiw3ICszNyw5IEBAIElOSVRTQ1JJUFRfUEFSQU1TXyR7UE59ID0gInN0YXJ0IDE1IDEgMiAz IDQgNSAuIHN0b3AgODUgMCA2IC4iCiBJTklUU0NSSVBUX05BTUVfJHtQTn0ta2VlcGFsaXZlID0g IndkX2tlZXBhbGl2ZSIKIElOSVRTQ1JJUFRfUEFSQU1TXyR7UE59LWtlZXBhbGl2ZSA9ICJzdGFy dCAxNSAxIDIgMyA0IDUgLiBzdG9wIDg1IDAgNiAuIgogCi1TWVNURU1EX1NFUlZJQ0VfJHtQTn0g PSAid2F0Y2hkb2cuc2VydmljZSB3ZF9rZWVwYWxpdmUuc2VydmljZSIKK1NZU1RFTURfUEFDS0FH RVMgPSAiJHtQTn0gJHtQTn0ta2VlcGFsaXZlIgorU1lTVEVNRF9TRVJWSUNFXyR7UE59ID0gIndh dGNoZG9nLnNlcnZpY2UiCitTWVNURU1EX1NFUlZJQ0VfJHtQTn0ta2VlcGFsaXZlID0gIndkX2tl ZXBhbGl2ZS5zZXJ2aWNlIgogCiBkb19pbnN0YWxsX2FwcGVuZCgpIHsKIAlpbnN0YWxsIC1kICR7 RH0ke3N5c3RlbWRfc3lzdGVtX3VuaXRkaXJ9CkBAIC01NCw2ICs1Nyw3IEBAIFBBQ0tBR0VTID0r ICIke1BOfS1rZWVwYWxpdmUiCiAKIEZJTEVTXyR7UE59LWtlZXBhbGl2ZSA9ICIgXAogICAgICR7 c3lzY29uZmRpcn0vaW5pdC5kL3dkX2tlZXBhbGl2ZSBcCisgICAgJHtzeXN0ZW1kX3N5c3RlbV91 bml0ZGlyfS93ZF9rZWVwYWxpdmUuc2VydmljZSBcCiAgICAgJHtzYmluZGlyfS93ZF9rZWVwYWxp dmUgXAogIgogCi0tIAoyLjQuMAoK