9035 2016-02-02 13:42:08 +0000 bitbake-prserv not exiting on SIGTERM when run in daemon mode 2016-02-11 20:59:00 +0000 1 1 7 Build System, Metadata & Runtime BitBake bitbake 2.0.2 x86 Multiple RESOLVED FIXED Medium normal 2.1 M3 1 Diego.SantaCruz richard.purdie poky.bs.watcher poky.watcher --- --- No (bug/feature does not impact docs) oldest_to_newest 58445 0 2963 Diego.SantaCruz 2016-02-02 13:42:08 +0000 Created attachment 2963 Fix for SIGTERM handling in daemon mode I am running bitbake-prserv in daemon mode and noticed that it does not exit when a SIGTERM is sent to it. Using "bitbake-prserv --stop" works fine, but sending a SIGTERM to the pid of the daemonized bitbake-prserv hungs it and then even a "bitbake-prserv --stop" is not able to stop it. The problem seems to be that the handler thread is not signaled to quit, so it stays there doing its work, as it is not a daemon thread (i.e. the python thread is created in non-daemon mode). The attached patch fixes this by setting the quit variable instead. The patch also does not raise the SystemExit exception any longer but instead leaves the quit flag to do its job and do a clean exit. This way the PID file is properly removed. 58923 1 richard.purdie 2016-02-11 20:59:00 +0000 Thanks for the patches! http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=abf8a8fbc35c8d45c2357701400477792e39f80e 2963 2016-02-02 13:42:08 +0000 2016-02-02 13:42:08 +0000 Fix for SIGTERM handling in daemon mode 0001-bitbake-prserv-SIGTERM-handling-hung-process.patch text/plain 1289 Diego.SantaCruz RnJvbSAxOWZhNTJkNTFkYjM4MGRjNmZkNDZhMDJjZjQ0N2U2N2MwMjkxN2Y4IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBEaWVnbyBTYW50YSBDcnV6IDxEaWVnby5TYW50YUNydXpAc3Bp bmV0aXguY29tPgpEYXRlOiBUdWUsIDIgRmViIDIwMTYgMTM6MDU6NTAgKzAxMDAKU3ViamVjdDog W1BBVENIXSBiaXRiYWtlOiBwcnNlcnY6IFNJR1RFUk0gaGFuZGxpbmcgaHVuZyBwcm9jZXNzCgpU aGUgY3VycmVudCBTSUdURVJNIGhhbmRsZXIgaHVuZ3MgdGhlIHByb2Nlc3MgaW5zdGVhZCBvZgpt YWtpbmcgaXQgZXhpdC4gVGhlIHByb2JsZW0gc2VlbXMgdG8gYmUgdGhhdCB0aGUgaGFuZGxlciB0 aHJlYWQKaXMgbm90IHNpZ25hbGVkIHRvIHF1aXQsIHNvIGl0IHN0YXlzIHRoZXJlIGRvaW5nIGl0 cyB3b3JrLCBhcwppdCBpcyBub3QgYSBkYWVtb24gdGhyZWFkLiBTZXR0aW5nIHRoZSBxdWl0IHZh cmlhYmxlIGZpeGVzIHRoaXMuCgpXaGlsZSBhdCBpdCwgdG8gbm90IHVzZSB0aGUgU3lzdGVtRXhp dCBleGNlcHRpb24gdG8gdGVybWluYXRlCnVwb24gU0lHVEVSTSBidXQgaW5zdGVhZCBsZWZ0IHRo ZSBxdWl0IGZsYWcgZG8gaXRzIGpvYi4gVGhpcyB3YXkKdGhlIFBJRCBmaWxlIGlzIHByb3Blcmx5 IHJlbW92ZWQuCgpTaWduZWQtb2ZmLWJ5OiBEaWVnbyBTYW50YSBDcnV6IDxEaWVnby5TYW50YUNy dXpAc3BpbmV0aXguY29tPgotLS0KIGJpdGJha2UvbGliL3Byc2Vydi9zZXJ2LnB5IHwgMiArLQog MSBmaWxlIGNoYW5nZWQsIDEgaW5zZXJ0aW9uKCspLCAxIGRlbGV0aW9uKC0pCgpkaWZmIC0tZ2l0 IGEvYml0YmFrZS9saWIvcHJzZXJ2L3NlcnYucHkgYi9iaXRiYWtlL2xpYi9wcnNlcnYvc2Vydi5w eQppbmRleCBjNjVkZDNjLi4xZGUzN2NmIDEwMDY0NAotLS0gYS9iaXRiYWtlL2xpYi9wcnNlcnYv c2Vydi5weQorKysgYi9iaXRiYWtlL2xpYi9wcnNlcnYvc2Vydi5weQpAQCAtMTAxLDcgKzEwMSw3 IEBAIGNsYXNzIFBSU2VydmVyKFNpbXBsZVhNTFJQQ1NlcnZlcik6CiAgICAgZGVmIHNpZ3Rlcm1f aGFuZGxlcihzZWxmLCBzaWdudW0sIHN0YWNrKToKICAgICAgICAgaWYgc2VsZi50YWJsZToKICAg ICAgICAgICAgIHNlbGYudGFibGUuc3luYygpCi0gICAgICAgIHJhaXNlIFN5c3RlbUV4aXQKKyAg ICAgICAgc2VsZi5xdWl0PVRydWUKIAogICAgIGRlZiBwcm9jZXNzX3JlcXVlc3Qoc2VsZiwgcmVx dWVzdCwgY2xpZW50X2FkZHJlc3MpOgogICAgICAgICBzZWxmLnJlcXVlc3RxdWV1ZS5wdXQoKHJl cXVlc3QsIGNsaWVudF9hZGRyZXNzKSkKLS0gCjEuOS40Cgo=