[d-kernel] Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[d-kernel] Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1172 bytes --]

----- Forwarded message from bugzilla-daemon@altlinux.ru -----

Date: Tue, 25 Nov 2003 17:04:14 +0300 (MSK)
From: bugzilla-daemon@altlinux.ru
Subject: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub
To: at@altlinux.org

http://bugzilla.altlinux.ru/show_bug.cgi?id=3318

           Summary: При установке любого ядра не прописывается
                    соотвествующая секция в меню grub
           Product: 1. Sisyphus
           Version: unstable
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: bootloader-utils
        AssignedTo: nidd@altlinux.org
        ReportedBy: taf@altlinux.org
         QAContact: at@altlinux.org


При установке любого ядра не прописывается соотвествующая секция в меню grub если
в файле /etc/fstab может встретиться такая конструкция:

#/dev/sda1 / ext2 defaults 1 1
/dev/sda1 / xfs defaults 1 1

скрипт /sbin/installkernel подхватывает название раздела с первой строки и,
естественно, обламывается.

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

----- End forwarded message -----

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [d-kernel] Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Sergey Vlasov]

[-- Attachment #1: Type: text/plain, Size: 198 bytes --]

On Tue, Nov 25, 2003 at 05:09:35PM +0300, Alexey Tourbin wrote:
>          Component: bootloader-utils
>         AssignedTo: nidd@altlinux.org

Так что, за bootloader-utils у нас никто не отвечает?

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 285 bytes --]

On Tue, Nov 25, 2003 at 05:16:44PM +0300, Sergey Vlasov wrote:
> On Tue, Nov 25, 2003 at 05:09:35PM +0300, Alexey Tourbin wrote:
> >          Component: bootloader-utils
> >         AssignedTo: nidd@altlinux.org
> Так что, за bootloader-utils у нас никто не отвечает?

Похоже что нет.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 658 bytes --]

On Tue, Nov 25, 2003 at 05:25:22PM +0300, Alexey Tourbin wrote:
> > >          Component: bootloader-utils
> > >         AssignedTo: nidd@altlinux.org
> > Так что, за bootloader-utils у нас никто не отвечает?
> 
> Похоже что нет.

Видимо, дело в этом:

