[PATCH] Enable grub_cpu_idle for i386 to halt the CPU

2008-08-11 Thread David Fries
Enable grub_cpu_idle for i386 to halt the CPU and modify the menu code to make use of it. This will save power when booting. Or maybe I should say it will keep the CPU from running so hot when the timer is counting down. It isn't safe to call halt in protected mode as interrupts are disabled. I

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread Bean
On Tue, Aug 12, 2008 at 10:02 AM, y.volta <[EMAIL PROTECTED]> wrote: > i just conveniently make this kind of archive be supported in grub2 - since > there is no other addtional work required. > > this is a little patch, so if there are mutli-files in this archive, we will > failed to decompress it.

Re: Is romfs supported in Grub2

2008-08-11 Thread Bean
On Tue, Aug 12, 2008 at 10:07 AM, y.volta <[EMAIL PROTECTED]> wrote: > romfs is useful for me, so is there any existing mod or functions existed in > current Grub2? > > if no, i'd like to add this mod to Grub2 from previous Grub0.97 patch. > > BTW. bean, did you have any plan to translate some grub

Idea: implementation of the password command

2008-08-11 Thread Bean
Hi, First of all, we can store the encrypted password in a variable, like PASSWORD. It's better to store it in grubenv. as it is more flexible, and also allows the user to change it from the grub environment. Then, we can add a password hook to normal.mod. Whenever we need password verification,

Is romfs supported in Grub2

2008-08-11 Thread y.volta
romfs is useful for me, so is there any existing mod or functions existed in current Grub2? if no, i'd like to add this mod to Grub2 from previous Grub0.97 patch. BTW. bean, did you have any plan to translate some grub4dos' functions to grub2 in recentlly? y.volta -

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread y.volta
i just conveniently make this kind of archive be supported in grub2 - since there is no other addtional work required. this is a little patch, so if there are mutli-files in this archive, we will failed to decompress it. As I know, there are many GUI tools(e.g. 7-Zip/Winrar/Winzip) can make the

Idea: Use extended blocklist format to support installing to raid device.

2008-08-11 Thread Bean
Hi, Currently, the blocklist format can only support traditional disk. I'm thinking about a new format that can be used in raid as well. The new format is like this: checksum_0 blocklist_0 checksum_1 blocklist_1 ... The checksum is the checksum of sectors in the blocklist that follows. For exam

Re: Idea: use menu hook to implement the savedefault command

2008-08-11 Thread Bean
On Tue, Aug 12, 2008 at 5:32 AM, Robert Millan <[EMAIL PROTECTED]> wrote: > On Tue, Aug 12, 2008 at 05:03:10AM +0800, Bean wrote: >> Hi, >> >> Now it's possible to implement savedefault with load_env and save_env, >> but the problem is we need to add it to every menuitem, it's tedious >> process, a

Re: [PATCH] decouple mmap parsing by using grub_available_iterate()

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 11:24:18PM +0200, Robert Millan wrote: > On Mon, Aug 11, 2008 at 06:39:17PM +0300, Vesa Jääskeläinen wrote: > > Robert Millan wrote: > > >Hi, > > > > > >This patch decouples memory map parsing into a separate > > >grub_available_iterate() > > >function, for i386-pc and i386

Re: [PATCH] Grub2 cannot find LVM volume groups with a dash (-) in the name

2008-08-11 Thread Sander Marechal
Felix Zielcke wrote: > Well attached is now my cute little asprintf patch with a very ugly LVM > part. > > It works for > > volumegroup: linuxvg > logicalvolume: lvol0 > /dev/mapper/linuxvg-lvol0 > > volume group: m-y--vg > logical volume: l-vol--0 > /dev/mapper/m--yvg-l--vol0 > > > Pl

Re: [PATCH] just skip over invalid/unsupported RAID

2008-08-11 Thread Felix Zielcke
Am Montag, den 11.08.2008, 23:22 +0200 schrieb Robert Millan: > > IIRC, the difference is that with the invalid BSD partitions, rising errors > is ineffective as there can't be anyone to handle them (since we're inside > an iteration). Ah, yes that's a `little' difference between that and the RAI

Re: Idea: use menu hook to implement the savedefault command

