Hello!
I have noticed that GRUB won't list all files on the boot partition of
my PowerMAC. I can still get the full list if I mount the partition in
Linux of use hfsutils.
I put the image here:
http://red-bean.com/proski/sda2.img.bz2
It's just 210k compressed. The problem can be reproduced on
On Tue, 2008-01-15 at 19:20 +0100, Robert Millan wrote:
> On Tue, Jan 15, 2008 at 01:08:51PM -0500, Pavel Roskin wrote:
> >
> > What hardware is it for? XO?
>
> Yes. Although I did the port on qemu (but it's known to work on XO).
That's great!
> > How is it different from i386-linuxbios? Do
On Tue, Jan 15, 2008 at 12:18:55AM +0100, Robert Millan wrote:
> Is this better?
> + char *msg = " The highlighted entry will be booted automatically in
> %ds.";
> + char *msg_end = grub_strchr (msg, '%');
> +
> + grub_printf (second_stage ? msg_end : msg, timeout);
Yes; howe
Hi,
i finish converting the idct and color code, now it should be ok.
/*
* GRUB -- GRand Unified Bootloader
* Copyright (C) 2008 Free Software Foundation, Inc.
*
* GRUB is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as pu
Here's an incomplete (missing powerpc & sparc) version of the patch that
would sanitize this function call.
In the meantime i386 just puts the value in %ecx (third argument) to match
what powerpc is doing.
--
Robert Millan
I know my rights; I want my phone call!
What use is a phone call, if
Committed.
--
Robert Millan
I know my rights; I want my phone call!
What use is a phone call, if you are unable to speak?
(as seen on /.)
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
Am Tue, 15 Jan 2008 19:05:47 +0100 schrieb Robert Millan:
> Is the "LBIO" signature schedule to change due to CoreBoot rename?
> Should we check for another one as well?
I'm not sure if the rename extends to that, given that it's an
incompatible interface change.
I'd go with just "LBIO" for the t
On Tue, Jan 15, 2008 at 06:36:40PM +, Patrick Georgi wrote:
> Am Tue, 15 Jan 2008 19:06:25 +0100 schrieb Robert Millan:
> > Is there a way to reproduce this problem without specific hardware?
> > (with qemu or so)
> qemu 0.9.0 exhibits this issue here (using coreboot v2 and grub2-cvs).
> the
Am Tue, 15 Jan 2008 19:09:19 +0100 schrieb Robert Millan:
> - I heard CoreBoot was deprecating ELF for size reasons.
Version 3 splits them up, yes. Version 2 will keep ELF.
> - On an earlier discussion (about boot issues on Apple hardware),
> Pavel
> proposed switching util/elf/grub-mki
Am Tue, 15 Jan 2008 19:06:25 +0100 schrieb Robert Millan:
> Is there a way to reproduce this problem without specific hardware?
> (with qemu or so)
qemu 0.9.0 exhibits this issue here (using coreboot v2 and grub2-cvs).
the keys "1","2","3" lead to "u","a","l" because of table mismatch.
Regards,
On Tue, Jan 15, 2008 at 01:08:51PM -0500, Pavel Roskin wrote:
>
> What hardware is it for? XO?
Yes. Although I did the port on qemu (but it's known to work on XO).
> How is it different from i386-linuxbios? Doesn't LinuxBIOS imply
> OpenBIOS, which is (or strives to be) ieee1275 compliant?
L
New patch. Same thing, but includes halt, reboot and suspend commands.
--
Robert Millan
I know my rights; I want my phone call!
What use is a phone call, if you are unable to speak?
(as seen on /.)
* configure.ac: Add `i386-ieee1275' to the list of supported targets.
* conf/i386-ieee1275.
On Sat, Jan 12, 2008 at 06:02:10PM +, Patrick Georgi wrote:
> Hi,
>
> third installment in my mini-series - the fourth one vanished on the way.
>
> Move segment table to the beginning of the ELF image, so it can be used
> while parsing the rest.
I have two concerns before investing time in
On Tue, 2008-01-15 at 18:59 +0100, Robert Millan wrote:
> This patch adds initial i386-ieee1275 support.
Just a few (almost) stupid questions.
What hardware is it for? XO?
How is it different from i386-linuxbios? Doesn't LinuxBIOS imply
OpenBIOS, which is (or strives to be) ieee1275 compliant
On Sat, Jan 12, 2008 at 05:53:42PM +, Patrick Georgi wrote:
> Hi,
>
> second installement of my mini-series:
>
> The AT keyboard driver assumes that the keyboard is set to scancode set
> #1. It seems like many keyboards use set #2 (or even #3) by default now,
> and some 8042-emulating chips
On Sat, Jan 12, 2008 at 05:44:42PM +, Patrick Georgi wrote:
>
> First issue: Right now, grub2 has a hard coded location for the lbtables
> and gives up if they're not there.
> According to the coreboot sources, these tables can start anywhere
> between 0x500 and 0x1000, or between 0xf an
On Tue, Jan 15, 2008 at 12:42:39PM -0500, Pavel Roskin wrote:
>
> On Tue, 2008-01-15 at 18:17 +0100, Robert Millan wrote:
> > - grub_dprintf ("mem", "Using memory for heap: addr=%p, size=%u\n",
> > addr, (unsigned int) size);
> > + grub_dprintf ("mem", "Using memory for heap: start=%p, end=%p\n"
This patch adds initial i386-ieee1275 support.
--
Robert Millan
I know my rights; I want my phone call!
What use is a phone call, if you are unable to speak?
(as seen on /.)
* configure.ac: Add `i386-ieee1275' to the list of supported targets.
* conf/i386-ieee1275.rmk: New file.
* include
On Tue, 2008-01-15 at 18:17 +0100, Robert Millan wrote:
> - grub_dprintf ("mem", "Using memory for heap: addr=%p, size=%u\n",
> addr, (unsigned int) size);
> + grub_dprintf ("mem", "Using memory for heap: start=%p, end=%p\n",
> addr, addr + size);
Maybe addr+size-1 would be better? Inclusive b
This has bothered me for a while. By changing %u to %p this debug
message becomes actually useful for checking offset problems.
--
Robert Millan
I know my rights; I want my phone call!
What use is a phone call, if you are unable to speak?
(as seen on /.)
* kern/mm.c (grub_mm_init_region):
On Tue, Jan 15, 2008 at 01:06:43PM +0100, Robert Millan wrote:
> > >/* Load pre-loaded modules and free the space. */
> > >grub_register_exported_symbols ();
> > > - grub_load_modules ();
> > > +// grub_load_modules ();
> >
> > Why?
>
> I still don't know. Memory corruption I think.
On Jan 16, 2008 12:01 AM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> The license state that their README should be distributed together
> with GRUB 2, if I understand it correctly. As you might guess, this
> is not acceptable. Please be careful when using code from other
> projects (or even bette
On Tue, Jan 15, 2008 at 09:41:19AM -0500, Pavel Roskin wrote:
> Quoting Robert Millan <[EMAIL PROTECTED]>:
>
> >On Tue, Jan 15, 2008 at 01:12:57PM +0100, Robert Millan wrote:
> >>
> >>Ok. Checked in the first one. I'll complete the second now.
> >
> >Here. Please can you test?
>
> It's working
Bean <[EMAIL PROTECTED]> writes:
[...]
>> > thanks, if there is no problem, i would commit this now.
>>
>> If you mean your patch, that's fine for me. Please do not forget to
>> rename script_1. Are there any other issues I brought up?
>
> ok, commited.
Thanks! :-)
--
Marco
___
Bean <[EMAIL PROTECTED]> writes:
[...]
>> Same for the other two lines. With this change, only the legal
>> question remains.
>
> oh, i see.
>
> The IJG's source code:
>
> http://www.ijg.org/files/jpegsrc.v6b.tar.gz
>
> there is a legal issue section in the README file.
The license state that t
On Jan 15, 2008 10:33 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
>
> > On Jan 15, 2008 8:41 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> >> Right. I am completely unhappy with the lexer as it is now... :-/
> >> The main reason for writing it manually is tha
Quoting Robert Millan <[EMAIL PROTECTED]>:
On Tue, Jan 15, 2008 at 01:12:57PM +0100, Robert Millan wrote:
Ok. Checked in the first one. I'll complete the second now.
Here. Please can you test?
It's working fine for me. I don't see any difference.
--
Regards,
Pavel Roskin
___
On Jan 15, 2008 10:27 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
>
> [...]
>
> >> > for (i = 0; i < cc; i++)
> >> > {
> >> > int id, ss;
> >> >
> >> > id = grub_jpeg_get_byte (data) - 1;
> >> > if ((id < 0) || (id >= 3))
> >> > re
Bean <[EMAIL PROTECTED]> writes:
> On Jan 15, 2008 8:41 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>> Right. I am completely unhappy with the lexer as it is now... :-/
>> The main reason for writing it manually is that flex depends on all
>> kinds of stuff I do not want in GRUB. Like reading f
Bean <[EMAIL PROTECTED]> writes:
[...]
>> > for (i = 0; i < cc; i++)
>> > {
>> > int id, ss;
>> >
>> > id = grub_jpeg_get_byte (data) - 1;
>> > if ((id < 0) || (id >= 3))
>> > return grub_error (GRUB_ERR_BAD_FILE_TYPE, "jpeg: invalid index");
>> >
>> > ss = gr
Bean <[EMAIL PROTECTED]> writes:
> On Jan 15, 2008 8:46 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>> Robert Millan <[EMAIL PROTECTED]> writes:
>>
>> > On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote:
>> >> >>
>> >> >> I do not think these algorithms can be copyrighted and are in t
On Tue, Jan 15, 2008 at 01:51:23PM +0100, Marco Gerards wrote:
> > +void
> > +print_timeout (int timeout, int offset, int second_stage)
>
> please make this function static.
>
> Otherwise, I do not see any problems.
Committed.
--
Robert Millan
I know my rights; I want my phone call!
What us
On Jan 15, 2008 8:41 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Right. I am completely unhappy with the lexer as it is now... :-/
> The main reason for writing it manually is that flex depends on all
> kinds of stuff I do not want in GRUB. Like reading files, etc.
>
> Thanks a lot for fixing
On Jan 15, 2008 7:01 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> > static grub_err_t
> > grub_jpeg_decode_sof (struct grub_jpeg_data *data)
> > {
> > int i, cc;
> > grub_uint32_t next_marker;
> >
> > next_marker = data->file->offset;
> > next_marker += grub_jpeg_get_word (data);
> >
> >
On Jan 15, 2008 8:46 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> > On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote:
> >> >>
> >> >> I do not think these algorithms can be copyrighted and are in the
> >> >> public domain? Can you check
Robert Millan <[EMAIL PROTECTED]> writes:
> On Tue, Jan 15, 2008 at 01:25:25PM +0100, Marco Gerards wrote:
>> Robert Millan <[EMAIL PROTECTED]> writes:
>>
>> > On Mon, Jan 14, 2008 at 09:32:39PM +0100, Yoshinori K. Okuji wrote:
>> >> On Monday 14 January 2008 15:10, Robert Millan wrote:
>> >> > A
Robert Millan <[EMAIL PROTECTED]> writes:
> On Tue, Jan 15, 2008 at 01:23:30PM +0100, Marco Gerards wrote:
>> > OFW expects %eax to be the first parameter in callbacks. Other than this,
>> > I assumed -mregparm is a useful size optimization. Is that not it? Why
>> > do we have it on i386-pc the
Robert Millan <[EMAIL PROTECTED]> writes:
> On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote:
>> >>
>> >> I do not think these algorithms can be copyrighted and are in the
>> >> public domain? Can you check this?
>> >
>> > Only actual code can be copyrighted. Algorithms can only be
On Tue, Jan 15, 2008 at 01:25:25PM +0100, Marco Gerards wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> > On Mon, Jan 14, 2008 at 09:32:39PM +0100, Yoshinori K. Okuji wrote:
> >> On Monday 14 January 2008 15:10, Robert Millan wrote:
> >> > As subject says. Based on suggestions from Vesa.
>
Bean <[EMAIL PROTECTED]> writes:
>> > if grub_error is set, it will effect the parser, caused menu not to be
>> > showed, etc.
>>
>> Right, but do you want to see a menu if not everything is correctly executed?
>
> the problem is that it's not always an error when grub_errno is set,
> for example,
On Tue, Jan 15, 2008 at 01:30:48PM +0100, Marco Gerards wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> > Integer properties are always in big endian. This (incomplete) patch is my
> > suggested approach to deal with this. I'd like to receive input about it
> > before I complete it (check
On Tue, Jan 15, 2008 at 01:23:30PM +0100, Marco Gerards wrote:
> > OFW expects %eax to be the first parameter in callbacks. Other than this,
> > I assumed -mregparm is a useful size optimization. Is that not it? Why
> > do we have it on i386-pc then?
>
> On i386-pc the size restrictions are mor
On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote:
> >>
> >> I do not think these algorithms can be copyrighted and are in the
> >> public domain? Can you check this?
> >
> > Only actual code can be copyrighted. Algorithms can only be patented.
> >
> > Copyright expires into public d
> > if grub_error is set, it will effect the parser, caused menu not to be
> > showed, etc.
>
> Right, but do you want to see a menu if not everything is correctly executed?
the problem is that it's not always an error when grub_errno is set,
for example, the test command use it to report whether
Robert Millan <[EMAIL PROTECTED]> writes:
> Integer properties are always in big endian. This (incomplete) patch is my
> suggested approach to deal with this. I'd like to receive input about it
> before I complete it (checking all calls to grub_ieee1275_get_property
> through GRUB and replacing
On Tue, Jan 15, 2008 at 01:17:31PM +0100, Marco Gerards wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> > On Tue, Jan 15, 2008 at 12:23:19PM +0100, Marco Gerards wrote:
> >> Robert Millan <[EMAIL PROTECTED]> writes:
> >>
> >> > New patch, a bit simpler (after giving it a though, I don't th
Robert Millan <[EMAIL PROTECTED]> writes:
> On Mon, Jan 14, 2008 at 09:32:39PM +0100, Yoshinori K. Okuji wrote:
>> On Monday 14 January 2008 15:10, Robert Millan wrote:
>> > As subject says. Based on suggestions from Vesa.
>>
>> Why do you want to use a nested function?
>
> No special reason; j
On Tue, Jan 15, 2008 at 01:12:57PM +0100, Robert Millan wrote:
>
> Ok. Checked in the first one. I'll complete the second now.
Here. Please can you test?
--
Robert Millan
I know my rights; I want my phone call!
What use is a phone call, if you are unable to speak?
(as seen on /.)
* incl
Robert Millan <[EMAIL PROTECTED]> writes:
[...]
>> Great! I'll just review the patch! :-)
>
> Please don't. It's not meant for review, was just a proof of concept. The
> patches I actually intend to get merged are being sent separately (see my
> later
> mails on this).
>
> Oh well, too late..
Robert Millan <[EMAIL PROTECTED]> writes:
> On Tue, Jan 15, 2008 at 12:01:47PM +0100, Marco Gerards wrote:
>> Bean <[EMAIL PROTECTED]> writes:
>>
>> > On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>> >> > /*
>> >> > * GRUB -- GRand Unified Bootloader
>> >> > * Copyright
Robert Millan <[EMAIL PROTECTED]> writes:
> On Tue, Jan 08, 2008 at 09:18:47PM +0800, Bean wrote:
>> On Jan 8, 2008 9:14 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
>> >
>> > On Tue, Jan 08, 2008 at 09:06:41PM +0800, Bean wrote:
>> > > On Jan 8, 2008 9:00 PM, Martin Braure de Calignon <[EMAIL PRO
Robert Millan <[EMAIL PROTECTED]> writes:
> On Tue, Jan 15, 2008 at 12:23:19PM +0100, Marco Gerards wrote:
>> Robert Millan <[EMAIL PROTECTED]> writes:
>>
>> > New patch, a bit simpler (after giving it a though, I don't think the
>> > gained
>> > speed is worth the extra complication)
>>
>> Wha
On Mon, Jan 14, 2008 at 11:49:30PM -0500, Pavel Roskin wrote:
> Quoting Robert Millan <[EMAIL PROTECTED]>:
>
> >>> Again, some testing on Apple/IBM/Genesi hardware would be nice.
> >>
> >>It doesn't apply cleanly,
> >
> >It depends on my previous patch, "restrict parsing of `available'".
>
> Appl
On Tue, Jan 15, 2008 at 12:31:30PM +0100, Marco Gerards wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> Hi,
>
> > I've been working for a few hours in a GRUB port to i386/OFW. There's
> > quite a bit of things that need cleanup/fix before it can be considered
> > complete, but it's in a s
On Tue, Jan 15, 2008 at 12:23:19PM +0100, Marco Gerards wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> > New patch, a bit simpler (after giving it a though, I don't think the gained
> > speed is worth the extra complication)
>
> What does this patch fix? Too many problems in one thread..
On Tue, Jan 15, 2008 at 12:01:47PM +0100, Marco Gerards wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> >> > /*
> >> > * GRUB -- GRand Unified Bootloader
> >> > * Copyright (C) 2008 Free Software Foundation, Inc.
> >>
> >>
Done.
On Tue, Jan 15, 2008 at 12:13:01PM +0100, Marco Gerards wrote:
> Christian Franke <[EMAIL PROTECTED]> writes:
>
>
> [...]
>
> > 2008-01-14 Christian Franke <[EMAIL PROTECTED]>
> >
> > * util/grub-emu.c: Replace by .
> > (argp_program_version): Remove variable.
> > (argp_pr
Robert Millan <[EMAIL PROTECTED]> writes:
Hi,
> I've been working for a few hours in a GRUB port to i386/OFW. There's
> quite a bit of things that need cleanup/fix before it can be considered
> complete, but it's in a stage that boots and lets you do basic things (like
> listing storage devices)
Robert Millan <[EMAIL PROTECTED]> writes:
> New patch, a bit simpler (after giving it a though, I don't think the gained
> speed is worth the extra complication)
What does this patch fix? Too many problems in one thread...
BTW, does it also work for the case both LVM and RAID are used?
--
Marc
"Yoshinori K. Okuji" <[EMAIL PROTECTED]> writes:
> On Sunday 13 January 2008 13:19, Robert Millan wrote:
>> This is becoming a nightmare. I think it's better if we just exclude
>> symbols starting with __ from our checks. They aren't really meant for us
>> to mess with.
>
> My original suggestio
Christian Franke <[EMAIL PROTECTED]> writes:
[...]
> 2008-01-14 Christian Franke <[EMAIL PROTECTED]>
>
> * util/grub-emu.c: Replace by .
> (argp_program_version): Remove variable.
> (argp_program_bug_address): Likewise.
> (options): Convert from struct argp_option to s
Vesa Jääskeläinen <[EMAIL PROTECTED]> writes:
> Robert Millan wrote:
>> On Sat, Jan 12, 2008 at 04:14:38PM -0800, walt wrote:
>>> Unfortunately, ata.mod still doesn't recognize the disk on the Promise
>>> sata controller. But, that's expected at this point?
>>
>> Yes. Some SATA controllers emula
Bean <[EMAIL PROTECTED]> writes:
> On Jan 14, 2008 2:42 AM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>> > @@ -99,6 +101,9 @@ grub_script_execute_cmdline (struct grub_script_cmd
>> > *cmd)
>> >grubcmd = grub_command_find (cmdline->cmdname);
>> >if (! grubcmd)
>> > {
>> > + /* Ign
Bean <[EMAIL PROTECTED]> writes:
> On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>> > /*
>> > * GRUB -- GRand Unified Bootloader
>> > * Copyright (C) 2008 Free Software Foundation, Inc.
>>
>> Did you write all the code yourself?
>
> yes, but i take some code segment from
Hi,
as Okuji suggested, the filter interface should be more general. I'm
thinking about the following implementaion.
1, the filehook structure:
struct grub_filehook
{
char *name;
int priority;
grub_file_t (*open_func) (grub_file_t file);
struct grub_filehook *next;
};
name is the text r
65 matches
Mail list logo