--- /usr/share/loader/grub-	2003-03-19 17:41:06 +0300
+++ /usr/share/loader/grub	2003-11-25 17:34:24 +0300
@@ -154,6 +154,7 @@ sub get_boot_partitions {
     local *F;
     open F, '/etc/fstab'; 
     while (<F>) {
+	next if /^#/;
  	my @s = split ' ';
 	$part = $s[0] if $s[1] =~ m|/$| and not $part;
 	if ($s[1] =~ m|/boot$|) {

Вообще, там есть что по части перла зафиксить.
Или на shell'е переписать.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Anton Farygin]

Alexey Tourbin wrote:
> On Tue, Nov 25, 2003 at 05:25:22PM +0300, Alexey Tourbin wrote:
> 
>>>>         Component: bootloader-utils
>>>>        AssignedTo: nidd@altlinux.org
>>>
>>>Так что, за bootloader-utils у нас никто не отвечает?
>>
>>Похоже что нет.
> 
> 
> Видимо, дело в этом:
> 
> --- /usr/share/loader/grub-	2003-03-19 17:41:06 +0300
> +++ /usr/share/loader/grub	2003-11-25 17:34:24 +0300
> @@ -154,6 +154,7 @@ sub get_boot_partitions {
>      local *F;
>      open F, '/etc/fstab'; 
>      while (<F>) {
> +	next if /^#/;
>   	my @s = split ' ';
>  	$part = $s[0] if $s[1] =~ m|/$| and not $part;
>  	if ($s[1] =~ m|/boot$|) {
> 
> Вообще, там есть что по части перла зафиксить.
> Или на shell'е переписать.

По хорошему - надо либо на shell либо на C переписывать.

Кстати, конфигуратор/инсталятор ведь тоже должны прописывать в конфиги 
загрузчиков ядра... можно совместить.

Rgds,
Rider

P.S.
По моему у RedHat есть неплохая реализация на C + shell

Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Michael Shigorin]

[-- Attachment #1: Type: text/plain, Size: 817 bytes --]

On Tue, Nov 25, 2003 at 05:54:28PM +0300, Anton Farygin wrote:
> >Вообще, там есть что по части перла зафиксить.
> >Или на shell'е переписать.

Догадываюсь.  Заодно было бы неплохо сделать и возможность
установки _более_ примитивных целей, чем полновесное ядро --
вроде того же memtest86, где попросту не получалось обойтись
installkernel -- он рвался поставить.сделать еще что-то :)

> По хорошему - надо либо на shell либо на C переписывать.
> Кстати, конфигуратор/инсталятор ведь тоже должны прописывать в
> конфиги загрузчиков ядра... можно совместить.

Так утилита отдельная всяко полезна; ее (или библиотечку) и
дергать.

> P.S.  По моему у RedHat есть неплохая реализация на C + shell

На несколько загрузчиков?

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Anton Farygin]

Michael Shigorin wrote:
> On Tue, Nov 25, 2003 at 05:54:28PM +0300, Anton Farygin wrote:
> 
>>>Вообще, там есть что по части перла зафиксить.
>>>Или на shell'е переписать.
> 
> 
> Догадываюсь.  Заодно было бы неплохо сделать и возможность
> установки _более_ примитивных целей, чем полновесное ядро --
> вроде того же memtest86, где попросту не получалось обойтись
> installkernel -- он рвался поставить.сделать еще что-то :)
> 
> 
>>По хорошему - надо либо на shell либо на C переписывать.
>>Кстати, конфигуратор/инсталятор ведь тоже должны прописывать в
>>конфиги загрузчиков ядра... можно совместить.
> 
> 
> Так утилита отдельная всяко полезна; ее (или библиотечку) и
> дергать.

Так может быть кто-нить реализует на C/C++ в виде библиотеки?


> 
> 
>>P.S.  По моему у RedHat есть неплохая реализация на C + shell
> 
> 
> На несколько загрузчиков?

Да. Grub + Lilo.

Только у них detectloader некорректный и у нас работать не будет.

Rgds,
Rider

[d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 812 bytes --]

On Tue, Nov 25, 2003 at 05:36:09PM +0300, Alexey Tourbin wrote:
> --- /usr/share/loader/grub-	2003-03-19 17:41:06 +0300
> +++ /usr/share/loader/grub	2003-11-25 17:34:24 +0300
> @@ -154,6 +154,7 @@ sub get_boot_partitions {
>      local *F;
>      open F, '/etc/fstab'; 
>      while (<F>) {
> +	next if /^#/;
>   	my @s = split ' ';
>  	$part = $s[0] if $s[1] =~ m|/$| and not $part;
>  	if ($s[1] =~ m|/boot$|) {
> 
> Вообще, там есть что по части перла зафиксить.
> Или на shell'е переписать.

В mdk юмористы (сравниваю с новой версией):

@@ -154,7 +187,7 @@ sub get_boot_partitions {
     local *F;
     open F, '/etc/fstab';
     while (<F>) {
-       next if /^#/;
+       s/#.*//;
        my @s = split ' ';
        $part = $s[0] if $s[1] =~ m|/$| and not $part;
        if ($s[1] =~ m|/boot$|) {

hackish

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Anton Farygin]

Alexey Tourbin wrote:
> On Tue, Nov 25, 2003 at 05:36:09PM +0300, Alexey Tourbin wrote:
> 
>>--- /usr/share/loader/grub-	2003-03-19 17:41:06 +0300
>>+++ /usr/share/loader/grub	2003-11-25 17:34:24 +0300
>>@@ -154,6 +154,7 @@ sub get_boot_partitions {
>>     local *F;
>>     open F, '/etc/fstab'; 
>>     while (<F>) {
>>+	next if /^#/;
>>  	my @s = split ' ';
>> 	$part = $s[0] if $s[1] =~ m|/$| and not $part;
>> 	if ($s[1] =~ m|/boot$|) {
>>
>>Вообще, там есть что по части перла зафиксить.
>>Или на shell'е переписать.
> 
> 
> В mdk юмористы (сравниваю с новой версией):
> 
> @@ -154,7 +187,7 @@ sub get_boot_partitions {
>      local *F;
>      open F, '/etc/fstab';
>      while (<F>) {
> -       next if /^#/;
> +       s/#.*//;
>         my @s = split ' ';
>         $part = $s[0] if $s[1] =~ m|/$| and not $part;
>         if ($s[1] =~ m|/boot$|) {
> 
> hackish

Мда... мне кажется, что я даже знаю - зачем это они сделали ... ;-)

Иначе им будет тяжело выводить об ошибке парсинга в какой-то 
определенной строке fstab.

Т.е. - я бы это реализовал например так (C++)

ifstream fd("/etc/fstab");
char line[512];
unsigned int linenumber = 0;

while(fd.getline(line, sizeof(line), '\n'))
{
    ++linenumber;
    if(line[0] == "#")
	continue;
     parse(line,linenumber);
}

[d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 738 bytes --]

On Wed, Nov 26, 2003 at 02:50:35PM +0300, Anton Farygin wrote:
> >@@ -154,7 +187,7 @@ sub get_boot_partitions {
> >     local *F;
> >     open F, '/etc/fstab';
> >     while (<F>) {
> >-       next if /^#/;
> >+       s/#.*//;
> >        my @s = split ' ';
> >        $part = $s[0] if $s[1] =~ m|/$| and not $part;
> >        if ($s[1] =~ m|/boot$|) {
> >
> >hackish
> 
> Мда... мне кажется, что я даже знаю - зачем это они сделали ... ;-)

А вообще, комментарий в /etc/fstab может быть только в начале строки,
или в любом месте?  чтение man getmntent, fstab, mount меня не
просветлило.  Кроме того, этот код не учитывает noauto.

Без четкой постановки задачи здесь нельзя написать корректный код.
Здесь по натуре требутеся "угадывание".

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Anton Farygin]

Alexey Tourbin wrote:
> On Wed, Nov 26, 2003 at 02:50:35PM +0300, Anton Farygin wrote:
> 
>>>@@ -154,7 +187,7 @@ sub get_boot_partitions {
>>>    local *F;
>>>    open F, '/etc/fstab';
>>>    while (<F>) {
>>>-       next if /^#/;
>>>+       s/#.*//;
>>>       my @s = split ' ';
>>>       $part = $s[0] if $s[1] =~ m|/$| and not $part;
>>>       if ($s[1] =~ m|/boot$|) {
>>>
>>>hackish
>>
>>Мда... мне кажется, что я даже знаю - зачем это они сделали ... ;-)
> 
> 
> А вообще, комментарий в /etc/fstab может быть только в начале строки,
> или в любом месте?  чтение man getmntent, fstab, mount меня не
> просветлило.  Кроме того, этот код не учитывает noauto.
> 
> Без четкой постановки задачи здесь нельзя написать корректный код.
> Здесь по натуре требутеся "угадывание".

Нет, проще посмотреть на парсер в lilo и реализовать подобный.

Rgds,
Rider

Re: [devel] Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Sergey Vlasov]

[-- Attachment #1: Type: text/plain, Size: 1162 bytes --]

On Wed, Nov 26, 2003 at 03:27:18PM +0300, Anton Farygin wrote:
> Alexey Tourbin wrote:
> >On Wed, Nov 26, 2003 at 02:50:35PM +0300, Anton Farygin wrote:
> >
> >>>@@ -154,7 +187,7 @@ sub get_boot_partitions {
> >>>   local *F;
> >>>   open F, '/etc/fstab';
> >>>   while (<F>) {
> >>>-       next if /^#/;
> >>>+       s/#.*//;
> >>>      my @s = split ' ';
> >>>      $part = $s[0] if $s[1] =~ m|/$| and not $part;
> >>>      if ($s[1] =~ m|/boot$|) {
> >>>
> >>>hackish
> >>
> >>Мда... мне кажется, что я даже знаю - зачем это они сделали ... ;-)
> >
> >
> >А вообще, комментарий в /etc/fstab может быть только в начале строки,
> >или в любом месте?  чтение man getmntent, fstab, mount меня не
> >просветлило.  Кроме того, этот код не учитывает noauto.
> >
> >Без четкой постановки задачи здесь нельзя написать корректный код.
> >Здесь по натуре требутеся "угадывание".
> 
> Нет, проще посмотреть на парсер в lilo и реализовать подобный.

А в lilo разве есть парсер fstab?  Насколько я понимаю, там просто
открывается файл и делается bmap, чтобы найти устройство и блок (там ещё
сверху этого навороты для RAID/LVM/..., чтобы добраться до реального
устройства).

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Re: [d-kernel] Re: Fwd: [Bug 3318] New: При установке любого ядра не прописывается соотвествующая секция в меню grub

[Anton Farygin]

Sergey Vlasov wrote:
> On Wed, Nov 26, 2003 at 03:27:18PM +0300, Anton Farygin wrote:
> 
>>Alexey Tourbin wrote:
>>
>>>On Wed, Nov 26, 2003 at 02:50:35PM +0300, Anton Farygin wrote:
>>>
>>>
>>>>>@@ -154,7 +187,7 @@ sub get_boot_partitions {
>>>>>  local *F;
>>>>>  open F, '/etc/fstab';
>>>>>  while (<F>) {
>>>>>-       next if /^#/;
>>>>>+       s/#.*//;
>>>>>     my @s = split ' ';
>>>>>     $part = $s[0] if $s[1] =~ m|/$| and not $part;
>>>>>     if ($s[1] =~ m|/boot$|) {
>>>>>
>>>>>hackish
>>>>
>>>>Мда... мне кажется, что я даже знаю - зачем это они сделали ... ;-)
>>>
>>>
>>>А вообще, комментарий в /etc/fstab может быть только в начале строки,
>>>или в любом месте?  чтение man getmntent, fstab, mount меня не
>>>просветлило.  Кроме того, этот код не учитывает noauto.
>>>
>>>Без четкой постановки задачи здесь нельзя написать корректный код.
>>>Здесь по натуре требутеся "угадывание".
>>
>>Нет, проще посмотреть на парсер в lilo и реализовать подобный.
> 
> 
> А в lilo разве есть парсер fstab?  Насколько я понимаю, там просто
> открывается файл и делается bmap, чтобы найти устройство и блок (там ещё
> сверху этого навороты для RAID/LVM/..., чтобы добраться до реального
> устройства).

Да, конечно не в lilo.

В том же mount должен быть парсер.

Rgds,
Rider