7366 2005-07-14 19:36:45 +0400 Патч для чтения содержимого /etc/fstab.d 2008-01-08 11:12:38 +0300 1 1 4 Development Sisyphus mount unstable all Linux CLOSED NOTABUG P2 normal --- 1 lav inger boyarsh eostapets evg glebfm icesik kas ldv legion mike morozov pilot placeholder sr the_arioch vt qa-sisyphus oldest_to_newest 27284 0 lav 2005-07-14 19:36:45 +0400 Предлагается внести в mount считывание файлов из /etc/fstab.d, реализованное предлагаемым патчем. Если к нему есть замечания, будем исправлять. Надеюсь, это позволит избежать повторения #5889, #5797, #5635 27286 1 988 lav 2005-07-14 19:37:23 +0400 Created attachment 988 /etc/fstab.d reading 27806 2 lav 2005-07-28 17:04:47 +0400 Известно, что /etc/fstab напрямую читается в kdelibs? и glibc 27809 3 eostapets 2005-07-28 18:42:07 +0400 Угу, в kdelibs за это отвечает kdecore/kmountpoint.cpp. Можно попробовать переписать этот код для поддрежки /etc/fstab.d 27882 4 lav 2005-07-30 01:11:29 +0400 А в glibc - misc/fstab.c Возникла идея заменить соответствующие части кода, дублирующие функциональность из glibc, на вызов функций из glibc. 27891 5 eostapets 2005-07-30 12:40:44 +0400 Для надежности было бы неплохо распаковать исходники все wm и их аплетов... И поискать еще вхождения /etc/fstab для того, чтобы представить себе объем работ... 27901 6 ldv 2005-07-31 20:02:44 +0400 Вы предлагаете изменить getfsent(3), или ограничиться утилитой mount(8)? Вообще, плохо, конечно, что так много софта реализуют свой собственный getfsent(8), надо бы такие программы обвешать багами. 27903 7 lav 2005-07-31 22:44:15 +0400 Предлагаю: - изменить getfsent (как это уже сделано для mount) - обнаружить и развесить баги на программы, которые читают /etc/fstab самобытно - написать для вышеозначенных программ патчи с тем, чтобы они использовали getfsent Не знаю, насколько данная задача для вас приоритетна; если данный план разумен, то я мог бы поручить это одному из разработчиков @etersoft 27904 8 ldv 2005-08-01 00:06:56 +0400 Не стану утверждать, что эта задача для меня приоритетна. План действий выглядит логично. Убедить glibc'шный upstream в необходимости расширения функциональности getfsent будет очень сложно. Впрочем, этот факт не является препятствием. 27945 9 eostapets 2005-08-02 13:41:48 +0400 После детельного изучения kdelibs с него снимаются все обвинения - используется getmntent из glibc 27946 10 sr 2005-08-02 14:13:18 +0400 > После детельного изучения kdelibs с него снимаются все обвинения - используется > getmntent из glibc Но это добавляет проблемы, если getfsent пропатчить на предмет /etc/fstab.d как два байта переслать, то интерфейс getmntent и так ориентирован на множество файлов и унесен на уровень конечного приложения, т.е. придется перелопачивать весь софт для поиска getmntent и патчить, патчить и еще раз патчить. 27947 11 eostapets 2005-08-02 14:23:15 +0400 Ага, начиная с coreutils и заканчивая несколькими сотнями аплетов для разнообразных WM... Стоит ли овчинка выделки? 29817 12 lav 2005-09-01 19:30:59 +0400 Я считаю что стоит и по возможности буду участвовать в приведении программ к использованию библиотечной функции из glibc. 29845 13 sr 2005-09-02 11:30:40 +0400 (In reply to comment #12) > Я считаю что стоит и по возможности буду участвовать в приведении программ к > использованию библиотечной функции из glibc. Какой именно, getfsent или getmntent? getfsent обсолитнули и суперсиднули getmntent. А обвязка вокруг getmntent для поддержки /etc/fstab.d будет выглядеть байт в байт как текущий getfsent с трехстрочным патчем. 31088 14 the_arioch 2005-09-28 13:19:35 +0400 fstab.d - это серьезно. Это надо апстрим убеждать соглажаться :-) IMHO нехорошо, что getmntent предполагает, что "/etc/fstab" и "/etc/mtab" (? "/ proc/mounts") зашиты в приложениях как константы. Нет функций, их возвращающих. Если сделать функции, возвращающие список файлов fstab(/etc/fstab и /etc/fstab. d) и список файлов mtab (пока один единств.) - то вполне можно реализовать getfsent через getmntent + один список. И аналогично сделать getmfsent (mounted fs). И вот это все проталкивать в upstream. Делать обвязку для getmntent, чтобы ей указывали на один файл, а она читала из другого - это жуть IMHO. ....а нельзя сделать что-то типа shadow ? Чтобы /etc/fstab был сокетом, за который отвечал бы демон. А демон бы рабивал fstab на постоянную часть и fstab.d ? Правда это уже по сложности что-то типа CVS получается 60582 15 lav 2008-01-08 07:29:21 +0300 Поскольку выбранным направлением стали новые методы автомонтирования, и изменение /etc/fstab более не требуется, и риску порчи он более не подвержен, закрываю. 988 2005-07-14 19:37:23 +0400 2005-07-14 19:37:23 +0400 /etc/fstab.d reading util-linux-2.12p-fstab.d.patch text/plain 1794 lav UGF0Y2ggZm9yIHJlYWRpbmcgL2V0Yy9mc3RhYi5kIGZpbGVzIGFmdGVyIC9ldGMvZnN0YWIKQXV0 aG9yOiBEYW5paWwgRGF2eWRvdiA8a29vY2h5QGV0ZXJzb2Z0LnJ1PgotLS0gL2hvbWUva29vY2h5 L1Byb2plY3RzL2ZzdGFiLmMJMjAwNC0xMi0yMSAyMjowOToyNCArMDMwMAorKysgdXRpbC1saW51 eC0yLjEycC9tb3VudC9mc3RhYi5jCTIwMDUtMDctMTQgMTg6MjQ6MzYgKzA0MDAKQEAgLTksNiAr OSw3IEBACiAjaW5jbHVkZSA8c3RkaW8uaD4KICNpbmNsdWRlIDxzdHJpbmcuaD4KICNpbmNsdWRl IDxzeXMvc3RhdC5oPgorI2luY2x1ZGUgPGRpcmVudC5oPgogI2luY2x1ZGUgIm1udGVudC5oIgog I2luY2x1ZGUgImZzdGFiLmgiCiAjaW5jbHVkZSAic3VuZHJpZXMuaCIKQEAgLTE3MywyNiArMTc0 LDU4IEBACiAJcmVhZF9tbnRlbnRjaG4obWZwLCBmbmFtLCBtYyk7CiB9CiAKLXN0YXRpYyB2b2lk Ci1yZWFkX2ZzdGFiKCkgewotCW1udEZJTEUgKm1mcCA9IE5VTEw7Ci0JY29uc3QgY2hhciAqZm5h bTsKLQlzdHJ1Y3QgbW50ZW50Y2huICptYyA9ICZmc3RhYjsKLQotCWdvdF9mc3RhYiA9IDE7Ci0J bWMtPm54dCA9IG1jLT5wcmV2ID0gTlVMTDsKK3N0YXRpYyBpbnQKK3JlYWRfZnN0YWJfZmlsZShj b25zdCBjaGFyICpmbmFtLCBzdHJ1Y3QgbW50ZW50Y2huICptYyl7CiAKLQlmbmFtID0gX1BBVEhf RlNUQUI7CisJbW50RklMRSAqbWZwID0gTlVMTDsKKwkKIAltZnAgPSBteV9zZXRtbnRlbnQgKGZu YW0sICJyIik7CisKIAlpZiAobWZwID09IE5VTEwgfHwgbWZwLT5tbnRlbnRfZnAgPT0gTlVMTCkg ewogCQlpbnQgZXJyc3YgPSBlcnJubzsKIAkJZXJyb3IoXygid2FybmluZzogY2FuJ3Qgb3BlbiAl czogJXMiKSwKLQkJICAgICAgX1BBVEhfRlNUQUIsIHN0cmVycm9yIChlcnJzdikpOwotCQlyZXR1 cm47CisJCSAgICAgIGZuYW0sIHN0cmVycm9yIChlcnJzdikpOworCQlyZXR1cm4gMDsKIAl9CiAJ cmVhZF9tbnRlbnRjaG4obWZwLCBmbmFtLCBtYyk7CisJcmV0dXJuIDE7CisKK30KKworc3RhdGlj IHZvaWQKK3JlYWRfZnN0YWIoKSB7CisKKwlzdHJ1Y3Qgc3RhdCBzdDsKKwljaGFyICpkaXJfcGF0 aDsKKwlESVIgKmRpcjsKKwlzdHJ1Y3QgZGlyZW50ICplbnRyeTsKKwljaGFyIGVudHJ5X3BhdGhb UEFUSF9NQVggKyAxXTsKKwlzaXplX3QgcGF0aF9sZW47CisJc3RydWN0IG1udGVudGNobiAqbWMg PSAmZnN0YWI7CisKKwltYy0+bnh0ID0gbWMtPnByZXYgPSBOVUxMOworCWdvdF9mc3RhYiA9IDE7 CisKKwlpZighcmVhZF9mc3RhYl9maWxlKF9QQVRIX0ZTVEFCLG1jKSkgcmV0dXJuOworCisJZGly X3BhdGggPSBfUEFUSF9GU1RBQiAiLmQvIjsJCisJc3RybmNweSAoZW50cnlfcGF0aCwgZGlyX3Bh dGgsIHNpemVvZiAoZW50cnlfcGF0aCkpOworCWRpciA9IG9wZW5kaXIgKGRpcl9wYXRoKTsKKwlw YXRoX2xlbiA9IHN0cmxlbiAoZGlyX3BhdGgpOworCisJd2hpbGUoKGVudHJ5PXJlYWRkaXIoZGly KSkhPU5VTEwpIHsKKworCQlzdHJuY3B5IChlbnRyeV9wYXRoICsgcGF0aF9sZW4sIGVudHJ5LT5k X25hbWUsCisJCQkgc2l6ZW9mIChlbnRyeV9wYXRoKSAtIHBhdGhfbGVuKTsKKwkJbHN0YXQgKGVu dHJ5X3BhdGgsICZzdCk7CisJCWlmIChTX0lTUkVHIChzdC5zdF9tb2RlKSkKKwkJCXJlYWRfZnN0 YWJfZmlsZShlbnRyeV9wYXRoLG1jKTsKKwkJCisJCQorCX0KKwljbG9zZWRpcihkaXIpOwogfQot ICAgICAKKwogCiAvKiBHaXZlbiB0aGUgbmFtZSBOQU1FLCB0cnkgdG8gZmluZCBpdCBpbiBtdGFi LiAgKi8gCiBzdHJ1Y3QgbW50ZW50Y2huICoK