Re: Keyfile Support for GRUBs LUKS

2013-11-21 Thread Vladimir 'phcoder' Serbinenko
Why do you need offset and size options? keyfile option should be repeteable. The whole array would be passed down and file would be opened instead before reading password and concatebated with it unless --no-password was specified as well. If you have remaining questions feel free to ask here or o

[PATCH] Add a --locale-directory option to grub-install and related tools.

2013-11-21 Thread Colin Watson
* include/grub/util/install.h (GRUB_INSTALL_OPTIONS): Add --locale-directory option. (enum grub_install_options): Add GRUB_INSTALL_OPTIONS_LOCALE_DIRECTORY. * util/grub-install-common.c (grub_install_help_filter): Handle GRUB_INSTALL_OPTIONS_LOCALE_DIRECTORY. (grub_install_parse): Likewise. (get_lo

[PATCH] Enable grub-install --no-nvram option for EFI targets

2013-11-21 Thread Keshav Padram Amburay
Hi, The attached patch enables --no-nvram option for EFI targets in grub-install so that it is possible to prevent grub-install from create a boot entry using efibootmgr in non-removable EFI installs. With Best Regards, Keshav commit 307c6b4440562245d2adfcc585b4175ef9c61260 Author: Keshav Pa

[PATCH] grub-legacy: fix 256 char limitation on kernel command line length

2013-11-21 Thread Paul Gortmaker
The legacy grub only copies 0xff chars of cmdline, which means that longer lines used with nfsroot and static network settings may get truncated. The code where this happens is in stage2/boot.c: while (dest < linux_data_tmp_addr + LINUX_CL_END_OFFSET && *src) *(dest++) = *(sr

Re: Keyfile Support for GRUBs LUKS

2013-11-21 Thread Ralf Ramsauer
On 11/21/13 16:31, Vladimir 'phcoder' Serbinenko wrote: > > Why do you need offset and size options? keyfile option should be > repeteable. The whole array would be passed down and file would be > opened instead before reading password and concatebated with it unless > --no-password was specified a

Re: Shouldn't unicode.pf2 be installed into /boot/grub ?

2013-11-21 Thread Vladimir 'phcoder' Serbinenko
This part is only for backward compatibility. New grub loads fonts from $prefix/fonts On Nov 21, 2013 3:24 PM, "Rémi Mathieu" wrote: > Hi > > Here is what can be found in grub.cfg > > /"search --no-floppy --fs-uuid --set=root // > //if loadfont /usr/share/grub/unicode.pf2 ; then//" > / > GNU/Linu

Re: Keyfile Support for GRUBs LUKS

2013-11-21 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 21.11.2013 20:34, Ralf Ramsauer wrote: > On 11/21/13 16:31, Vladimir 'phcoder' Serbinenko wrote: >> >> Why do you need offset and size options? keyfile option should be >> repeteable. The whole array would be passed down and file would be >> opened instead before reading password and concatebate

Shouldn't unicode.pf2 be installed into /boot/grub ?

2013-11-21 Thread Rémi Mathieu
Hi Here is what can be found in grub.cfg /"search --no-floppy --fs-uuid --set=root // //if loadfont /usr/share/grub/unicode.pf2 ; then//" / GNU/Linux file hierarchie has been defined to keep together what is needed for any operation. So, shouldn't all material needed by grub to be kept in the

Re: [PATCH] Add a --locale-directory option to grub-install and related tools.

2013-11-21 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 21.11.2013 19:32, Colin Watson wrote: > * include/grub/util/install.h (GRUB_INSTALL_OPTIONS): Add > --locale-directory option. > (enum grub_install_options): Add > GRUB_INSTALL_OPTIONS_LOCALE_DIRECTORY. > * util/grub-install-common.c (grub_install_help_filter): Handle > GRUB_INSTALL_OPTIONS_LOCA

Re: grub2 boot root-on-zfs errors

2013-11-21 Thread Beeblebrox
@ Vladimir: >> Please attach your config.log Enclosed as requested (for latest run) A new problem: I updated the git repo to latest and tried to re-build. Build was started with: ./autogen.sh USE_GCC=any --disable-werror ./configure USE_GCC=any --disable-werror gmake USE_GCC=any However, compile f

Re: grub2 boot root-on-zfs errors

2013-11-21 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 22.11.2013 08:09, Beeblebrox wrote: > @ Vladimir: >>> Please attach your config.log > Enclosed as requested (for latest run) > No you didn't. No config.log is attached. Please send it. > A new problem: I updated the git repo to latest and tried to re-build. > Build was started with: > ./autogen

Re: grub2 boot root-on-zfs errors

2013-11-21 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 22.11.2013 08:09, Beeblebrox wrote: > grub-core/osdep/unix/platform.c:34:28: error: use of undeclared > identifier 'PATH_MAX' > char *ret = xmalloc (2 * PATH_MAX); Please try diff --git a/grub-core/osdep/unix/platform.c b/grub-core/osdep/unix/platform.c index 65c93f1..a527a1c 100644 --- a/grub

Re: [PATCH 2/2] grub-efi: allow compilation without large model support

2013-11-21 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 22.11.2013 08:23, jackie.hu...@windriver.com wrote: > From: Jackie Huang > > -mcmodel=large is not supported by gcc with version lower > than 4.4, but we don't need to use memory over 4GiB, so add > a patch to allow compilation without large model support. > This patch is wrong and would rein