2008-08-11 Thread Robert Millan
On Tue, Aug 12, 2008 at 05:03:10AM +0800, Bean wrote: > Hi, > > Now it's possible to implement savedefault with load_env and save_env, > but the problem is we need to add it to every menuitem, it's tedious > process, and new item don't get it automatically. After discussing this on IRC, I just th

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 07:22:53PM +0200, Marco Gerards wrote: > Hi, > > "y.volta" <[EMAIL PROTECTED]> writes: > > > i've made a little patch for pkziped ( deflate mode ) file. by > > using this, we can have both gunzip and pkunzip supported: they have > > only header different with the same

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 11:41:38PM +0800, y.volta wrote: > hi, > > i've made a little patch for pkziped ( deflate mode ) file. by using > this, we can have both gunzip and pkunzip supported: they have only header > different with the same file. ;-) > > please note, if your file is too s

Re: [PATCH] decouple mmap parsing by using grub_available_iterate()

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 06:39:17PM +0300, Vesa Jääskeläinen wrote: > Robert Millan wrote: > >Hi, > > > >This patch decouples memory map parsing into a separate > >grub_available_iterate() > >function, for i386-pc and i386-coreboot. It is mostly intended as a > >cleanup. > >Makes the code more mo

Re: [PATCH] just skip over invalid/unsupported RAID

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 06:50:18PM +0200, Felix Zielcke wrote: > 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.

Re: [PATCH] New command checktime

2008-08-11 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: > On Mon, Aug 11, 2008 at 2:04 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: >> Hi, >> >> Bean <[EMAIL PROTECTED]> writes: >> >>> I implement this command for grub4dos some time ago, now it's ported to >>> grub2. >>> >>> Usage: >>> >>> checktime min hour day_of_m

Idea: use menu hook to implement the savedefault command

2008-08-11 Thread Bean
Hi, Now it's possible to implement savedefault with load_env and save_env, but the problem is we need to add it to every menuitem, it's tedious process, and new item don't get it automatically. I'm thinking about using menu hook to solve this. I can think of two implementation: 1. Function inter

Idea: use menu hook to implement the savedefault command

2008-08-11 Thread Bean
Hi, Now it's possible to implement savedefault with load_env and save_env, but the problem is we need to add it to every menuitem, it's tedious process, and new item don't get it automatically. I'm thinking about using menu hook to solve this. I can think of two implementation: 1. Function inter

Re: [PATCH] Split of raid scan code

2008-08-11 Thread Robert Millan
On Tue, Aug 12, 2008 at 04:24:26AM +0800, Bean wrote: > - disk/lvm.c disk/raid.c grub_probe_init.c > + disk/lvm.c disk/raid.c disk/mdraid_linux.c grub_probe_init.c Hi, I haven't tried, but I think this module split would break grub-install & update-grub. Please check those out before com

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread Gregg Levine
On Mon, Aug 11, 2008 at 1:22 PM, Marco Gerards <[EMAIL PROTECTED]> wrote: > Hi, > > "y.volta" <[EMAIL PROTECTED]> writes: > >> i've made a little patch for pkziped ( deflate mode ) file. by >> using this, we can have both gunzip and pkunzip supported: they have >> only header different with the

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread Marco Gerards
Hi, "y.volta" <[EMAIL PROTECTED]> writes: > i've made a little patch for pkziped ( deflate mode ) file. by > using this, we can have both gunzip and pkunzip supported: they have > only header different with the same file. ;-) > > please note, if your file is too small, pkzip will not comp

Re: [PATCH] add pkzip support for gzio

2008-08-11 Thread Colin D Bennett
On Mon, 11 Aug 2008 23:41:38 +0800 "y.volta" <[EMAIL PROTECTED]> wrote: > hi, > > i've made a little patch for pkziped ( deflate mode ) file. by > using this, we can have both gunzip and pkunzip supported: they have > only header different with the same file. ;-) > > please note, if your

Re: [PATCH] just skip over invalid/unsupported RAID

2008-08-11 Thread Felix Zielcke
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

[PATCH] Grub2 cannot find LVM volume groups with a dash (-) in the name

2008-08-11 Thread Felix Zielcke
I still can't see any use in this `escape a dash with a dash because we use a dash to seperate the vg part from the lv one' from LVM. As I already wrote, grub-probe works fine if you remove that double escaping from the file name and remount it. LVM (lv* vg* commands) shouldn't use the filename to

