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

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

    <bug>
          <bug_id>39567</bug_id>
          
          <creation_ts>2021-01-19 18:55:54 +0300</creation_ts>
          <short_desc>В инсталяторе не работает форматирование в xfs</short_desc>
          <delta_ts>2021-08-10 11:08:44 +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>libevms</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>P5</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>33000</blocked>
    
    <blocked>40561</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Антон Мидюков">antohami</reporter>
          <assigned_to name="Олег Соловьев">mcpain</assigned_to>
          <cc>aen</cc>
    
    <cc>boyarsh</cc>
    
    <cc>klark</cc>
    
    <cc>mcpain</cc>
    
    <cc>mike</cc>
    
    <cc>rider</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>195537</commentid>
    <comment_count>0</comment_count>
    <who name="Антон Мидюков">antohami</who>
    <bug_when>2021-01-19 18:55:54 +0300</bug_when>
    <thetext>В инсталяторе не работает форматирование в xfs. Проблема началась после обновления xfsprogs до версии 5.9.0-alt1.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>199059</commentid>
    <comment_count>1</comment_count>
    <who name="Антон Мидюков">antohami</who>
    <bug_when>2021-06-09 01:09:52 +0300</bug_when>
    <thetext>Плагин для работы с файловой системой xfs находится в пакете libevms.

Воспроизведение проблемы:
На шаге &quot;Подготовка диска&quot; выбрать форматирование одного из разделов в xfs.
Нажать кнопку далее. Появляется окно с критической ошибкой: операция не позволена.

В /var/log/evms-engine.log:

июн 09 01:01:57 localhost.localdomain _2_ Engine: is_in_use: Object sda1 is used as volume /dev/evms/sda1.
июн 09 01:02:00 localhost.localdomain _5_ Engine: is_object_change_pending: Change pending: Object sda_mbr is dirty.
июн 09 01:02:00 localhost.localdomain _5_ Engine: is_object_change_pending: Change pending: Object sda3 needs to be activated.
июн 09 01:02:00 localhost.localdomain _5_ Engine: is_volume_change_pending: Change pending: Volume /dev/evms/sda2 needs to have the XFS file system put on it.
июн 09 01:02:00 localhost.localdomain _5_ Engine: is_volume_change_pending: Change pending: Volume /dev/evms/sda3 needs to be activated.
июн 09 01:02:00 localhost.localdomain _5_ Engine: is_volume_change_pending: Change pending: Volume /dev/evms/sda3 needs to have the XFS file system put on it.
июн 09 01:02:02 localhost.localdomain _5_ Engine: is_object_change_pending: Change pending: Object sda_mbr is dirty.
июн 09 01:02:02 localhost.localdomain _5_ Engine: is_object_change_pending: Change pending: Object sda3 needs to be activated.
июн 09 01:02:02 localhost.localdomain _5_ Engine: is_volume_change_pending: Change pending: Volume /dev/evms/sda2 needs to have the XFS file system put on it.
июн 09 01:02:02 localhost.localdomain _5_ Engine: is_volume_change_pending: Change pending: Volume /dev/evms/sda3 needs to be activated.
июн 09 01:02:02 localhost.localdomain _5_ Engine: is_volume_change_pending: Change pending: Volume /dev/evms/sda3 needs to have the XFS file system put on it.
июн 09 01:02:03 localhost.localdomain _2_ XFS: xfs_create: mkfs.xfs completed with exit code 256</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200852</commentid>
    <comment_count>2</comment_count>
    <who name="Slava Aseev">ptrnine</who>
    <bug_when>2021-07-23 14:37:25 +0300</bug_when>
    <thetext>mkfs.xfs не нравится log size, который EVMS хочет задать при форматировании:

[root@localhost altlinux]# fdisk -l /dev/evms/sdb1               
Disk /dev/evms/sdb1: 5 GiB, 5365563392 bytes, 10479616 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes

