Hi,
any suggestion about this patch ? i would like to commit it soon.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
);
return GRUB_ERR_NONE;
}
#endif
static struct grub_video_bitmap_reader jpg_reader = {
.extension = ".jpg",
.reader = grub_video_reader_jpeg,
.next = 0
};
static struct grub_video_bitmap_reader jpeg_reader = {
.extension = ".jpeg",
.reader = grub_video_reader_jpeg,
.next = 0
};
GRUB_MOD_INIT (video_reader_jpeg)
{
grub_video_bitmap_reader_register (&jpg_reader);
grub_video_bitmap_reader_register (&jpeg_reader);
#if defined(JPEG_DEBUG)
grub_register_command ("jpegtest", grub_cmd_jpegtest,
GRUB_COMMAND_FLAG_BOTH, "jpegtest FILE",
"Tests loading of JPEG bitmap.", 0);
#endif
}
GRUB_MOD_FINI (video_reader_jpeg)
{
#if defined(JPEG_DEBUG)
grub_unregister_command ("jpegtest");
#endif
grub_video_bitmap_reader_unregister (&jpeg_reader);
grub_video_bitmap_reader_unregister (&jpg_reader);
}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
newlines commands '}'
> > {
> > @@ -204,10 +203,17 @@ commandblock: '{'
> > ;
> >
> > /* A menu entry. Carefully save the memory that is allocated. */
> > -menuentry: "menuentry" argument newlines commandblock
> > +menuentry: "menuentry" argument
> > + {
> > + grub_script_lexer_ref (state->lexerstate);
> > + } newlines '{'
> > + {
> > + grub_script_lexer_record_start (state->lexerstate);
> > + } newlines commands '}'
>
>
> What was the problem here?
I don't like the idea of deref in another statement, and now we can
use {} separately.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 14, 2008 3:51 AM, Yoshinori K. Okuji <[EMAIL PROTECTED]> wrote:
> On Sunday 13 January 2008 08:38, Bean wrote:
> > Hi,
> >
> > any suggestion about this patch ? i would like to commit it soon.
>
> I am sorry that I forgot to reply.
>
> I would like
if (argc != 1)
return grub_error (GRUB_ERR_BAD_ARGUMENT, "file name required");
grub_video_reader_jpeg (&bitmap, args[0]);
if (grub_errno != GRUB_ERR_NONE)
return grub_errno;
grub_video_bitmap_destroy (bitmap);
return GRUB_ERR_NONE;
}
#endif
static struct grub_video_bitmap_reader jpg_reader = {
.extension = ".jpg",
.reader = grub_video_reader_jpeg,
.next = 0
};
static struct grub_video_bitmap_reader jpeg_reader = {
.extension = ".jpeg",
.reader = grub_video_reader_jpeg,
.next = 0
};
GRUB_MOD_INIT (video_reader_jpeg)
{
grub_video_bitmap_reader_register (&jpg_reader);
grub_video_bitmap_reader_register (&jpeg_reader);
#if defined(JPEG_DEBUG)
grub_register_command ("jpegtest", grub_cmd_jpegtest,
GRUB_COMMAND_FLAG_BOTH, "jpegtest FILE",
"Tests loading of JPEG bitmap.", 0);
#endif
}
GRUB_MOD_FINI (video_reader_jpeg)
{
#if defined(JPEG_DEBUG)
grub_unregister_command ("jpegtest");
#endif
grub_video_bitmap_reader_unregister (&jpeg_reader);
grub_video_bitmap_reader_unregister (&jpg_reader);
}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
t argc, char **args)
{
struct grub_video_bitmap *bitmap = 0;
if (argc != 1)
return grub_error (GRUB_ERR_BAD_ARGUMENT, "file name required");
grub_video_reader_png (&bitmap, args[0]);
if (grub_errno != GRUB_ERR_NONE)
return grub_errno;
grub_video_bitmap_destroy (bitmap
ight want to apply a specify filter to a file, we
can do this use the filter fileystem, such as:
filter -a aa (hd0,1)/aa gzip.be_to_le
now (filter)/aa is (hd0,1)/aa after gzip decompression and big endian
to little endian conversion.
to delete the mapping:
filter -d aa
This c
er_record_stop in menuentry, but now it's both inside
menuentry. i think it's better this way. besides, if you use
commandblock alone in old implemenation, it will cause problem becuase
there is no matching grub_script_lexer_record_stop.
>
> Btw, does this patch incorporate the pr
id = grub_jpeg_get_byte (data) - 1;
> > if ((id < 0) || (id >= 3))
> > return grub_error (GRUB_ERR_BAD_FILE_TYPE, "jpeg: invalid index");
> >
> > ss = grub_jpeg_get_byte (data); /* sampling factor */
> > if (!id)
> > {
&
anks a lot for fixing bugs related to scripting. Lately I am rather
> busy and I was the only person who worked on it. More testing, better
> error handling, etc is appriciated.
>
> I still have an unfinished "test" command on my HD.
thanks, if there is no
e
> >> > they reform copyright law every 20 years to extend it.
> >> >
> >> > I think only patents actually expire in practice.
> >>
> >> What I meant was, that the code Bean used is public domain already.
> >> AFAIK that's not un
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;
> >> >
> >
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... :-/
>
GRUB 2.
I think the color conversion is easy, i can change it myself. but idct
is tricky, it use AA&N algorithm, which is the only effective
solution. besides, the code is not exactly the same, i just take the
basic idea. (it's in jidctfst.c)
--
Bean
_
UB_ERR_BAD_ARGUMENT, "file name required");
grub_video_reader_jpeg (&bitmap, args[0]);
if (grub_errno != GRUB_ERR_NONE)
return grub_errno;
grub_video_bitmap_destroy (bitmap);
return GRUB_ERR_NONE;
}
#endif
static struct grub_video_bitmap_reader jpg_reader = {
.extension
void *hook)
+ auto int NESTED_FUNC_ATTR grub_elf32_load_segment (grub_elf_t elf,
Elf32_Phdr *phdr, void *hook);
+ int NESTED_FUNC_ATTR grub_elf32_load_segment (grub_elf_t elf,
Elf32_Phdr *phdr, void *hook)
{
grub_elf32_load_hook_t load_hook = (grub_elf32_load_hook_t) hook;
grub_addr_t load_addr;
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 17, 2008 8:21 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Thu, Jan 17, 2008 at 04:15:23PM +0800, Bean wrote:
> >
> > You need to add NESTED_FUNC_ATTR to nested callback function that use
> > local variable. here is the patch:
>
> Glad to see you foun
;
+} while (! isleaf);
+
/* If there was a matching record in this leaf node, continue the
iteration until the last record was found. */
- if (isleaf)
-{
- grub_hfs_iterate_records (data, 0, next, 1, it_dir);
- return grub_errno;
-}
-
- return grub_hfs_iterate_dir (data, found, dir, hook);
+ grub_hfs_iterate_records (data, 0, next, 1, it_dir);
+ return grub_errno;
}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 21, 2008 7:24 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Sun, Dec 30, 2007 at 02:48:24PM +0100, Robert Millan wrote:
> > On Sat, Dec 29, 2007 at 05:05:16PM +0800, Bean wrote:
> > > Hi,
> > >
> > > This patch is based on robert's memdis
2008/1/22 heguanbo <[EMAIL PROTECTED]>:
> Dear all
>
> Can anybody give out a list of file systems the GRUB2 support?Thanks a lot.
file system modules are listed in fs.lst:
affs
cpio
ext2
fat
hfs
hfsplus
iso9660
jfs
minix
ntfs
reiserfs
sfs
uf
s about tar format to figure this out.
>
> I guess the meaning of GRUB_ERR_NONE is confusing (no file vs. no
> error), and the return value of grub_cpio_find_file() is not well
> defined.
>
> Perhaps it should be GRUB_ERR_FILE_NOT_FOUND instead.
you're right, i didn't notice this previously.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 21, 2008 6:38 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> Hi Bean,
>
> Thanks for picking this one up!
>
> [...]
>
> >> Moreover, if I mount the image as loop in Linux and remove files "grub"
>
On Jan 21, 2008 6:21 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> Hi,
>
> > i finish converting the idct and color code, now it should be ok.
>
> If you are 100% sure all code is yours and there are no legal issues,
> f
On Jan 22, 2008 9:57 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Tue, Jan 22, 2008 at 12:25:49PM +0800, Bean wrote:
> > > but, what is the advantage in that? Is there any use case in which the
> > > first
> > > option is not good but the second is?
>
as a compatibility layer, yes.
>
> But it seems you want it as a general-purpose option; for that, why not make
> it saner like multiboot? I don't think it's a good idea to compromise our
> boot semantics because of the ones legacy Linux has
+ grub_free (data);
+
+ return grub_errno;
}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
d be
> included. Perhaps a better explaination of the problem to solve,
> instead of what the patch does might help here.
the most important usage of external initrd is to overcome the size
limit of core.img.
--
Bean
___
Grub-devel mailing list
Gr
On Jan 24, 2008 12:15 AM, walt <[EMAIL PROTECTED]> wrote:
> Bean wrote:
> > This patch should fix the problem...
>
> Yes, thanks! I had to use the -l flag before patch would use it,
> and even then the last hunk needed increased fuzz.
>
> Are you going to commit it
On Jan 23, 2008 5:01 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > On Jan 21, 2008 6:38 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> >> Bean <[EMAIL PROTECTED]> writes:
> >>
UNDI support is not difficult, i have written one for grub4dos. maybe
i can port it to grub2 sometime.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
+
+ if (grub_errno != GRUB_ERR_NONE)
+{
+ grub_video_bitmap_destroy (*bitmap);
+ *bitmap = 0;
+ }
+
+ grub_file_close (file);
+ return grub_errno;
+}
+
+#if defined(PNG_DEBUG)
+static grub_err_t
+grub_cmd_pngtest (struct grub_arg_list *state __attribute__ ((unused)),
+ int argc, char **args)
+{
+ struct grub_video_bitmap *bitmap = 0;
+
+ if (argc != 1)
+return grub_error (GRUB_ERR_BAD_ARGUMENT, "file name required");
+
+ grub_video_reader_png (&bitmap, args[0]);
+ if (grub_errno != GRUB_ERR_NONE)
+return grub_errno;
+
+ grub_video_bitmap_destroy (bitmap);
+
+ return GRUB_ERR_NONE;
+}
+#endif
+
+static struct grub_video_bitmap_reader png_reader = {
+ .extension = ".png",
+ .reader = grub_video_reader_png,
+ .next = 0
+};
+
+GRUB_MOD_INIT (video_reader_png)
+{
+ grub_video_bitmap_reader_register (&png_reader);
+#if defined(PNG_DEBUG)
+ grub_register_command ("pngtest", grub_cmd_pngtest,
+GRUB_COMMAND_FLAG_BOTH, "pngtest FILE",
+"Tests loading of PNG bitmap.", 0);
+#endif
+}
+
+GRUB_MOD_FINI (video_reader_png)
+{
+#if defined(PNG_DEBUG)
+ grub_unregister_command ("pngtest");
+#endif
+ grub_video_bitmap_reader_unregister (&png_reader);
+}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 24, 2008 12:18 AM, Bean <[EMAIL PROTECTED]> wrote:
> On Jan 23, 2008 4:54 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> > When is this feature useful? Can you give an example? More features
> > can mean more bugs, more maintainance work, etc. If the featur
On Jan 24, 2008 3:14 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Thu, Jan 24, 2008 at 03:01:33AM +0800, Bean wrote:
> > On Jan 24, 2008 12:18 AM, Bean <[EMAIL PROTECTED]> wrote:
> > > On Jan 23, 2008 4:54 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>
Committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
Committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
copy initrd there.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 24, 2008 5:15 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
>
> On Thu, Jan 24, 2008 at 05:04:56AM +0800, Bean wrote:
> > On Jan 24, 2008 4:15 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> > > The region where memdisk is normally put has no size limit, only
he boot mechanism ourselves?
I'm not familiar with multiboot spec, does it support arbitrary kernel size ?
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 24, 2008 4:29 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > On Jan 23, 2008 6:36 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> >> > static grub_uint32_t
> >> > grub_png_get_dword (struct
ck the size of the file here
>
> while (size--)
> {
> read pixels;
> }
>
>
png is compressed, it's not easy to separate the pixels.
> Or just check for grub_errno or so after reading the picture?
yes, after the picture is decoded, grub_errno is checked
On Jan 24, 2008 10:25 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
>
> On Thu, Jan 24, 2008 at 07:49:23PM +0800, Bean wrote:
> > On Jan 24, 2008 7:32 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> > > Sorry, my question was confusing; what I meant is
hanks for your help,
>
> (Looks like the first version of the email with the attached diskimage
> didn't make it, so here it is:
> http://people.videolan.org/~dionoea/disk-image.bz2 . It's only 57KB
> compressed)
i can see the problem here. grub_reiserfs_iterate_dir only enu
y manager or so). Does
> this work for you?
i think this is great, we don't have to worry about memdisk size, and
no need for the initrd hack anymore.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
&directory_item) != GRUB_ERR_NONE)
goto fail;
block_number = directory_item.block_number;
block_position = directory_item.block_position;
+ next_offset = directory_item.next_offset;
}
while (block_number);
--
Bean
__
.block_number;
block_position = directory_item.block_position;
+ next_offset = directory_item.next_offset;
}
while (block_number);
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
int argc, char **args)
+{
+ struct grub_video_bitmap *bitmap = 0;
+
+ if (argc != 1)
+return grub_error (GRUB_ERR_BAD_ARGUMENT, "file name required");
+
+ grub_video_reader_png (&bitmap, args[0]);
+ if (grub_errno != GRUB_ERR_NONE)
+return grub_errno;
+
+ grub_video_bitmap_destroy (bitmap);
+
+ return GRUB_ERR_NONE;
+}
+#endif
+
+static struct grub_video_bitmap_reader png_reader = {
+ .extension = ".png",
+ .reader = grub_video_reader_png,
+ .next = 0
+};
+
+GRUB_MOD_INIT (video_reader_png)
+{
+ grub_video_bitmap_reader_register (&png_reader);
+#if defined(PNG_DEBUG)
+ grub_register_command ("pngtest", grub_cmd_pngtest,
+GRUB_COMMAND_FLAG_BOTH, "pngtest FILE",
+"Tests loading of PNG bitmap.", 0);
+#endif
+}
+
+GRUB_MOD_FINI (video_reader_png)
+{
+#if defined(PNG_DEBUG)
+ grub_unregister_command ("pngtest");
+#endif
+ grub_video_bitmap_reader_unregister (&png_reader);
+}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
Committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
}
-
+
+ if (real_entry)
+entry = real_entry;
+
return grub_errno;
}
@@ -306,6 +322,8 @@ grub_multiboot (int argc, char *argv[])
if (! mbi)
goto fail;
+ grub_memset (mbi, 0, sizeof (struct grub_multiboot_info));
+
mbi->flags = MULTIBOOT_INFO_MEMORY;
/* Convert from bytes
NE_INSTALL_BSD_PART)
ljmp$(DATA_ADDR >> 4), $0
@@ -261,8 +255,8 @@ move_memory:
2:
- leal(%esi, %eax), %esi
- leal(%edi, %eax), %edi
+ addl%eax, %esi
+ addl%eax, %edi
subl%eax, %ecx
jnz 1b
--
Bean
On Jan 27, 2008 2:19 AM, walt <[EMAIL PROTECTED]> wrote:
>
> On Sat, 2008-01-26 at 18:24 +0100, Robert Millan wrote:
> > On Sat, Jan 26, 2008 at 11:01:49PM +0800, Bean wrote:
> > > this is the patch, problems found:
> > >
> > > 1, the mbi structure is
On Jan 27, 2008 2:24 AM, Bean <[EMAIL PROTECTED]> wrote:
> On Jan 27, 2008 2:19 AM, walt <[EMAIL PROTECTED]> wrote:
> >
> > On Sat, 2008-01-26 at 18:24 +0100, Robert Millan wrote:
> > > On Sat, Jan 26, 2008 at 11:01:49PM +0800, Bean wrote:
>
On Jan 27, 2008 5:24 AM, walt <[EMAIL PROTECTED]> wrote:
>
> On Sat, 2008-01-26 at 23:01 +0800, Bean wrote:
> > this is the patch, problems found:
> >
> > 1, the mbi structure is not initialized to all zeros, this means some
> > important member, like mods_c
ide of lzo decompression
code which ends at 0x6A2 (you can verify this by adding a text string
at the end of lzo1x.S). it's a very delicate situation. when you add
more code, it push the lzo decompression code further back, making
more code inside compress
grub_rescue_cmd_aout (argc, args);
+ return grub_errno;
+}
+
+GRUB_MOD_INIT (aout_normal)
+{
+ (void) mod; /* To stop warning. */
+ grub_register_command ("aout", grub_normal_aout_command,
+GRUB_COMMAND_FLAG_BOTH,
+"aout FILE [ARGS...]", "Load an a.out kernel.", 0);
+}
+
+GRUB_MOD_FINI (aout_normal)
+{
+ grub_unregister_command ("aout");
+}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 28, 2008 3:41 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Mon, Jan 28, 2008 at 02:53:14AM +0800, Bean wrote:
> > Hi,
> >
> > This patch add support for a.out kernel, which includes the 4th loader
> > of BSD system. For example, to start FreeBSD:
>
On Jan 28, 2008 4:06 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
>
> On Mon, Jan 28, 2008 at 03:55:19AM +0800, Bean wrote:
> > On Jan 28, 2008 3:41 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> > > On Mon, Jan 28, 2008 at 02:53:14AM +0800, Bean wrote:
> >
On Jan 28, 2008 4:13 AM, walt <[EMAIL PROTECTED]> wrote:
>
> On Sun, 2008-01-27 at 09:35 +0800, Bean wrote:
> > On Jan 27, 2008 5:24 AM, walt <[EMAIL PROTECTED]> wrote:
> > >
> > > Apparently "something bad" happens to the kernel in the proces
On Jan 27, 2008 2:15 AM, Bean <[EMAIL PROTECTED]> wrote:
> On Jan 27, 2008 1:21 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> > > FreeBSD:
> > >
> > > lnxboot.S: Assembler messages:
> > > lnxboot.S:49: Error: `0x200+data_start - data_next
On Jan 28, 2008 9:54 AM, walt <[EMAIL PROTECTED]> wrote:
>
> On Mon, 2008-01-28 at 04:25 +0800, Bean wrote:
> ...
> > > > please make a small ufs image containing the netbsd kernel, i don't a
> > > > a bsd system at hand
>
> http://leaf.dragonfly
#x27;m thinking maybe this has something to
> >> do with the size of the netbsd kernel?
> >
> > I just committed a check in grub-probe that attempts to read and verify
> > files
> > using GRUB filesystems and compares them with output from your system. E.g.
> &
leal(%esi, %eax), %esi
- leal(%edi, %eax), %edi
+ addl%eax, %esi
+ addl%eax, %edi
subl%eax, %ecx
jnz 1b
@@ -270,8 +268,11 @@ move_memory:
popw%dx
ret
-// Parameters:
-// si: message
+/*
+ * Parameters:
+ * si: message
+ */
+
On Jan 28, 2008 5:11 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Mon, Jan 28, 2008 at 04:10:42AM +0800, Bean wrote:
> > > > >
> > > > > How FreeBSD-specific is this? Will the same module apply to other
> > > > > a.out
> &g
On Jan 28, 2008 9:00 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > On Jan 28, 2008 9:54 AM, walt <[EMAIL PROTECTED]> wrote:
> >>
> >> On Mon, 2008-01-28 at 04:25 +0800, Bean wrote:
> >> ...
>
On Jan 28, 2008 10:37 PM, walt <[EMAIL PROTECTED]> wrote:
>
> On Mon, 2008-01-28 at 19:48 +0800, Bean wrote:
> > On Jan 28, 2008 5:11 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> > > On Mon, Jan 28, 2008 at 04:10:42AM +0800, Bean wrote:
> > > > >
cmd = CMD_CMP;
+ nparm = 2;
+ }
+ else if (!grub_strcmp (argv[optind], "hex"))
+ {
+ cmd = CMD_HEX;
+ }
+ else if (!grub_strcmp (argv[optind], "blocklist"))
+ {
+ cmd = CMD_BLOCKLIST;
+ }
+ else
+ {
+ fprintf (stderr, "Invalid command %s.\n", argv[optind]);
+ usage (1);
+ }
+
+ if (optind + 1 + nparm > argc)
+ {
+ fprintf (stderr, "Invalid parameter for command %s.\n",
+ argv[optind]);
+ usage (1);
+ }
+
+ optind++;
+}
+ else
+{
+ fprintf (stderr, "No command is specified.\n");
+ usage (1);
+}
+
+ grub_hostfs_init ();
+
+ /* Initialize all modules. */
+ grub_init_all ();
+
+ if (is_raw)
+grub_env_set ("filehook", "0");
+
+ /* Do it. */
+ fstest (image_path + 1, cmd, argc - optind, argv + optind);
+
+ /* Free resources. */
+ grub_fini_all ();
+
+ grub_hostfs_fini ();
+
+ return 0;
+}
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 29, 2008 1:27 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Mon, Jan 28, 2008 at 10:48:13PM +0800, Bean wrote:
> > +#include
> > +#include
> > +#include
> > +#include
> > +
> > +#define _GNU_SOURCE 1
> > +#include
>
> I s
On Jan 29, 2008 6:04 AM, walt <[EMAIL PROTECTED]> wrote:
> Bean wrote:
> > On Jan 28, 2008 10:37 PM, walt<[EMAIL PROTECTED]> wrote:
> ...
> >> Good news and bad news. With this patch and your most recent ufs patch
> >> I can aout /boot/loader or mu
linked to run at 0xc010.
>
> This is trivial to fix, but the trivial fix may not be the right fix.
>
> Now if only I was having more luck when grub2 was loaded from coreboot.
> But that's another thread maybe.
my previous multiboot patch fix this problem.
--
Be
On Jan 29, 2008 4:48 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > On Jan 28, 2008 5:11 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> >> On Mon, Jan 28, 2008 at 04:10:42AM +0800, Bean wrote:
> >> > > &g
using the ata driver would be a better approach. When our driver
> > > doesn't "taint" bios-based CD access, we could merge the eltorito loader
> > > from
> > > GSoC codebase, without having to solve the problems in biosdisk CD access.
> >
>
On Jan 29, 2008 5:10 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > add two sanity check, one for chuck size, make sure the on disk
> > structure is correct, one for raw image, make sure the image won't
> > overflo
Some changes in this new patch:
Remove the _GNU_SOURCE line.
Partition number is now string, so that you can use bsd sub partition.
Code cleanup for function fstest.
2008-01-29 Bean <[EMAIL PROTECTED]>
* Makefile.in (enable_grub_fstest): New variable.
* conf/comm
On Jan 29, 2008 5:14 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> > On Jan 29, 2008 4:48 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> >> Bean <[EMAIL PROTECTED]> writes:
> >>
> >> >
On Jan 29, 2008 9:45 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
>
> On Tue, Jan 29, 2008 at 06:18:35PM +0800, Bean wrote:
> > --- a/conf/i386-pc.rmk
> > +++ b/conf/i386-pc.rmk
> > @@ -61,6 +61,9 @@ sbin_UTILITIES = grub-setup grub-mkdevicemap grub-probe
>
uraging. If unifont has some issues how can we use
> other fonts? What is the *.pff extension? This again may be a repost
> hence apologies if they are.
> --
> Regards,
> Shirish Agarwal
> This email is licensed under http://creativecommons.org/li
> I don't like that you hardcode 0x1badb002. Could you use the macro from
> > multiboot.h ?
> >
> > Also for the comments, I think /* */ is preferred (at least, it is
> > consistent
> > with the rest of GRUB).
>
> And do not forget "." :-)
Fixed and committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 30, 2008 2:11 AM, <[EMAIL PROTECTED]> wrote:
>
> On Tue, Jan 29, 2008 at 03:20:12PM +0800, Bean wrote:
> > On Jan 29, 2008 3:11 PM, <[EMAIL PROTECTED]> wrote:
> > > On Mon, Jan 28, 2008 at 06:37:16AM -0800, walt wrote:
> > > >
> > >
/i386/pc/init.c b/kern/i386/pc/init.c
index acaf20b..1a4e9df 100644
--- a/kern/i386/pc/init.c
+++ b/kern/i386/pc/init.c
@@ -71,9 +71,12 @@ make_install_device (void)
}
else if (grub_install_dos_part != -2)
{
- grub_sprintf (dev, "(%cd%u",
- (grub_boot_drive &
ed to from the bug
> > page above.
>
> Available at: http://stash.poggs.com/grub-xfs-example.bz2
>
> It can be reproduced with qemu at least. I gave it a try but get lost too
> easily on filesystem stuff. Anyone feels like having a look? Bean what do
> you think? :-)
when
t ino, char
*filename)
{
struct grub_fshelp_node *fdiro;
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 30, 2008 8:08 PM, Marco Gerards <[EMAIL PROTECTED]> wrote:
>
> Bean <[EMAIL PROTECTED]> writes:
>
> > I figure it out, the problem is caused by nested function:
> >
> > int call_hook (grub_uint64_t ino, char *filename)
> >
> > it would take
t
> recent ufs and aout patches applied.
in this case, the ufs driver should be ok.
btw, do you remember to apply the multiboot patch ? you can also make
a qemu image for me to test.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
44
--- a/kern/i386/pc/init.c
+++ b/kern/i386/pc/init.c
@@ -71,9 +71,12 @@ make_install_device (void)
}
else if (grub_install_dos_part != -2)
{
- grub_sprintf (dev, "(%cd%u",
- (grub_boot_drive & 0x80) ? 'h' : 'f',
-
Changes in this new version:
Fix a bug in lnxboot.
Use macro for magic values.
Try 3 times when reading from cdrom.
2008-01-31 Bean <[EMAIL PROTECTED]>
* boot/i396/pc/lnxboot.S (data_start): Add cdrom boot support.
* disk/i386/pc/biosdisk.c (cd_start): New va
Hi,
ok, this is the new patch, the cdboot function is implemented in
separate file cdboot.S. To create a eltorito boot file, use:
cat cdboot.img core.img > grub2cd.bin
2008-01-31 Bean <[EMAIL PROTECTED]>
* conf/i386-pc.rmk (pkglib_IMAGES): Add cdboot.img.
(cdboot_im
sed by eltorito boot?
>
> I meant `code' of course.
the error report routine is shared, other code is separate.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
Hi,
I add btree support for the xfs driver, and fix a few bugs. Now i' m
able to list a directory of over 5000 files, please test it.
2008-02-01 Bean <[EMAIL PROTECTED]>
* fs/xfs.h (grub_xfs_sblock): New member log2_dirblk.
(grub_xfs_btree_node): Ne
On Feb 1, 2008 9:46 AM, walt <[EMAIL PROTECTED]> wrote:
>
> Bean wrote:
> > On Jan 30, 2008 8:24 PM, walt<[EMAIL PROTECTED]> wrote:
> >> Sorry, I gave you incomplete information. I can use 'multiboot' to load
> >> the netbsd from any
On Feb 1, 2008 3:57 AM, Marco Gerards <[EMAIL PROTECTED]> wrote:
> Bean <[EMAIL PROTECTED]> writes:
>
> Hi Bean!
>
> > I add btree support for the xfs driver, and fix a few bugs. Now i' m
> > able to list a directory of over 5000 files, please test it.
>
Hi,
change in this new patch:
Fix small bug in xfs
Fix a bug in fshelp
Define NESTED_FUNC_ATTR as regparm(1) and us it for call_hook.
With this patch, i'm able to load a highly sparse file about 200M.
2008-02-01 Bean <[EMAIL PROTECTED]>
* fs/xfs.c (grub_xfs_s
Hi,
This patch add joliet extension support for iso9660.
2008-02-02 Bean <[EMAIL PROTECTED]>
* fs/iso9660.c (GRUB_ISO9660_VOLDESC_BOOT): New macro.
(GRUB_ISO9660_VOLDESC_PRIMARY): Likewise.
(GRUB_ISO9660_VOLDESC_SUPP): Likewise.
(GRUB_ISO9660_VOLDES
Committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
Committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Jan 31, 2008 8:43 PM, Bean <[EMAIL PROTECTED]> wrote:
> 2008-01-31 Bean <[EMAIL PROTECTED]>
>
> * conf/i386-pc.rmk (pkglib_IMAGES): Add cdboot.img.
> (cdboot_img_SOURCES): New variable.
> (cdboot_img_ASFLAGS): New variable.
> (cdb
Hi,
i can see that there are linux specific code in util/biosdisk.c,
perhaps we should move them to a separate file such as util/osdep.c ?
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Feb 3, 2008 12:33 AM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Sat, Feb 02, 2008 at 10:19:55PM +0800, Bean wrote:
> >
> > any objection for this patch ? i would like to commit it soon.
>
> Looks fine to me. Just a minor comment:
>
> > +#define GR
non-existant-file
> > > echo this point should never be reached
> > > read
> > >
> > > You'll see that the echo is reached, even if cat returned an error.
> > >
> > > This becomes much worse when found in situations like:
> >
Committed.
--
Bean
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
On Feb 3, 2008 6:55 PM, Robert Millan <[EMAIL PROTECTED]> wrote:
> On Sun, Feb 03, 2008 at 11:05:26AM +0800, Bean wrote:
> > > > > Try the following grub.cfg:
> > > > >
> > > > > cat (memdisk)/non-existant-file
> > >
Hi,
Some change in this version:
Move grub-fstest.c_DEPENDENCIES and grub_fstest_SOURCES to common.rmk.
Add new option -d for grub-fstest, it's used to set the debug
environment variable.
This could be the final version, if nobody objects, i will check it in.
2008-02-03 Bean <[EMAIL P
1 - 100 of 1124 matches
Mail list logo
