grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [RFC] change grub_print_error to use stderr for the utils


From: Felix Zielcke
Subject: Re: [RFC] change grub_print_error to use stderr for the utils
Date: Sun, 17 Aug 2008 17:47:03 +0200

Am Sonntag, den 17.08.2008, 17:34 +0200 schrieb Robert Millan:
> Hiding errors is a very bad practice.  And hiding them because the output
> is ugly or because something is reporting false positives is even worse
> IMHO.
> 
> This "out of disk" sounds like a false positive indeed.  It's a bug,
> somewhere.  Something is handling errors in a place where this particular
> error shouldn't be reported.  Maybe it should handle them somewhere else,
> or maybe they should be handled selectively.

This is because of my nForce 4 Hardware RAID 0 (i.e. dmraid).
Linux sees 2 disks (sda,sdb) and the partition table of the first is for
the 2 disks combined.
Both disk have only 80 GB but both combined have 160 GB.

> > The error won't be shown on real GRUB anyway.
> 
> Why not?

Oh I should have said in my case.
The BIOS presents GRUB only one (hd0) for the combined 2 disks.
Whereas the ata.mod from Marco sees both (well it fails to detect one
but in theory)
The BIOS does handle the RAID 0 for booting, but just for that.
As soon as the OS takes over it has to handle it.

> > Maybe just grub-install and update-grub should be a little changed to
> > supress repeating the errors that much, i.e. not on every grub-probe
> > run?
> 
> That's difficult to archieve due to current design, but sounds to me as
> a minor problem in comparison with what we had before.

Yes it isn't that important,
as I already said I only want to find a clean solution for the long
term.
The current way is better then before, but at least I don't like it to
have a error message shown more then needed.

This doestn't look that good either ;)

[...]
error: out of disk
Installation finished. No error reported.






reply via email to

[Prev in Thread] Current Thread [Next in Thread]