From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1MOQbl-0006oA-4j for mharc-grub-devel@gnu.org; Wed, 08 Jul 2009 02:26:21 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MOQbj-0006mo-Iy for grub-devel@gnu.org; Wed, 08 Jul 2009 02:26:19 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MOQbg-0006kD-2e for grub-devel@gnu.org; Wed, 08 Jul 2009 02:26:19 -0400 Received: from [199.232.76.173] (port=52651 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MOQbf-0006k5-R3 for grub-devel@gnu.org; Wed, 08 Jul 2009 02:26:15 -0400 Received: from c60.cesmail.net ([216.154.195.49]:4602) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.60) (envelope-from ) id 1MOQbf-0001MQ-6C for grub-devel@gnu.org; Wed, 08 Jul 2009 02:26:15 -0400 Received: from unknown (HELO smtprelay2.cesmail.net) ([192.168.1.112]) by c60.cesmail.net with ESMTP; 08 Jul 2009 02:26:13 -0400 Received: from [192.168.1.151] (c-69-141-194-35.hsd1.pa.comcast.net [69.141.194.35]) by smtprelay2.cesmail.net (Postfix) with ESMTPSA id 02E5034C6A for ; Wed, 8 Jul 2009 02:34:38 -0400 (EDT) From: Pavel Roskin To: The development of GRUB 2 In-Reply-To: <8s7bi6-vqf.ln1@ppp121-45-136-118.lns11.adl6.internode.on.net> References: <4A51AE6A.6060708@impulze.org> <2m2ai6-i7b.ln1@ppp121-45-136-118.lns11.adl6.internode.on.net> <1246927104.31361.43.camel@mj> <1246931931.2549.21.camel@mj> <1246940583.2549.57.camel@mj> <8s7bi6-vqf.ln1@ppp121-45-136-118.lns11.adl6.internode.on.net> Content-Type: text/plain Date: Wed, 08 Jul 2009 02:26:05 -0400 Message-Id: <1247034365.2895.16.camel@ct> Mime-Version: 1.0 X-Mailer: Evolution 2.26.3 (2.26.3-1.fc11) Content-Transfer-Encoding: 7bit X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Subject: Re: [PATCH] Caseless UUID comparsion in search command X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Jul 2009 06:26:19 -0000 On Tue, 2009-07-07 at 16:21 +0930, Arthur Marsh wrote: > ok, after applying the "second take" search.c patch, I get: > > sh:grub> ls -l > Device hd0: Partition table > Partition hd0,7: Filesystem type ext2, Last modification time > 2009-07-07 > 06:24:14 Tuesday, UUID 96c96a61-8615-4715-86d0-09cb8c62638c > Partition hd0,6: Filesystem type fat, UUID 7417-5aff > Partition hd0,5: Unknown filesystem > Partition hd0,1: Filesystem type ext2, Last modification time > 2009-07-07 > 06:26:23 Tuesday, UUID bfdeb6d6-0b77-4beb-a63d-bdc3e455b8ea So it's something triggered by a condition in the real bootloader. > >>> sh:grub> search -l "" > >>> Segmentation fault > >> > >> This should be fixed in Subversion. My mistake. Please test it. The > >> patch for unifying search won't help solve this problem. > > now I get: > > sh:grub> search -l "" > hd0,7 hd0,1 > sh:grub> Good. > >>> In real grub: > >>> > >>> ls -l > >>> > >>> hd0: Partition table > >>> Partition hd0,1: Filesystem cannot be accessed > >>> Device hd1: filesysetm cannot be accessed > >>> Device hd2: filesystem cannot be accessed > >>> Device fd0: Filesystem cannot be accessed > >>> error: no such disk > > With the second-take patch I get the same result. It looks like there are several issues are at play here. There is an issue with hd0,1, and then there is an issue with error handling. I think we should deal with the error handling first, as it will stand in the way. > With real grub and the first and second-take patch I get: > > search -l "" > > hd1,5 hd1,3 It looks like hd0 is ignored. > > gcc --version > > gcc.real (Debian 4.3.3-13) 4.3.3 I think this version has no issues with nested functions. > > If needed, I could try a different version of gcc: > > > > /usr/bin/gcc-3.4 /usr/bin/gcc-4.2 /usr/bin/gcc-4.4 > > /usr/bin/gcc-4.1 /usr/bin/gcc-4.3 No, I don't think anymore that gcc is the real culprit here. -- Regards, Pavel Roskin