Bug 133993 - resume line missing from grub's menu.lst
Summary: resume line missing from grub's menu.lst
Status: RESOLVED FIXED
: 145025 148208 156598 157010 157452 (view as bug list)
Alias: None
Product: SUSE Linux 10.1
Classification: openSUSE
Component: Installation (show other bugs)
Version: Beta 2
Hardware: Other Other
: P5 - None : Blocker (vote)
Target Milestone: ---
Assignee: Jiri Srain
QA Contact: Klaus Kämpf
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-16 14:43 UTC by Andreas Jaeger
Modified: 2006-03-27 15:19 UTC (History)
9 users (show)

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments
Yast log files (879.17 KB, application/x-bzip2)
2005-11-16 14:46 UTC, Andreas Jaeger
Details
menu.lst created during upgrade from 10.0 to 10.1beta2 (900 bytes, text/plain)
2006-01-30 07:43 UTC, Will Stephenson
Details
I could reproduce this. These are the logs from /var/log/YaST2. (135.18 KB, application/x-gtar)
2006-02-22 14:03 UTC, Olaf Dabrunz
Details
Log from build 801 (2.28 MB, text/plain)
2006-03-17 22:26 UTC, Jiri Srain
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Jaeger 2005-11-16 14:43:28 UTC
diff -u /boot/grub/menu.lst.old /boot/grub/menu.lst
--- /boot/grub/menu.lst.old     2005-09-29 14:32:35.000000000 +0200
+++ /boot/grub/menu.lst 2005-11-16 10:32:00.000000000 +0100
@@ -1,4 +1,4 @@
-# Modified by YaST2. Last modification on Thu Sep 29 14:32:35 CEST 2005
+# Modified by YaST2. Last modification on Wed Nov 16 10:32:00 CET 2005

 color white/blue black/light-gray
 default
@@ -6,13 +6,13 @@
 gfxmenu (hd0,5)/boot/message

 ###Don't change this comment - YaST2 identifier: Original name: linux###
-title SUSE LINUX 10.0.42.42
+title SUSE LINUX 10.0.42
     root (hd0,5)
-    kernel /boot/vmlinuz root=/dev/hda6 vga=0x317 selinux=0 resume=/dev/hda5 splash=silent  showopts
+    kernel /boot/vmlinuz root=/dev/hda6 vga=0x317 selinux=0 splash=silent  showopts
     initrd /boot/initrd

 ###Don't change this comment - YaST2 identifier: Original name: failsafe###
-title Failsafe -- SUSE LINUX 10.0.42.42
+title Failsafe -- SUSE LINUX 10.0.42
     root (hd0,5)
     kernel /boot/vmlinuz root=/dev/hda6 vga=normal showopts ide=nodma apm=off acpi=off noresume selinux=0 nosmp noapic maxcpus=0 edd=off 3
     initrd /boot/initrd
Comment 1 Andreas Jaeger 2005-11-16 14:44:06 UTC
So, resume= is missing - this was an update from 10.1 Alpha1 to 10.1 Alpha3.
Comment 2 Andreas Jaeger 2005-11-16 14:46:15 UTC
Created attachment 57518 [details]
Yast log files
Comment 3 Jiri Srain 2005-12-15 12:12:45 UTC
Reasigning to new maintainer
Comment 4 Andreas Jaeger 2005-12-16 06:37:07 UTC
This is still happening with Alpha4, please get this fixed asap.
Comment 5 Andreas Jaeger 2006-01-15 07:21:06 UTC
Is this fixed now?
Comment 6 Andreas Jaeger 2006-01-20 10:12:22 UTC
Still with 10.1 Beta1.
Comment 7 Olaf Dabrunz 2006-01-20 20:45:59 UTC
Finally got a test-inst-sys on an x86_64 machine running.

The bug does not reproduce on my test machine with SLES10 Beta1. As I doubt
this is specific to 10.1 Beta1 it seems to depend on the machine.

To avoid a lot of guesswork, I need to do some testing in an inst-sys on a
machine where the bug reproduces.

