Am Montag, den 11.08.2008, 16:28 +0200 schrieb Robert Millan:
> GRUB_MOD_INIT(raid) could handle them just fine, with something like:
> 
>   grub_print_error ();
>   grub_errno = GRUB_ERR_NONE;
> 
> if an error was found.
> 

Thanks for your hint, this is now something I thought initially with the
invalid BSD partitions.
Showing an error but not aborting the whole thing.
Attached is now a patch wich just adds these 2 things.

I should really take me the time to just look at the Sourcecode and
trying to understand it :)

I don't care much about the changelog, if you e.g. don't like the 2nd
sentence then just remove it please on commit.


2008-08-11  Felix Zielcke  <[EMAIL PROTECTED]>

        * disk/raid.c (GRUB_MOD_INIT): Use grub_print_error() to show RAID 
        errors and reset grub_errno. Do not give errors to the underlying layer.

Index: disk/raid.c
===================================================================
--- disk/raid.c	(Revision 1799)
+++ disk/raid.c	(Arbeitskopie)
@@ -581,6 +581,8 @@
 {
   grub_device_iterate (&grub_raid_scan_device);
   grub_disk_dev_register (&grub_raid_dev);
+  grub_print_error();
+  grub_errno = GRUB_ERR_NONE;
 }
 
 GRUB_MOD_FINI(raid)
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to