[root@localhost altlinux]# mkfs.xfs -lsize=4194304 /dev/evms/sdb1
log size 1024 blocks too small, minimum size is 1986 blocks</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200869</commentid>
    <comment_count>3</comment_count>
    <who name="AEN">aen</who>
    <bug_when>2021-07-23 20:11:26 +0300</bug_when>
    <thetext>(Ответ для Slava Aseev на комментарий #2)
&gt; mkfs.xfs не нравится log size, который EVMS хочет задать при форматировании:
&gt; 
&gt; [root@localhost altlinux]# fdisk -l /dev/evms/sdb1               
&gt; Disk /dev/evms/sdb1: 5 GiB, 5365563392 bytes, 10479616 sectors
&gt; Units: sectors of 1 * 512 = 512 bytes
&gt; Sector size (logical/physical): 512 bytes / 512 bytes
&gt; I/O size (minimum/optimal): 512 bytes / 512 bytes
&gt; 
&gt; [root@localhost altlinux]# mkfs.xfs -lsize=4194304 /dev/evms/sdb1
&gt; log size 1024 blocks too small, minimum size is 1986 blocks

Есть предложения?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200887</commentid>
    <comment_count>4</comment_count>
    <who name="Anton V. Boyarshinov">boyarsh</who>
    <bug_when>2021-07-26 10:52:53 +0300</bug_when>
    <thetext>(Ответ для AEN на комментарий #3)
&gt; (Ответ для Slava Aseev на комментарий #2)
&gt; &gt; mkfs.xfs не нравится log size, который EVMS хочет задать при форматировании:
&gt; &gt; 
&gt; &gt; [root@localhost altlinux]# fdisk -l /dev/evms/sdb1               
&gt; &gt; Disk /dev/evms/sdb1: 5 GiB, 5365563392 bytes, 10479616 sectors
&gt; &gt; Units: sectors of 1 * 512 = 512 bytes
&gt; &gt; Sector size (logical/physical): 512 bytes / 512 bytes
&gt; &gt; I/O size (minimum/optimal): 512 bytes / 512 bytes
&gt; &gt; 
&gt; &gt; [root@localhost altlinux]# mkfs.xfs -lsize=4194304 /dev/evms/sdb1
&gt; &gt; log size 1024 blocks too small, minimum size is 1986 blocks
&gt; 
&gt; Есть предложения?

Надо менять ecms, какие ещё могут быть варианты?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200926</commentid>
    <comment_count>5</comment_count>
    <who name="Leonid Krivoshein">klark</who>
    <bug_when>2021-07-26 21:59:20 +0300</bug_when>
    <thetext>(In reply to Anton V. Boyarshinov from comment #4)
&gt; (Ответ для AEN на комментарий #3)
&gt; &gt; Есть предложения?
&gt; Надо менять ecms, какие ещё могут быть варианты?
Видимо имелось ввиду патчить xfs-плагин libevms?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200930</commentid>
    <comment_count>6</comment_count>
    <who name="Anton V. Boyarshinov">boyarsh</who>
    <bug_when>2021-07-27 11:10:23 +0300</bug_when>
    <thetext>(Ответ для Leonid Krivoshein на комментарий #5)
&gt; (In reply to Anton V. Boyarshinov from comment #4)
&gt; &gt; (Ответ для AEN на комментарий #3)
&gt; &gt; &gt; Есть предложения?
&gt; &gt; Надо менять ecms, какие ещё могут быть варианты?
&gt; Видимо имелось ввиду патчить xfs-плагин libevms?

Я не имею ни малейшего представления о внутреннем устройстве evms, но да, очевидно её  надо приводить в соответствие с этими новыми реалиями.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200951</commentid>
    <comment_count>7</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2021-07-27 21:08:19 +0300</bug_when>
    <thetext>https://www.altlinux.org/Bug_Severity_Policy</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200952</commentid>
    <comment_count>8</comment_count>
    <who name="AEN">aen</who>
    <bug_when>2021-07-27 21:11:08 +0300</bug_when>
    <thetext>Как угодно, но мне кажется, что xfs выбирается в установщике &quot;широким кругом пользователей&quot;, потому major.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200953</commentid>
    <comment_count>9</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2021-07-27 21:11:53 +0300</bug_when>
    <thetext>severity в данном случае никак явно не повлияет ни на что, понятно что это надо поправить к дистрибутивам на p10.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201221</commentid>
    <comment_count>10</comment_count>
    <who name="Олег Соловьев">mcpain</who>
    <bug_when>2021-08-04 16:14:46 +0300</bug_when>
    <thetext>evms при расчёте размера лога берёт количество блоков (=512 байт) целевой ФС, делит его на 8196 (очень странная константа, как минимум тем, что не 8192 = 2^13) и округляет его до 4 мегабайт.

Однако в подсказке к параметру evms сам пишет, что размер лога как правило задаётся 0.4% от размера раздела, а расчётный получается в 32 раза меньше.

libxfs считает, что размер лога - от 512 блоков (=2М) до (1024 * 1024) блоков (=4G), а evms считает, что 128М - уже много.

Придётся патчить и это.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201291</commentid>
    <comment_count>11</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2021-08-06 15:44:01 +0300</bug_when>
    <thetext>(Ответ для Олег Соловьев на комментарий #10)
&gt; evms при расчёте размера лога берёт количество блоков (=512 байт) целевой
&gt; ФС, делит его на 8196 (очень странная константа, как минимум тем, что не
&gt; 8192 = 2^13) и округляет его до 4 мегабайт.

Возможно, им надо было добиться значения чуть меньше, чем &quot;круглое&quot;?
А _не_ передавать вручную размер лога, оставив этот вопрос mkfs.xfs -- можно?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201292</commentid>
    <comment_count>12</comment_count>
    <who name="Олег Соловьев">mcpain</who>
    <bug_when>2021-08-06 15:52:16 +0300</bug_when>
    <thetext>(Ответ для Michael Shigorin на комментарий #11)
&gt; Возможно, им надо было добиться значения чуть меньше, чем &quot;круглое&quot;?
С учетом последующего округления до мегабайтов - крайне странное решение.


(Ответ для Michael Shigorin на комментарий #11)
&gt; А _не_ передавать вручную размер лога, оставив этот вопрос mkfs.xfs -- можно?
Он не передаётся вручную. Это дефолтное значение в evms.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201299</commentid>
    <comment_count>13</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2021-08-06 18:24:19 +0300</bug_when>
    <thetext>Возможно, это всего лишь опечатка.  Попытался найти следы вопросов-ответов -- яндекс да гугл ничего внятного по запросу &quot;evms&quot; &quot;xfs&quot; why &quot;8196&quot; not &quot;8192&quot; внятного не выдали; а упоминание 8196 в http://www.kernel.org/doc/mirror/ols2002.pdf (там же и по EVMS был доклад) -- похоже, явная опечатка в ряду: &quot;1024, 2048, 4096, *8196*, 16384, 32768 and 65536&quot; (поскольку без комментариев о причине &quot;особого случая&quot;).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201366</commentid>
    <comment_count>14</comment_count>
    <who name="Олег Соловьев">mcpain</who>
    <bug_when>2021-08-09 18:36:30 +0300</bug_when>
    <thetext>(Ответ для Michael Shigorin на комментарий #13)
&gt; -- похоже, явная опечатка в ряду: &quot;1024, 2048, 4096, *8196*, 16384, 32768
&gt; and 65536&quot; (поскольку без комментариев о причине &quot;особого случая&quot;).

как я уже писал в комм. 10 - расчётное дефолтное значение logsize в 32 раза меньше указанного в самом evms рекомендуемого 0.4% (~1/256) от размера раздела.

Получается 1/8196 от размера раздела, а это слишком мало.

Миш, я не хочу отправлять пакет с молчаливого одобрения, поэтому прошу явного.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201376</commentid>
    <comment_count>15</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2021-08-10 00:02:56 +0300</bug_when>
    <thetext>Очень бегло заглянул в plugins/xfs/ и мне более уверенно кажется, что попросту не надо указывать mkfs.xfs, какой ей делать журнал, если не было явных пожеланий от клиента libevms (или alterator-vm/guile-evms умеют эту подробность?).

Бишь попытки посчитать-влепить свой размер _по умолчанию_ силами evms надо оторвать.

Олег, можешь сделать?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201381</commentid>
    <comment_count>16</comment_count>
    <who name="Олег Соловьев">mcpain</who>
    <bug_when>2021-08-10 09:04:51 +0300</bug_when>
    <thetext>(Ответ для Michael Shigorin на комментарий #15)
&gt; Олег, можешь сделать?

А есть информация, как при этом поведет себя mkfs.xfs?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201388</commentid>
    <comment_count>17</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2021-08-10 10:26:11 +0300</bug_when>
    <thetext>mkfs.xfs /dev/sdXN без дополнительных параметров всю жизнь себя нормально вёл:

basalt:/tmp/.private/mike&gt; dd if=/dev/zero of=test.img bs=1M count=256
256+0 записей получено
256+0 записей отправлено
268435456 байт (268 MB, 256 MiB) скопирован, 0,107019 s, 2,5 GB/s
basalt:/tmp/.private/mike&gt; /sbin/mkfs.xfs test.img
meta-data=test.img               isize=512    agcount=4, agsize=16384 blks
         =                       sectsz=512   attr=2, projid32bit=1
         =                       crc=1        finobt=1, sparse=1, rmapbt=0
         =                       reflink=1    bigtime=0 inobtcount=0
data     =                       bsize=4096   blocks=65536, imaxpct=25
         =                       sunit=0      swidth=0 blks
naming   =version 2              bsize=4096   ascii-ci=0, ftype=1
log      =internal log           bsize=4096   blocks=1368, version=2
         =                       sectsz=512   sunit=0 blks, lazy-count=1
realtime =none                   extsz=4096   blocks=0, rtextents=0
basalt:/tmp/.private/mike&gt; rm test.img
basalt:/tmp/.private/mike&gt; _

:-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201390</commentid>
    <comment_count>18</comment_count>
    <who name="Олег Соловьев">mcpain</who>
    <bug_when>2021-08-10 10:52:42 +0300</bug_when>
    <thetext>(Ответ для Michael Shigorin на комментарий #15)
&gt; Очень бегло заглянул в plugins/xfs/ и мне более уверенно кажется, что
&gt; попросту не надо указывать mkfs.xfs, какой ей делать журнал, если не было
&gt; явных пожеланий от клиента libevms (или alterator-vm/guile-evms умеют эту
&gt; подробность?).

В коде установки параметра -lsize не делается никакой разницы между значением по умолчанию и значением, прилетевшим от пользователя.

Архитектура evms подразумевает, что если имеется какая-то настраиваемая штуковина, то для этой штуковины требуется значение по умолчанию, которое в данный момент зависит от размера раздела.

Если пользователь хочет создать раздел и не хочет мудрить с размером, то для пользователя нет никакой разницы, кто будет мудрить за него и какое значение установит, пока это не создаст никаких проблем.

Если пользователь хочет свой размер, он его укажет в явном виде.

Проблему, описанную в баге, я устранил и намерен отправить изменение.
Насчёт хотелки про отгрызание я подумаю, как реализовать, но _не_ в рамках исправления этого бага.

Таску отправляю.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201393</commentid>
    <comment_count>19</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2021-08-10 11:02:58 +0300</bug_when>
    <thetext>evms-2.5.5-alt50 -&gt; sisyphus:

 Wed Aug 04 2021 Oleg Solovyov &lt;mcpain@altlinux&gt; 2.5.5-alt50
 - plugins/xfs: fix incorrect log size calculation (Closes: #39567)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201394</commentid>
    <comment_count>20</comment_count>
    <who name="AEN">aen</who>
    <bug_when>2021-08-10 11:05:15 +0300</bug_when>
    <thetext>Неужто работает? 
Пожалуйста, тогда в p10.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201395</commentid>
    <comment_count>21</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2021-08-10 11:08:44 +0300</bug_when>
    <thetext>Да, отправьте в p10 пожалуйста.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>