Maybe also newer logs may sched some light on the problem, although I doubt
they will answer all questions.
Comment 8 Andreas Jaeger 2006-01-21 08:40:31 UTC
Have you done an update from a previous system?

Come to my office on monday and let's talk, I have my machine with me.
Comment 9 Olaf Dabrunz 2006-01-22 14:07:37 UTC
Ah right, this was an update.

Bootloader installation for updates is done from the post-install script of the
kernel RPM. AFAI have been told and AFAICS YaST is not directly involved here
(other than providing hints like $YAST_IS_RUNNING). But this may not be
completely accurate.

Anyway this seems to point to package kernel-source, post.sh and further to
/sbin/new-kernel-pkg from mkinitrd as well as probably /sbin/update-bootloader
(and all the perl functions involved) from perl-Bootloader.

As I have not pinpointed the problematic code yet, comments from the
maintainers of these packages should be most helpful. Putting them in CC:.
Comment 10 Andreas Jaeger 2006-01-24 18:38:35 UTC
*** Bug 145025 has been marked as a duplicate of this bug. ***
Comment 11 Christian Boltz 2006-01-29 19:34:31 UTC
same problem after updating to 10.1 beta2...
Comment 12 Will Stephenson 2006-01-30 07:43:49 UTC
Created attachment 65582 [details]
menu.lst created during upgrade from 10.0 to 10.1beta2

