Re: [PATCH] bless command

2009-06-21 Thread Robert Millan
On Fri, Jun 19, 2009 at 01:53:18PM +0200, Vladimir 'phcoder' Serbinenko wrote: > On Fri, Jun 19, 2009 at 8:08 AM, Peter Cros wrote: > > > Hi again, > > > > Could the hfspbless patch be updated and hopefully committted to provide > > the hfspbless module as standard in grub2. > > > Attached to th

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Fri, Jun 19, 2009 at 12:42:26PM +0200, Vladimir 'phcoder' Serbinenko wrote: > On Fri, Jun 19, 2009 at 12:37 PM, Robert Millan wrote: > > > On Thu, Jun 18, 2009 at 09:56:07PM +0100, Carles Pina i Estany wrote: > > > > > -Copy ca.mo to /usr/share/locale/ca/LC_MESSAGES/grub.mo > > > > > >

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Sat, Jun 20, 2009 at 12:08:44AM +0100, Carles Pina i Estany wrote: > > a) I afraid that having the .mo files in /usr/share/... makes Grub > quite much tight to GNU/Linux. In my opinion, boot loaders should be as > much as operating system agnostic as could be. Sounds like a good point. I don

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Fri, Jun 19, 2009 at 04:32:35PM +0200, Vladimir 'phcoder' Serbinenko wrote: > > - lose, and get the C locale messages > > - try to set it up using the correct grub device that contains the > > locales. > > What if partition is in btrfs? Well, now I think using $prefix/locale is not that bad.

Re: [PATCH] Support GRUB_DISABLE_LINUX_RECOVERY

2009-06-21 Thread Robert Millan
Committed, as this is small enough not to be copyright-significant. Btw, Colin please use inline attachments in the future. It makes it easier to manage. On Wed, Jun 17, 2009 at 12:23:23PM +0100, Colin Watson wrote: > This patch fixes > https://bugs.launchpad.net/ubuntu/+source/grub2/+bug/19020

Re: Proposal for GNU indent compatibility

2009-06-21 Thread Robert Millan
On Fri, Jun 19, 2009 at 12:37:20PM -0400, Pavel Roskin wrote: > Hello! > > In order to encourage developers to use proper indentation on the newly > added code, I suggest that we accept the default style of GNU indent. > That is, the space after "!" should not be needed. That's the only > differe

[PATCH] File access library for lua

2009-06-21 Thread Bean
Hi, I've added a few function in the lua grub library to access files: enum_device - enumerate devices enum_file - enumerate files file_open - open file file_close - close file file_seek - seek file file_read - read n bytes file_getline - read a line file_getsize - get file size file_getpos - get

Re: [PATCH] fix building bug with --enable-grub-emu-usb

2009-06-21 Thread Robert Millan
On Fri, Jun 19, 2009 at 11:46:06AM -0400, Pavel Roskin wrote: > > Generally, please think how to reduce complexity, not how to add to it. > Complex systems are harder to maintain. In general, I completely agree with this. However grub-emu has very special requirements that could make it desireab

Re: Proposal for GNU indent compatibility

2009-06-21 Thread Alfred M. Szmidt
> In order to encourage developers to use proper indentation on the > newly added code, I suggest that we accept the default style of > GNU indent. That is, the space after "!" should not be needed. > That's the only difference I'm aware of. What does GCS say about spaces after "!"

Re: [PATCH] fix building bug with --enable-grub-emu-usb

2009-06-21 Thread Vladimir 'phcoder' Serbinenko
> Anyway, given that grub-emu is a debugging/development tool, I'm not sure > it's that useful to export those flags to autoconf. We could just live > with #ifdefs in the code IMHO. Distributors are going to ship whatever we > choose as default for grub-emu anyway. > Actually the problem isn't gr

Re: [PATCH] load FreeBSD ELF modules

2009-06-21 Thread Vladimir 'phcoder' Serbinenko
Committed On Sat, Jun 13, 2009 at 12:23 PM, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > Hello > > On Fri, Jun 12, 2009 at 8:14 PM, Vladimir 'phcoder' > Serbinenko wrote: > > Hello. Unlike some other types of modules (mfsroot, splash, ..) > > FreeBSD relies on ELF modules being par

Re: Fw: gettext support

