<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>21738</bug_id>
          
          <creation_ts>2009-09-27 04:12:46 +0400</creation_ts>
          <short_desc>incorrect USEMODULES detect</short_desc>
          <delta_ts>2010-02-21 20:42:30 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>startup</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>15333</blocked>
    
    <blocked>22929</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter>led</reporter>
          <assigned_to name="Alexey Gladkov">legion</assigned_to>
          <cc>eostapets</cc>
    
    <cc>glebfm</cc>
    
    <cc>ldv</cc>
    
    <cc>legion</cc>
    
    <cc>mike</cc>
    
    <cc>sr</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>100182</commentid>
    <comment_count>0</comment_count>
    <who name="">led</who>
    <bug_when>2009-09-27 04:12:46 +0400</bug_when>
    <thetext>В /etc/rc.d/rc.sysinit USEMODULES выставляется в &quot;y&quot; только при наличии /proc/ksyms или /proc/kallsyms:

f [ -z &quot;$USEMODULES&quot; ]; then
        if [ -f /proc/ksyms -o -f /proc/kallsyms ] &amp;&amp; ! grep -iwqs nomodules /proc/cmdline; then
                USEMODULES=y
        else
                USEMODULES=n
        fi
fi

что, как мне кажется, неверным.
Предлагаю заменить строку

if [ -f /proc/ksyms -o -f /proc/kallsyms ] &amp;&amp; ! grep -iwqs nomodules /proc/cmdline; then

на

if [ -f /proc/modules ] &amp;&amp; ! grep -iwqs nomodules /proc/cmdline; then

Тогда не придётся собирать ядро с обязательным включением CONFIG_KALLSYMS=y или выставлением других костылей.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100183</commentid>
    <comment_count>1</comment_count>
    <who name="">led</who>
    <bug_when>2009-09-27 04:38:09 +0400</bug_when>
    <thetext>http://git.altlinux.org/people/led/packages/startup.git?p=startup.git;a=commitdiff;h=a80295598c0ec9a991e049cbcb5b36033c077645</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100316</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2009-09-28 20:59:33 +0400</bug_when>
    <thetext>Ах так вот кто меня заставил влепить лишних пару сотен килобайт в терминальное ядро! :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100448</commentid>
    <comment_count>3</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-29 20:54:47 +0400</bug_when>
    <thetext>А давно старый код сломался?  Я бы хотел указать в commit message версию ядра, начиная с которой он мог работать неправильно.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100450</commentid>
    <comment_count>4</comment_count>
    <who name="">led</who>
    <bug_when>2009-09-29 21:01:03 +0400</bug_when>
    <thetext>(В ответ на комментарий №3)
&gt; А давно старый код сломался?  Я бы хотел указать в commit message версию ядра,
&gt; начиная с которой он мог работать неправильно.

USEMODULES указывает на то, собрано ли ядро с CONFIG_MODULES=y или нет. /proc/ksyms и /proc/kallsyms никогда не являлось показателем того,что ядро модульное. /proc/modules же присутствует всегда, когда ядро собрано с CONFIG_MODULES=y (по крайней мере, начиная с 2.6.12 - точно)
В текущем же состоянии для того, чтобы нормально работала автозагрузка модулей, приходится собирать ядро с CONFIG_KALLSYMS=y, который (по определению) нужен только для отладки</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100453</commentid>
    <comment_count>5</comment_count>
    <who name="">led</who>
    <bug_when>2009-09-29 21:03:42 +0400</bug_when>
    <thetext>Я посмотрел по истории пакета startup: проверка модульности по /proc/ksyms была принесена из mandrake, после появления ядра 2.6 была сделана калька для него (/proc/kallsyms), но эта рповерка абсолютно не является проверкой ядра на &quot;модульность&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100466</commentid>
    <comment_count>6</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2009-09-30 03:55:06 +0400</bug_when>
    <thetext>startup-0.9.8.25-alt1 -&gt; sisyphus:

* Wed Sep 30 2009 Dmitry V. Levin &lt;ldv@altlinux&gt; 0.9.8.25-alt1

- rc.sysinit: Detect USEMODULES by /proc/modules only (Led; closes: #21738).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100467</commentid>
    <comment_count>7</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2009-09-30 03:56:42 +0400</bug_when>
    <thetext>(In reply to comment #3)
&gt; Я бы хотел указать в commit message версию ядра,
&gt; начиная с которой он мог работать неправильно.
Наблюдал на kernel-image-tmc-tc с 2.6.27-alt1 по 2.6.27-alt3, в alt4 по
подсказке led@ (случайно наступившего на те же грабли у себя) сделал
CONFIG_KALLSYMS=y.

(In reply to comment #6)
&gt; startup-0.9.8.25-alt1 -&gt; sisyphus:
&gt; * Wed Sep 30 2009 Dmitry V. Levin &lt;ldv@altlinux&gt; 0.9.8.25-alt1
&gt; - rc.sysinit: Detect USEMODULES by /proc/modules only (Led; closes: #21738).
Спасибо!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>106962</commentid>
    <comment_count>8</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2010-02-21 16:30:46 +0300</bug_when>
    <thetext>Странно, но поймал те же грабли на p5/branch со startup-0.9.8.26-alt1 и kernel-image-tmc-tc-2.6.27-alt6.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>106966</commentid>
    <comment_count>9</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2010-02-21 16:52:49 +0300</bug_when>
    <thetext>Вставил отладку после if..fi, который правился коммитом по ссылке в #c1 -- USEMODULES=y.  Сейчас попробую подсунуть ядро с CONFIG_KALLSYMS=y.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>106977</commentid>
    <comment_count>10</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2010-02-21 20:42:30 +0300</bug_when>
    <thetext>Не, без разницы -- извиняюсь за шум.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>