Hell[o]
Some time ago I decide to give a try to GRUB on my Pegasos 2 machine
(G4 1GHz, 1G of ram) and generaly it works with some problems.
1) Booting recent CVS snap on OF version dated 2005.12.16 gives me
only "Welcome to GRUB" and "Out of memory" message. After OF update to
half official 2006.
Hell[o]
Now it is time for efika.
1) Backspace key is not working as on my Pegasos
2) It gives me "Out of memory" message on boot. I wonder why patch
from http://lists.gnu.org/archive/html/grub-devel/2007-08/msg00031.html
is not on cvs ? As it fixes grub on efika for me. I tired also a
previous
Hell[o]
Finaly I found a few free minutes to look at ofconsole as it never
correctly work for me. There was 3 problems on my Pegasos:
1) Backspace key not works with USB keyboard.
2) menu looks ugly like hell as it contains some random characters in
place of borders.
3) Console works wrong (the c
Hell[o]
> [...]
OK, split fixes in to separate patches this time as requested.
01-grub2-simple_console.patch
==
This is a idea how to handle ascii&&cp473&&utf based menus for
diffrent consoles. I added a two term flags GRUB_TERM_SIMPLE_MENU
(ascii menu) and GRUB_TERM_CP4
Hell[o]
> Finally, I think I got this right now. My efika boots and all the other
> setups should be happy as well. Please, review/test this and report.
Tested on ODW && Efika, works fine in both cases.
--
--- Marcin 'Morgoth' Kurek ---
___
Grub-d
Hell[o]
> Is this a bitmask? In this case, looks like you aren't using the right
> numbers. Now GRUB_TERM_SIMPLE_MENU == GRUB_TERM_DUMB | GRUB_TERM_NO_EDIT
> OTOH if that's what you intended, I think it's better to build one macro
> using the others.
Eeeek, why ? The flags are checked using 'fl
Hell[o]
> [...]
I just checked cvs grub2 with this patch on older version of pegasos
firmware (1.2 - 20040810) and it seems to fail with out of memory
message.
Then it seems to fix problem for efika and for latest version of
pegasos firmware.
--
--- Marcin 'Morgoth' Kurek ---
___
Hell[o]
Attached current version of console fix'es patches, generaly synced
with cvs && cleanups for all of them and some changes in
04-grub2-ofconsole_fix.patch.
First it seems pegasos OF uses 24 lines console by default, but it
seems when no framebuffer is used the real number of rows is 25 not
Hell[o]
The affs suppors is broken ATM for ieee1275 as it uses total_sectors
from grub_disk_t and this field is not valid on OF. There is still
some problems for partitions with block size != 512, but at last it
makes possible to use it in most cases.
--
--- Marcin 'Morgoth' Kurek ---
diff -urN
Hell[o]
> What happens if you increase HEAP_MIN_SIZE ?
Still fails I am affraid. I wonder is there any reason to use only
first 4MB for heap ? I guess higer memory areas can be still accessed
excluding 4-16 MB ?
I attached my proposed patch to solve this problem. It tires to
allocate 4MB heap at
Hell[o]
> When GRUB starts initial_context is in an unknown state. This can lead to
> GRUB detecting bogus variables from it. Attached patch zeroes that struct
> during initialisation to avoid that.
This change broken command line argument handling for ieee1275.
Arguments are read by grub_machi
Hell[o]
> Avoid the cosmetical changes! (there are more thorough the patch)
Arghh, I was sure I removed it :/ Anyway attached updated console patches.
01-grub2-simple_console.patch: Cleanups only. Should have no more
"cosmetical" changes inside.
04-grub2-ofconsole_fix.patch: Added detection of
Hell[o]
When testing grub on Pegasos 1 machine I was hit by quite weird
problem when reading partition table. It seems my friends firmware
version do not set status to 0 after grub_ieee1275_read() even if the
read was fine.
Attached patch fixed the problem.
--
--- Marcin 'Morgoth' Kurek ---
dif
Hell[o]
> woops. actual is never != actual!! what is this supposed to do? If it works
> like grub_ieee1275_seek, I suppose what we want is to zero it and then compare
> it with 0? (instead of with itself ;-)).
I must be completly blind to not notice this ;(
--
--- Marcin 'Morgoth' Kurek ---
Hell[o]
> Caught you! ;-)
Holy shit. Lesson one take one more look at diff before send it :)
> I know it seems burdensome, but please can you split this in three patches,
> one
> for each fix? Then it's easier to review just one and say "this is good", and
> also easier to figure out why every
Hell[o]
> Right, this is because it seems that the MSB is lost when writing a
> character to the screen. For the borders specific characters are used
> above 127 (non-ASCII). I wonder if the Pegasos console supports this
> ASCII extension if you change some settings or so...
The problem is in f
Hell[o]
> I prefer if it can be detected if this is Efika or for example an
> apple implementation of OF and handle these characters depending on
> that. IIRC this code does work on the apple, but unfortunately I
> cannot check this anymore.
Attached current console patchset. The graphical frame
Hell[o]
Attached rest of my grub changes.
affs: As mentioned on grub_ofdisk_open() the total_sectors value is
hosed on ieee1275 then we can not use it here. Anyway the code is
still a bit broken :( I will try to fix it in a free time in future.
Read: Fixed grub_ofdisk_read() for old versions of
Hell[o]
> [...]
I test this patch on Pegasos 1 and Pegasos 2 and in both cases grub
image hangs on OF prompt when I tired to boot it :( Reverting it makes
grub bootable again.
--
--- Marcin 'Morgoth' Kurek ---
___
Grub-devel mailing list
Grub-devel@
Hell[o]
> What do we need to do with this one? Perhaps you could investigate and
> report?
If I remember correctly I alreay send a patch for this one ... or I
forgot ? Sorry I have some personal troubles around and only a little
free time for computers :(
--
--- Marcin 'Morgoth' Kurek ---
_
Hell[o]
> You sent a patch, but the patch didn't fix the problem. The code we have is
> obviously buggy, but we don't know exactly what should it do.
Eeeek, why ? No one told me that there is any problem in it. Maybe you
talk about first one ? Attached current.
--
--- Marcin 'Morgoth' Kurek --
Hell[o]
> Can you provide something that makes use of that?
Hmm, I should have example code somewhere on disk. I will try to find it.
> Will do :-)
I already send e-mail to fsf today as there was no reply to my previous e-mail.
--
--- Marcin 'Morgoth' Kurek ---
_
22 matches
Mail list logo
