10095 2006-10-06 10:23:34 +0400 pppd виснет при получении SIGTERM во время выполнения chat, etc. 2010-06-12 12:00:24 +0400 1 1 4 Development Sisyphus ppp unstable all Linux CLOSED FIXED http://ppp.samba.org/cgi-bin/ppp-bugs/expired?id=1285 P2 normal --- 1 lvu mike mike qa-sisyphus oldest_to_newest 41175 0 lvu 2006-10-06 10:23:35 +0400 1. Если pppd получает SIGTERM во время выполнения chat скрипта, он входит в бесконенчный цикл, посылая сам себе этот сигнал. Причина - сигнал, отправленный в то время, когда самим pppd он игнорируется, доставляется уже после того, как pppd снова начинает его смотреть. Решено в ppp-2.4.2-alt-killmypg.patch, но мое решение мне нравится больше :) 2. Если SIGTERM отправлен до запуска chat'а, то замечает его pppd только после того, как chat завершится. LCP сессия после этого подвисает. Причина - хотя в ф-ции connect_tty() и проверяется переменная kill_link, она не может быть выставлена, т.к. не вызывается handle_events(). 2. Если вызвать pppd например на /dev/ttyS0, на котором в данный момент нет модема, то после отвала chat'а по таймауту pppd не завершается, убить его можно только SIGKILL'ом. После одного SIGTERM'а выводит что-то насчет tcsetattr: interrupted system call, на последующие не реагирует. Причина - вызов tcsetattr похоже виснет, если указан параметр TCSAFLUSH, а флашить некуда. Мое решение (замена TCSAFLUSH на TCSANOW) мне не нравится, но лучшего не придумал. Steps to Reproduce: *.1. Пересобрать pppd с моим патчем - бэкпортом фичи из версии 2.4.3 - создание linkpidfile при запуске pppd, а не после установки соединения. Тогда, используя netscripts, после ifup ppp0 можно сразу давить ^C, и pppd получит SIGTERM. *.2. Прописать все нужные настройки для ppp0 (в т.ч. DEBUG=yes) и включить модем. 1.3. ifup ppp0, подождать пару секунд и нажать ^C. 2.3. ifup ppp0, и очень быстро нажать ^C. 3.3. Выключить модем, ifup ppp0, дождаться обвала chat по таймауту (см. 12-ю коснсоль). После этого нажать ^C. Actual Results: 1. pppd не завершается, порт занят. 2. pppd не завершается, порт занят. 3. pppd не завершается и не убивается иначе как SIGKILL'ом, порт занят. Expected Results: 1. pppd завершился, порт свободен. 2. pppd завершился, порт свободен. 3. pppd завершился, порт свободен. Или хотя бы убился SIGTERM'ом. 41176 1 1656 lvu 2006-10-06 10:25:09 +0400 Created attachment 1656 Бэкпорт из pppd 2.4.3 - создание linkpidfile при старте pppd 41177 2 1657 lvu 2006-10-06 10:28:42 +0400 Created attachment 1657 Патч, который вроде как фиксит все указанные ошибки Патч не накатывается при накаченном ppp-2.4.2-alt-killmypg.patch, его надо исключить из спека. 41178 3 vvk 2006-10-06 11:54:28 +0400 Change Product to Sisyphus 41182 4 voins 2006-10-06 21:09:30 +0400 перевешиваю на текущего майнтейнера. 41234 5 mike 2006-10-10 21:31:04 +0400 2 sr: забираем? 42395 6 mike 2006-11-22 13:34:01 +0300 2 mithraen: смотрю, у тебя в git забрано? 47174 7 mithraen 2007-03-25 22:47:55 +0400 В 2.4.2 да, у меня это было забрано. С 2.4.4 будем разбираться. 59202 8 mike 2007-12-08 19:50:54 +0300 В 2.4.4-alt9 наблюдаю: Patch16: ppp-2.4.2-lvu-linkpidfile-backport.patch Patch17: ppp-2.4.4-lvu-sigterm.patch # ... # patch16 -p1 %patch17 -p1 Если 16 -- это бэкпорт из 2.4.3, то по идее уже неактуален, а второй приложен => закрываю как FIXED. 59204 9 mike 2007-12-08 19:51:41 +0300 *** Bug 8312 has been marked as a duplicate of this bug. *** 109838 10 mike 2010-06-12 12:00:24 +0400 Истории ради: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=100028 1656 2006-10-06 10:25:09 +0400 2006-10-06 10:25:09 +0400 Бэкпорт из pppd 2.4.3 - создание linkpidfile при старте pppd ppp-2.4.2-lvu-linkpidfile-backport.patch text/plain 320 lvu SW5kZXg6IHBwcC9wcHBkL21haW4uYwpkaWZmIC1jIHBwcC9wcHBkL21haW4uYzoxLjE0NSBwcHAv cHBwZC9tYWluLmM6MS4xNDYKKioqIHBwcC9wcHBkL21haW4uYzoxLjE0NQlGcmkgTm92IDEyIDAy OjMwOjUxIDIwMDQKLS0tIHBwcC9wcHBkL21haW4uYwlGcmkgTm92IDEyIDAzOjIxOjQxIDIwMDQK KioqKioqKioqKioqKioqCioqKiA0NzAsNDc1ICoqKioKLS0tIDQ3MCw0NzcgLS0tLQogIAogICAg ICBzZXR1cF9zaWduYWxzKCk7CiAgCisgICAgIGNyZWF0ZV9saW5rcGlkZmlsZShnZXRwaWQoKSk7 CisgCiAgICAgIHdhaXRpbmcgPSAwOwogIAogICAgICAvKgo= 1657 2006-10-06 10:28:42 +0400 2006-10-06 10:28:42 +0400 Патч, который вроде как фиксит все указанные ошибки ppp-2.4.2-lvu-sigterm.patch text/plain 3480 lvu ZGlmZiAtcnUgcHBwLTIuNC4yL3BwcGQvbWFpbi5jIHBwcC0yLjQuMi1uZXcvcHBwZC9tYWluLmMK LS0tIHBwcC0yLjQuMi9wcHBkL21haW4uYwkyMDA0LTAxLTEzIDA2OjAwOjM0ICswMjAwCisrKyBw cHAtMi40LjItbmV3L3BwcGQvbWFpbi5jCTIwMDYtMDktMjYgMTc6MDk6MTUgKzAzMDAKQEAgLTE3 Nyw2ICsxNzcsOCBAQAogCiBpbnQgZXJyb3JfY291bnQ7CiAKK3NpZ3NldF90IGlnbm9yZWRfc2ln bmFsczsgLyogdG8gYmUgaWdub3JlZCBkdWUgdG8ga2lsbF9teV9wZygpIGNhbGwgKi8KKwogLyoK ICAqIFdlIG1haW50YWluIGEgbGlzdCBvZiBjaGlsZCBwcm9jZXNzIHBpZHMgYW5kCiAgKiBmdW5j dGlvbnMgdG8gY2FsbCB3aGVuIHRoZXkgZXhpdC4KQEAgLTIxNyw3ICsyMTksNiBAQAogc3RhdGlj IHZvaWQgY2xlYW51cF9kYiBfX1AoKHZvaWQpKTsKICNlbmRpZgogCi1zdGF0aWMgdm9pZCBoYW5k bGVfZXZlbnRzIF9fUCgodm9pZCkpOwogc3RhdGljIHZvaWQgcHJpbnRfbGlua19zdGF0cyBfX1Ao KHZvaWQpKTsKIAogZXh0ZXJuCWNoYXIJKnR0eW5hbWUgX19QKChpbnQpKTsKQEAgLTY0NCw3ICs2 NDcsNyBAQAogLyoKICAqIGhhbmRsZV9ldmVudHMgLSB3YWl0IGZvciBzb21ldGhpbmcgdG8gaGFw cGVuIGFuZCByZXNwb25kIHRvIGl0LgogICovCi1zdGF0aWMgdm9pZAordm9pZAogaGFuZGxlX2V2 ZW50cygpCiB7CiAgICAgc3RydWN0IHRpbWV2YWwgdGltbzsKQEAgLTc2Nyw2ICs3NzAsOCBAQAog ICAgICAqIGJlIHN1ZmZpY2llbnQuCiAgICAgICovCiAgICAgc2lnbmFsKFNJR1BJUEUsIFNJR19J R04pOworCisgICAgc2lnZW1wdHlzZXQoJmlnbm9yZWRfc2lnbmFscyk7CiB9CiAKIC8qCkBAIC0x MzM0LDEzICsxMzM5LDE1IEBACiBraWxsX215X3BnKHNpZykKICAgICBpbnQgc2lnOwogewotICAg IHN0cnVjdCBzaWdhY3Rpb24gYWN0LCBvbGRhY3Q7Ci0KLSAgICBhY3Quc2FfaGFuZGxlciA9IFNJ R19JR047Ci0gICAgYWN0LnNhX2ZsYWdzID0gMDsKLSAgICBzaWdhY3Rpb24oc2lnLCAmYWN0LCAm b2xkYWN0KTsKLSAgICBraWxsKDAsIHNpZyk7Ci0gICAgc2lnYWN0aW9uKHNpZywgJm9sZGFjdCwg TlVMTCk7CisgICAgaWYgKHNpZ2lzbWVtYmVyKCZpZ25vcmVkX3NpZ25hbHMsIHNpZykpCisgICAg eworCXNpZ2RlbHNldCgmaWdub3JlZF9zaWduYWxzLCBzaWcpOworICAgIH0KKyAgICBlbHNlCisg ICAgeworCXNpZ2FkZHNldCgmaWdub3JlZF9zaWduYWxzLCBzaWcpOworCWtpbGwoMCwgc2lnKTsK KyAgICB9CiB9CiAKIApAQCAtMTM4Miw2ICsxMzg5LDcgQEAKIAkvKiBTZW5kIHRoZSBzaWduYWwg dG8gdGhlIFtkaXNdY29ubmVjdG9yIHByb2Nlc3MoZXMpIGFsc28gKi8KIAlraWxsX215X3BnKHNp Zyk7CiAgICAgbm90aWZ5KHNpZ3JlY2VpdmVkLCBzaWcpOworICAgIHN0YXR1cyA9IEVYSVRfVVNF Ul9SRVFVRVNUOwogICAgIGlmICh3YWl0aW5nKQogCXNpZ2xvbmdqbXAoc2lnam1wLCAxKTsKIH0K T25seSBpbiBwcHAtMi40LjItbmV3L3BwcGQ6IG1haW4uY34KT25seSBpbiBwcHAtMi40LjItbmV3 L3BwcGQ6IE1ha2VmaWxlCmRpZmYgLXJ1IHBwcC0yLjQuMi9wcHBkL3BwcGQuaCBwcHAtMi40LjIt bmV3L3BwcGQvcHBwZC5oCi0tLSBwcHAtMi40LjIvcHBwZC9wcHBkLmgJMjAwMy0wNC0wNyAwMzow MTo0NiArMDMwMAorKysgcHBwLTIuNC4yLW5ldy9wcHBkL3BwcGQuaAkyMDA2LTA5LTI2IDE2OjQ5 OjI4ICswMzAwCkBAIC00NTYsNiArNDU2LDcgQEAKICAqLwogCiAvKiBQcm9jZWR1cmVzIGV4cG9y dGVkIGZyb20gbWFpbi5jLiAqLwordm9pZCBoYW5kbGVfZXZlbnRzIF9fUCgodm9pZCkpOwkvKiB3 YWl0IGZvciBzbXRoIHRvIGhhcHBlbiBhbmQgcmVzcG9uZCB0byBpdCovCiB2b2lkIHNldF9pZnVu aXQgX19QKChpbnQpKTsJLyogc2V0IHN0dWZmIHRoYXQgZGVwZW5kcyBvbiBpZnVuaXQgKi8KIHZv aWQgZGV0YWNoIF9fUCgodm9pZCkpOwkvKiBEZXRhY2ggZnJvbSBjb250cm9sbGluZyB0dHkgKi8K IHZvaWQgZGllIF9fUCgoaW50KSk7CQkvKiBDbGVhbnVwIGFuZCBleGl0ICovCmRpZmYgLXJ1IHBw cC0yLjQuMi9wcHBkL3N5cy1saW51eC5jIHBwcC0yLjQuMi1uZXcvcHBwZC9zeXMtbGludXguYwot LS0gcHBwLTIuNC4yL3BwcGQvc3lzLWxpbnV4LmMJMjAwNC0wMS0xMyAwNjowNToyMCArMDIwMAor KysgcHBwLTIuNC4yLW5ldy9wcHBkL3N5cy1saW51eC5jCTIwMDYtMDktMjYgMTY6MDQ6MDIgKzAz MDAKQEAgLTk5NSw3ICs5OTUsOSBAQAogCWlmICghZGVmYXVsdF9kZXZpY2UpCiAJICAgIGluaXR0 ZXJtaW9zLmNfbGZsYWcgJj0gfihFQ0hPIHwgRUNIT05MKTsKIAotCWlmICh0Y3NldGF0dHIodHR5 X2ZkLCBUQ1NBRkxVU0gsICZpbml0dGVybWlvcykgPCAwKSB7CisJLy8gVWdseSBoYWNrIC0gd2l0 aCBUQ1NBRkxVU0ggaXQgaGFuZ3MgaWYgbm8gbW9kZW0gaXMgY29ubmVjdGVkCisJLy8gdG8gQ09N IHBvcnQuCisJaWYgKHRjc2V0YXR0cih0dHlfZmQsIFRDU0FOT1csICZpbml0dGVybWlvcykgPCAw KSB7CiAJICAgIGlmICghIG9rX2Vycm9yIChlcnJubykpCiAJCXdhcm4oInRjc2V0YXR0cjogJW0g KGxpbmUgJWQpIiwgX19MSU5FX18pOwogCX0KZGlmZiAtcnUgcHBwLTIuNC4yL3BwcGQvdHR5LmMg cHBwLTIuNC4yLW5ldy9wcHBkL3R0eS5jCi0tLSBwcHAtMi40LjIvcHBwZC90dHkuYwkyMDA0LTAx LTEzIDA2OjE3OjU5ICswMjAwCisrKyBwcHAtMi40LjItbmV3L3BwcGQvdHR5LmMJMjAwNi0wOS0y NiAxNjo1MzoxMSArMDMwMApAQCAtNjcxLDYgKzY3MSw3IEBACiAJCQkJc3RhdHVzID0gRVhJVF9J TklUX0ZBSUxFRDsKIAkJCQlyZXR1cm4gLTE7CiAJCQl9CisJCQloYW5kbGVfZXZlbnRzKCk7CiAJ CQlpZiAoa2lsbF9saW5rKSB7CiAJCQkJZGlzY29ubmVjdF90dHkoKTsKIAkJCQlyZXR1cm4gLTE7 CkBAIC02ODEsOSArNjgyLDExIEBACiAJCWlmIChjb25uZWN0b3IgJiYgY29ubmVjdG9yWzBdKSB7 CiAJCQlpZiAoZGV2aWNlX3NjcmlwdChjb25uZWN0b3IsIHR0eWZkLCB0dHlmZCwgMCkgPCAwKSB7 CiAJCQkJZXJyb3IoIkNvbm5lY3Qgc2NyaXB0IGZhaWxlZCIpOwotCQkJCXN0YXR1cyA9IEVYSVRf Q09OTkVDVF9GQUlMRUQ7CisJCQkJaWYgKHN0YXR1cyAhPSBFWElUX1VTRVJfUkVRVUVTVCkKKwkJ CQkgICAgc3RhdHVzID0gRVhJVF9DT05ORUNUX0ZBSUxFRDsKIAkJCQlyZXR1cm4gLTE7CiAJCQl9 CisJCQloYW5kbGVfZXZlbnRzKCk7CiAJCQlpZiAoa2lsbF9saW5rKSB7CiAJCQkJZGlzY29ubmVj dF90dHkoKTsKIAkJCQlyZXR1cm4gLTE7CkBAIC03MTAsNiArNzEzLDcgQEAKIAkJCQllcnJvcigi RmFpbGVkIHRvIHJlb3BlbiAlczogJW0iLCBkZXZuYW0pOwogCQkJCXN0YXR1cyA9IEVYSVRfT1BF Tl9GQUlMRUQ7CiAJCQl9CisJCQloYW5kbGVfZXZlbnRzKCk7CiAJCQlpZiAoIXBlcnNpc3QgfHwg ZXJybm8gIT0gRUlOVFIgfHwgaHVuZ3VwIHx8IGtpbGxfbGluaykKIAkJCQlyZXR1cm4gLTE7CiAJ CX0K