From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1ObHhx-0007D0-AJ for mharc-grub-devel@gnu.org; Tue, 20 Jul 2010 14:38:25 -0400 Received: from [140.186.70.92] (port=35617 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1ObHhu-0007Cv-Pj for grub-devel@gnu.org; Tue, 20 Jul 2010 14:38:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1ObHht-0000Ob-HL for grub-devel@gnu.org; Tue, 20 Jul 2010 14:38:22 -0400 Received: from mail.gmx.net ([213.165.64.20]:59966) by eggs.gnu.org with smtp (Exim 4.69) (envelope-from ) id 1ObHht-0000OD-9e for grub-devel@gnu.org; Tue, 20 Jul 2010 14:38:21 -0400 Received: (qmail invoked by alias); 20 Jul 2010 18:38:18 -0000 Received: from 75.217.91-91.rev.gaoland.net (EHLO united.mine.nu) [91.91.217.75] by mail.gmx.net (mp068) with SMTP; 20 Jul 2010 20:38:18 +0200 X-Authenticated: #276141 X-Provags-ID: V01U2FsdGVkX19FfA5CQeNFtP6NdVelBrNdv4+KuMGGcTKevrpPsf MYaYsWxp0Y9dGM Received: from dspam-out (localhost.cryosphere.corp [127.0.0.1]) by nospam.cryosphere.corp (Postfix) with SMTP id 903C520DF1 for ; Tue, 20 Jul 2010 20:38:05 +0200 (CEST) Received: from [192.168.1.13] (snowman.cryosphere.corp [192.168.1.13]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by united.mine.nu (Postfix) with ESMTPS id 1A4CB20D36; Tue, 20 Jul 2010 20:38:04 +0200 (CEST) Message-ID: <4C45ED0C.4010105@gmx.net> Date: Tue, 20 Jul 2010 20:38:04 +0200 From: Jimmy.Jazz@gmx.net User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; fr-FR; rv:1.9.2.4) Gecko/20100627 Lightning/1.0b2pre Lanikai/3.1 MIME-Version: 1.0 To: The development of GNU GRUB References: <1279641407-17848-1-git-send-email-jimmy.jazz@gmx.net> <20100720160831.GQ21862@riva.ucam.org> In-Reply-To: <20100720160831.GQ21862@riva.ucam.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit X-Y-GMX-Trusted: 0 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. Cc: Colin Watson Subject: Re: [PATCH] grub-1.98: support for partitionable array v1.2 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Jul 2010 18:38:24 -0000 Le 20/07/2010 18:08, Colin Watson a écrit : > On Tue, Jul 20, 2010 at 05:56:46PM +0200, Gaëtan Schmidt wrote: >> I modify grub2 to understand mdadm 1.2 partition array without the >> need to map either the metadevice or the partitions in the device.map >> file. Just the physical devices are required. As the meta devices are >> added, they will be recognized automatically. Only raid 1 is supported >> for the moment but it can be extended to other layouts. > You've sent this *just* after I committed patches to trunk to understand > mdadm 1.x metadata. Could you please look at trunk instead of 1.98? > It looks like I hurt your feelings. Sorry for that. For the patches I propose, better late than never :) Anyway, I appreciate your implication in the grub project. >> mdadm 1.1 is not fit to be bootable, its superblock overlaps the boot >> sector 0. > Only if you RAID entire disk devices; it works fine if you RAID > partitions. > That is the main difference and here the patches i *propose* come in handy. You don't have to repartition or reinstall grub on the failed disk. Just add the spare and let's go. It is more like a fake raid but harder to implement w/o scripts. >> I don't use mdX and md/X syntaxes. md_dX is more relevant because of >> the mdadm default 1.x kernel device scheme /dev/md_dX. > We already discussed this and agreed to use md/*. The current mdadm(8) > says: > > From kernel version, 2.6.28 the "non-partitioned array" can actually > be partitioned. So the "md_dNN" names are no longer needed, and > partitions such as "/dev/mdNNpXX" are possible. > > I prefer the explicit namespacing of md/*, and it makes it easier to > support named arrays in a clear way. > That's just about cosmetic. You don't need to declare md partitions or whatever at all in device.map. It's just the way it appears in grub.cfg but the names have to be different from 0.9 arrays' names. >> grub-setup.c and grub-probe.c have been heavily altered to understand >> mdadm 1.2 partitionable array. That means, grub-setup.c does not >> support embedded core image in the MBR first sectors gap (<64k) >> anymore. >> >> To avoid any confusion, grub-setup should be renamed grub-mdraid-setup >> or something like that. > No renames were necessary with the branch recently merged into trunk. > If you need further work to support partitionable arrays, then let's > discuss that; I don't think it's desirable to create a forked version of > grub-setup or grub-probe, though. > Sorry you did not read the code here >> For the try and fix version of the patches, please forget it. It is >> really messy. I am using git for the development and i was not able to >> convert the bzr repository to git, so i don't have a valid tree. >> Anyway, if you are interested in it, please let me know. > Just use bzr instead. :-) > IMHO, that's not the best way to spread a project in the community. But you know now the reason why I'm using grub tarball and delayed so long to share the patches. Also they need to be reviewed. > Regards, > Jj