From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KEXdr-0001Gs-6J for mharc-grub-devel@gnu.org; Thu, 03 Jul 2008 18:51:07 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KEXdo-0001Gb-IT for grub-devel@gnu.org; Thu, 03 Jul 2008 18:51:04 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KEXdn-0001G7-GK for grub-devel@gnu.org; Thu, 03 Jul 2008 18:51:03 -0400 Received: from [199.232.76.173] (port=42261 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KEXdn-0001G2-AE for grub-devel@gnu.org; Thu, 03 Jul 2008 18:51:03 -0400 Received: from aybabtu.com ([69.60.117.155]:42808) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1KEXdm-0000ib-RY for grub-devel@gnu.org; Thu, 03 Jul 2008 18:51:02 -0400 Received: from [192.168.10.10] (helo=thorin) by aybabtu.com with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.69) (envelope-from ) id 1KEXZo-0007Tv-V6 for grub-devel@gnu.org; Fri, 04 Jul 2008 00:46:57 +0200 Received: from rmh by thorin with local (Exim 4.63) (envelope-from ) id 1KEXd7-00073b-PD for grub-devel@gnu.org; Fri, 04 Jul 2008 00:50:21 +0200 Date: Fri, 4 Jul 2008 00:50:21 +0200 From: Robert Millan To: The development of GRUB 2 Message-ID: <20080703225021.GA4074@thorin> References: <20080606162339.GA14162@thorin> <87prpu4wkt.fsf@xs4all.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <87prpu4wkt.fsf@xs4all.nl> Organization: free as in freedom X-Message-Flag: Worried about Outlook viruses? Switch to Thunderbird! www.mozilla.com/thunderbird X-Debbugs-No-Ack: true User-Agent: Mutt/1.5.13 (2006-08-11) X-detected-kernel: by monty-python.gnu.org: Genre and OS details not recognized. Subject: Re: [PATCH] disk/fs_uuid.c 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: Thu, 03 Jul 2008 22:51:04 -0000 On Thu, Jul 03, 2008 at 09:05:54PM +0200, Marco Gerards wrote: > Robert Millan writes: > > > This disk driver allows access to partitions that contain a filesystem with > > known UUID via "(UUID=xxxxxxx)" syntax. > > > > The iterator is unimplemented on purpose, because it makes the code smaller > > and it isn't useful to waste time inspecting the same devices twice while > > scanning for LVM / whatsoever. > > > > Also note that as it currently stands it is highly inefficient. I can't > > understand why; new UUIDs are searched everytime on each open() call, but > > this shouldn't be much of an issue since we have a disk cache in kernel to > > make that efficient. At least, it isn't e.g. for grub_fs_probe() which is > > called multiple times. > > > > Even with this efficiency problem, since it doesn't interfere with anything > > else I'm inclined to commit it, if nobody objects. > > Why didn't you just extend the search command? I think that would be > favorable. I _did_ extend the search command (before this patch). The reason for providing this is basicaly that it makes grub.cfg more readable (and easier to understand), and can't be used outside the context of scripting (for example, to set your root/prefix to it, but that would require a patch I intend to put together soon). -- Robert Millan I know my rights; I want my phone call! What good is a phone call… if you are unable to speak? (as seen on /.)