[PATCH] add pkzip support for gzio

2008-08-11 Thread y.volta
hi, i've made a little patch for pkziped ( deflate mode ) file. by using this, we can have both gunzip and pkunzip supported: they have only header different with the same file. ;-) please note, if your file is too small, pkzip will not compress it but store the orginal data to the out

Re: [PATCH] decouple mmap parsing by using grub_available_iterate()

2008-08-11 Thread Vesa Jääskeläinen
Robert Millan wrote: Hi, This patch decouples memory map parsing into a separate grub_available_iterate() function, for i386-pc and i386-coreboot. It is mostly intended as a cleanup. Makes the code more modular so that, for example, the multiboot loader can construct a memory map without having

[PATCH] decouple mmap parsing by using grub_available_iterate()

2008-08-11 Thread Robert Millan
Hi, This patch decouples memory map parsing into a separate grub_available_iterate() function, for i386-pc and i386-coreboot. It is mostly intended as a cleanup. Makes the code more modular so that, for example, the multiboot loader can construct a memory map without having specific knowledge of

Re: grub2 disk error with kvm

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 02:08:58PM +0200, Daniel Dehennin wrote: > Hello, > > I post a bug report here as asked by nyu on #grub. > > === IRC LOG === > [13:32] nebuchadnezzar: could you report the grub one first? the > problem > is grub_biosdisk_get_diskinfo_standard() fails, but this call

Re: [PATCH] just skip over invalid/unsupported RAID

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 01:42:16PM +0200, Felix Zielcke wrote: > GRUB shouldn't fail to rescue mode, if an unsupport RAID (in this case > 6) is used on disks which GRUB doestn't even need to access. I think the right solution for this would be to handle the errors in the upper layer instead of rem

Re: [PATCH] New command checktime

2008-08-11 Thread Robert Millan
On Mon, Aug 11, 2008 at 07:06:11PM +0800, Bean wrote: > --- /dev/null > +++ b/commands/datetime.c > @@ -0,0 +1,132 @@ > +/* datetime.c - Module to export datetime variables. */ Shouldn't this be somewhere else? (other than commands/) -- Robert Millan The DRM opt-in fallacy: "Your data belon

Re: grub-probe detects ext4 wronly as ext2

2008-08-11 Thread Javier Martín
Hi there, After reading Felix's reply I've once again found your post and implemented your request, so here is a new version of the patch ("version 6"). Sorry for missing your message in the first instance... u_u 2008/7/19 Robert Millan <[EMAIL PROTECTED]>: > On Sat, Jul 05, 2008 at 08:36:13PM +0

grub2 disk error with kvm

2008-08-11 Thread Daniel Dehennin
Hello, I post a bug report here as asked by nyu on #grub. === IRC LOG === [13:32] nebuchadnezzar: could you report the grub one first? the problem is grub_biosdisk_get_diskinfo_standard() fails, but this call wasn't used when probing for drives (and I think it should be) === IRC LOG ===

[PATCH] just skip over invalid/unsupported RAID

2008-08-11 Thread Felix Zielcke
I said this already on Bean's RAID topic, but I think I just make now a new topic instead of messing him's up. So here's again a patch which changes grub_error into grub_dprintf, it's the same reason as with these invalid BSD disklabels/partitions See [0] for a good reason. GRUB shouldn't fail to

Re: [PATCH] New command checktime

2008-08-11 Thread Bean
On Mon, Aug 11, 2008 at 2:04 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: > Hi, > > Bean <[EMAIL PROTECTED]> writes: > >> I implement this command for grub4dos some time ago, now it's ported to >> grub2. >> >> Usage: >> >> checktime min hour day_of_month month day_of_week > > Personally I would

Re: grub-probe detects ext4 wronly as ext2

2008-08-11 Thread Felix Zielcke
Am Montag, den 11.08.2008, 02:35 +0200 schrieb Javier Martín: > El mié, 06-08-2008 a las 12:36 +0200, Felix Zielcke escribió: > > [0] http://lists.gnu.org/archive/html/grub-devel/2008-07/msg8.html > > [1] http://lists.gnu.org/archive/html/grub-devel/2008-07/msg00333.html > > > > > > Thanks f