Me too, upgraded from 10.0 to beta2.  Was unable to suspend as a result (#146103).
Comment 13 Gavin Hyde 2006-01-30 14:44:09 UTC
After upgrading 10.1 beta1 to 10.1 beta2 I still had the problem, once I added resume=/dev/hda5 like I was told before everything worked again. If you think it's machine specific I have a pentium 4 with hyper threading. 
Comment 14 Andreas Jaeger 2006-02-04 11:49:50 UTC
*** Bug 148208 has been marked as a duplicate of this bug. ***
Comment 15 Andreas Jaeger 2006-02-04 11:51:08 UTC
Bug is still in Beta3.
Comment 16 Stefan Fent 2006-02-06 17:44:42 UTC
Could so. please  post the yast2-logs?
To comment #13: So this entry was there with Beta1 (resume worked) and disappeared after an update to beta2?
A fresh installation of beta2 has this parameter, btw. 
Comment 17 Andreas Jaeger 2006-02-06 19:57:18 UTC
See comment #2 for the log files when this was first reported.

I fixed it manually with each installation, so the entry was there when I did the update from Beta1/Beta2 to the next BETA - and disappeared through the beta process.
Comment 18 Andreas Jaeger 2006-02-06 19:58:09 UTC
For log files see also the duplicate of this PR: 148208.

If you need current ones, please ask again.
Comment 19 m. bracher 2006-02-07 07:25:54 UTC
you can find the actual logs in Bug 148208  (Upgrade from Beta1 -> Beta3)

additional info:
It seems that also the memory parameter for xen was lost
Comment 20 Olaf Dabrunz 2006-02-22 13:58:03 UTC
The bootloader does not provide a resume parameter because it does not find any
available swap partitions:

2005-08-24 01:58:13 <1> linux(3031) [YCP] bootloader/routines/misc.ycp:1296 Available swap partitions: $[]

The disk still has a swap partition after the update (from "disk_hda"):

Partition: 5 /dev/hda5 2096419 3 5 0 261 82 logical

But in the y2log-2 the last time I see the "mount" "swap" assigned to /dev/hda5
is before package installation:

2005-08-24 00:57:17 <1> linux(3031) [YCP] RootPart.ycp:762 mounting /dev/hda5 to swap

But this information comes from RootPart.ycp in yast2-update, which reads the
fstab itself. (The last time Storage.ycp has a ``"mount":"swap"´´ in the
TargetMap is earlier.)

After RootPart.ycp, libstorage does this:

2005-08-24 00:57:17,161 INFO  libstorage - Storage.cc(changeMountPoint):1142 device:/dev/hda5 mount:
2005-08-24 00:57:17,161 INFO  libstorage - Storage.cc(findVolume):3140 device:/dev/hda5 ret:1 c->device:/dev/hda v->device:/dev/hda5
2005-08-24 00:57:17,161 INFO  libstorage - Partition.cc(changeMount):195 device:/dev/hda5 val:
2005-08-24 00:57:17,161 INFO  libstorage - Volume.cc(changeMount):377 device:/dev/hda5 mount:
2005-08-24 00:57:17,161 INFO  libstorage - Volume.cc(changeMount):396 ret:0
2005-08-24 00:57:17,161 INFO  libstorage - Partition.cc(changeMount):200 ret:0

After this the ``"mount":"swap"´´ is gone from the TargetMap:

2005-08-24 00:56:57 <1> linux(3031) [YCP] Storage.ycp:1113 getContainerInfo
container [...]$["detected_fs":`swap, "device":"/dev/hda5", "fsid":130,
"fstype":"Linux swap", "inactive":true, "mount":"swap", "name":"hda5", "nr":5,
"region":[0, 261], "size_k":2096419, "type":`logical, "used_fs":`swap],[...]

This seems to be a problem in yast2-storage.
Comment 21 Olaf Dabrunz 2006-02-22 14:03:57 UTC
Created attachment 69782 [details]
I could reproduce this. These are the logs from /var/log/YaST2.
Comment 22 Holger Macht 2006-03-10 13:46:17 UTC
*** Bug 157010 has been marked as a duplicate of this bug. ***
Comment 23 Stephan Binner 2006-03-11 20:49:43 UTC
*** Bug 156598 has been marked as a duplicate of this bug. ***
Comment 24 Stephan Binner 2006-03-11 20:49:52 UTC
*** Bug 157452 has been marked as a duplicate of this bug. ***
Comment 25 Thomas Fehr 2006-03-14 18:54:54 UTC
Jiri from the logs it looks like the map given to Storage::SetTargetMap in
SetFormatPartitions in RootPart.ycp contains the wrong (empty) mount point settings.

Maybe it would make sense to add a log line with the variable target_map.
Otherwise I do not have any idea why Storage.ycp should forget about the
swap mount point.
Comment 26 Thomas Fehr 2006-03-15 10:50:19 UTC
This is a problem happening during update so I do not see why Olaf should be 
involved. 

Jiri, are you still resonsible for yast2-update?
Comment 27 Jiri Srain 2006-03-15 15:56:34 UTC
Yes, I am.

I added more verbose logging, so that it is clear what OnDiskTarget returned, what was sent for SetTargetMap and also which operations no the target map were performed. Done in SVN, will go to build in a moment.

But this code has not changed since ages...
Comment 28 Thomas Fehr 2006-03-15 16:04:15 UTC
Might very well be that the code is still ok, and the problem is interaction
with changed yast2-storage. But we should see that with your added log entries.

Might also be that all this works fine with a current beta, beta#3 is pretty
old.
Comment 29 Olaf Dabrunz 2006-03-16 16:23:56 UTC
I can not see why this bug was reassigned to me. From the comments this seems
to be between yast2-update and yast2-storage.

Tentatively assigning to fehr@suse.de.
Comment 30 Thomas Fehr 2006-03-16 16:26:39 UTC
Jiri, could you try an update with more verbose yast2-update, so that we can
see why this gets lost.
Comment 31 Jiri Srain 2006-03-17 22:26:45 UTC
Created attachment 73776 [details]
Log from build 801

Look for "On disk target map".

IMHO this log looks OK...
Comment 32 Thomas Fehr 2006-03-20 12:03:35 UTC
No, there is no call to SetPartitionData for setting the mount point of the
swap partition /dev/hda2 to "swap" in the target map. Therefore later the
bootloader is not  able to find an active swap partition in the target map.
Active for a swap partition means is has mount points "swap", since it is also 
possible to explicity not use a existing swap partition. 
This results in the missing resume parameter.

You should call SetPartitionData also for the swap partition like for all 
other partitions with a non-empty mount points.
Comment 33 Andreas Jaeger 2006-03-24 17:56:01 UTC
This bug still exists in today's Build 806 :-(