2009-06-21 Thread Carles Pina i Estany
Hello, On Jun/21/2009, Robert Millan wrote: > On Fri, Jun 19, 2009 at 12:42:26PM +0200, Vladimir 'phcoder' Serbinenko wrote: > > On Fri, Jun 19, 2009 at 12:37 PM, Robert Millan wrote: > > > > > On Thu, Jun 18, 2009 at 09:56:07PM +0100, Carles Pina i Estany wrote: > > > > > > -Copy ca.mo

[PATCH] i386-qemu port

2009-06-21 Thread Robert Millan
Hi, I ported GRUB to QEMU. It's mostly based on the coreboot port, with the main difference being that we include code to transition from i8086 mode, an i386 firmware entry point and produce raw code images instead of ELF. The result is you can do e.g. $ sudo grub-mkrawimage at_keyboard norm

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 06:22:36PM +0100, Carles Pina i Estany wrote: > > > It's not a problem. We need two locations anyway: one where make install > > > copies them and the other one where ./grub-install copies them. Would it > > > be > > > possible to use /usr/share/grub/locale/$LANG.mo rather

does module area require alignment? (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 08:17:48PM +0200, Robert Millan wrote: > grub_addr_t > grub_arch_modules_addr (void) > { > - return ALIGN_UP((grub_addr_t) _end, GRUB_MOD_ALIGN); > +// return ALIGN_UP((grub_addr_t) _end, GRUB_MOD_ALIGN); > + return _end; > } Some comments on this one. It seems core

[PATCH] move grub_stop() (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
Move grub_stop to init.c to ease code sharing with i386-qemu. -- Robert Millan The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and how) you may access your data; but nobody's threatening your freedom: we still allow you to remove your data and not access it at all."

Re: [PATCH] i386-qemu port

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 20:17 +0200, Robert Millan wrote: > Hi, > > I ported GRUB to QEMU. It's mostly based on the coreboot port, with the > main difference being that we include code to transition from i8086 mode, > an i386 firmware entry point and produce raw code images instead of ELF. That's

Re: [PATCH] move grub_stop() (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 20:54 +0200, Robert Millan wrote: > Move grub_stop to init.c to ease code sharing with i386-qemu. That's not quite a movement. grub_cpu_idle() does nothing. I think we need to have several implementations of grub_stop: hard halt with interrupts disabled, exit from qemu, exi

Re: does module area require alignment? (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 20:50 +0200, Robert Millan wrote: > Does anyone know why do we align ELF targets? When I did the coreboot port, > the ELF part was based on existing Ieee1275 code, so I guess I just mimicked > it. Is there some issue with non-i386 CPUs or with some Ieee1275 > implementation

Re: Fw: gettext support

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 20:42 +0200, Robert Millan wrote: > On Sun, Jun 21, 2009 at 06:22:36PM +0100, Carles Pina i Estany wrote: > > > > It's not a problem. We need two locations anyway: one where make install > > > > copies them and the other one where ./grub-install copies them. Would > > > > it

[PATCH] rename kernel.elf to kernel.img (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
If you check my earlier patch, you'll see i386-qemu.rmk is just a stub that includes i386-coreboot.rmk. This is to reduce code duplication (untill we have a more flexible build system). This patch renames kernel.elf to kernel.img in ELF platforms, so that more variables / etc can be shared with

Re: [PATCH] move grub_stop() (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 03:05:56PM -0400, Pavel Roskin wrote: > On Sun, 2009-06-21 at 20:54 +0200, Robert Millan wrote: > > Move grub_stop to init.c to ease code sharing with i386-qemu. > > That's not quite a movement. grub_cpu_idle() does nothing. Well, the major problem with grub_cpu_idle() do

Re: [PATCH] i386-qemu port

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 03:00:42PM -0400, Pavel Roskin wrote: > > The result is you can do e.g. > > > > $ sudo grub-mkrawimage at_keyboard normal etc -o /usr/share/qemu/grub > > Please consider if we can call it grub-mkimage unless grub-mkimage would > make sense as a separate executable for th

Re: does module area require alignment? (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 03:08:19PM -0400, Pavel Roskin wrote: > On Sun, 2009-06-21 at 20:50 +0200, Robert Millan wrote: > > > Does anyone know why do we align ELF targets? When I did the coreboot port, > > the ELF part was based on existing Ieee1275 code, so I guess I just mimicked > > it. Is th

[PATCH] swap real_to_prot() and prot_to_real() (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
When doing the i386-coreboot port I made this choice completely backwards. I thought real_to_prot() was only useful on i386-pc, because we needed it for returning from BIOS, and prot_to_real() was useful elsewhere, because the Linux loader would use it. Turns out we need real_to_prot() on i386-q

Re: [PATCH] i386-qemu port

2009-06-21 Thread Robert Millan
New patch, after a bunch of misc cleanup, turning hardcoded numbers into macros, improving comments, etc. -- Robert Millan The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and how) you may access your data; but nobody's threatening your freedom: we still allow you to

Re: [PATCH] i386-qemu port

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 03:00:42PM -0400, Pavel Roskin wrote: > On Sun, 2009-06-21 at 20:17 +0200, Robert Millan wrote: > > Hi, > > > > I ported GRUB to QEMU. It's mostly based on the coreboot port, with the > > main difference being that we include code to transition from i8086 mode, > > an i386

Re: [PATCH] i386-qemu port

2009-06-21 Thread Vladimir 'phcoder' Serbinenko
On Sun, Jun 21, 2009 at 10:34 PM, Robert Millan wrote: > On Sun, Jun 21, 2009 at 03:00:42PM -0400, Pavel Roskin wrote: > > On Sun, 2009-06-21 at 20:17 +0200, Robert Millan wrote: > > > Hi, > > > > > > I ported GRUB to QEMU. It's mostly based on the coreboot port, with > the > > > main difference

Re: Fw: gettext support

2009-06-21 Thread Carles Pina i Estany
Hola, On Jun/21/2009, Pavel Roskin wrote: > On Sun, 2009-06-21 at 20:42 +0200, Robert Millan wrote: > > On Sun, Jun 21, 2009 at 06:22:36PM +0100, Carles Pina i Estany wrote: > > > > > It's not a problem. We need two locations anyway: one where make > > > > > install > > > > > copies them and the

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 03:14:59PM -0400, Pavel Roskin wrote: > > Do we really need to localize the bootloader? I think localization of > the tools should be sufficient. > > Even Linux kernel is not localized, and it's more likely to tell the > user something important than the bootloader. Besi

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 10:09:14PM +0100, Carles Pina i Estany wrote: > > > > Do we really need to localize the bootloader? I think localization of > > the tools should be sufficient. > > By the moment I translated this string (as an example): > +" Use the %C and %C keys to select which ent

Re: Fw: gettext support

2009-06-21 Thread Vladimir 'phcoder' Serbinenko
On Sun, Jun 21, 2009 at 11:09 PM, Carles Pina i Estany wrote: > Well, we could have another approach and this is also nice: > > _all_ strings that appears in a normal boot process (like the ones that > I showed above) are part of the grub.cfg file. So, when grub.cfg file is > generated we could ha

Re: Fw: gettext support

2009-06-21 Thread Carles Pina i Estany
Hi, On Jun/21/2009, Robert Millan wrote: > Sounds like an interesting option. It's a pity you already spent the effort > on implementing gettext in grub side, though :-( Yes it's pity, but I will not force to use what I implemented if now we think something better. So we can discuss that I wil

Re: Fw: gettext support

2009-06-21 Thread Vladimir 'phcoder' Serbinenko
On Sun, Jun 21, 2009 at 11:28 PM, Carles Pina i Estany wrote: > > Hi, > > On Jun/21/2009, Robert Millan wrote: > > > Sounds like an interesting option. It's a pity you already spent the > effort > > on implementing gettext in grub side, though :-( > > Yes it's pity, but I will not force to use wh

Re: Fw: gettext support

2009-06-21 Thread Carles Pina i Estany
Hi, On Jun/22/2009, Vladimir 'phcoder' Serbinenko wrote: > On Sun, Jun 21, 2009 at 11:28 PM, Carles Pina i Estany wrote: > > > > > Hi, > > > > On Jun/21/2009, Robert Millan wrote: > > > > > Sounds like an interesting option. It's a pity you already spent the > > effort > > > on implementing get

Re: Fw: gettext support

2009-06-21 Thread Robert Millan
On Sun, Jun 21, 2009 at 11:31:57PM +0100, Carles Pina i Estany wrote: > On Jun/22/2009, Vladimir 'phcoder' Serbinenko wrote: > > On Sun, Jun 21, 2009 at 11:28 PM, Carles Pina i Estany > > wrote: > > Actually I think it would be a good move to provide translations of e.g. > > error strings, output

[PATCH] access gdtdesc on segment 0 unconditionally (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Robert Millan
In this line of code in real_to_prot(): DATA32 ADDR32 lgdt%cs:gdtdesc GAS generates an absolute address for `gdtdesc' (not relative to segment), and so for the code to work %cs must be zero. In current usage of real_to_prot(), %cs is always zero because we jump to 0x0:0x82xx early

Re: Fw: gettext support

2009-06-21 Thread Colin Watson
On Mon, Jun 22, 2009 at 12:47:15AM +0200, Robert Millan wrote: > On Sun, Jun 21, 2009 at 11:31:57PM +0100, Carles Pina i Estany wrote: > > On Jun/22/2009, Vladimir 'phcoder' Serbinenko wrote: > > > On Sun, Jun 21, 2009 at 11:28 PM, Carles Pina i Estany > > > wrote: > > > Actually I think it would

Re: Fw: gettext support

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 22:09 +0100, Carles Pina i Estany wrote: > > Do we really need to localize the bootloader? I think localization of > > the tools should be sufficient. > > By the moment I translated this string (as an example): > +" Use the %C and %C keys to select which entry is highl

Re: [PATCH] File access library for lua

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 19:33 +0800, Bean wrote: > I've added a few function in the lua grub library to access files: ... > I also enable the string library from lua. The patch doesn't introduce any compiler warnings, and that's good. It makes lua.mod bigger by about 10%, but it's very little comp

Re: [PATCH] access gdtdesc on segment 0 unconditionally (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Pavel Roskin
On Mon, 2009-06-22 at 00:53 +0200, Robert Millan wrote: > In this line of code in real_to_prot(): > > DATA32 ADDR32 lgdt%cs:gdtdesc > > GAS generates an absolute address for `gdtdesc' (not relative to segment), > and so for the code to work %cs must be zero. In current usage of > rea

Re: [PATCH] i386-qemu port

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 22:22 +0200, Robert Millan wrote: > New patch, after a bunch of misc cleanup, turning hardcoded numbers into > macros, improving comments, etc. kernel_img_FORMAT is defined but never used. GRUB_MEMORY_MACHINE_LINK_ADDR doesn't need to be conditional, it's the same for corebo

Re: [PATCH] swap real_to_prot() and prot_to_real() (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 21:52 +0200, Robert Millan wrote: > When doing the i386-coreboot port I made this choice completely backwards. > > I thought real_to_prot() was only useful on i386-pc, because we needed it > for returning from BIOS, and prot_to_real() was useful elsewhere, because > the Linux

Re: [PATCH] move grub_stop() (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 21:25 +0200, Robert Millan wrote: > On Sun, Jun 21, 2009 at 03:05:56PM -0400, Pavel Roskin wrote: > > On Sun, 2009-06-21 at 20:54 +0200, Robert Millan wrote: > > > Move grub_stop to init.c to ease code sharing with i386-qemu. > > > > That's not quite a movement. grub_cpu_idl

Re: [PATCH] rename kernel.elf to kernel.img (Re: [PATCH] i386-qemu port)

2009-06-21 Thread Pavel Roskin
On Sun, 2009-06-21 at 21:19 +0200, Robert Millan wrote: > If you check my earlier patch, you'll see i386-qemu.rmk is just a stub > that includes i386-coreboot.rmk. This is to reduce code duplication > (untill we have a more flexible build system). > > This patch renames kernel.elf to kernel.img i

Re: [PATCH] File access library for lua

2009-06-21 Thread Bean
Hi, On Mon, Jun 22, 2009 at 8:42 AM, Pavel Roskin wrote: > On Sun, 2009-06-21 at 19:33 +0800, Bean wrote: > >> I've added a few function in the lua grub library to access files: > ... >> I also enable the string library from lua. > > The patch doesn't introduce any compiler warnings, and that's go

Re: [PATCH] File access library for lua

2009-06-21 Thread Pavel Roskin
On Mon, 2009-06-22 at 11:38 +0800, Bean wrote: > Actually, this is about to change. The file function here can be used > in a script osdetect.lua to generate menu items at runtime, and I also > plans to integrate lua with menu viewer. OK, that sounds interesting! -- Regards, Pavel Roskin