[PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Colin Watson
(Part zero was a patch I already committed that dealt with some trivial cases.) As I mentioned on #grub, and following up on https://lists.gnu.org/archive/html/grub-devel/2009-04/msg00406.html and thread, I'm working on a patch set to eliminate nested functions from GRUB. I'd like to add a couple

[PATCH] Remove unnecessary zero-initialisations from *io.c

2013-01-01 Thread Colin Watson
This patch removes explicit zero-initialisations from *io.c in favour of grub_zalloc. Is there any good reason to initialise these explicitly? This reduces the size of all four of these modules on i386-pc as follows: bufio -56 gzio -24 lzopio-20 xzio -64

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Seth Goldberg
Yay!! Does this change the minimum GCC version needed to build? Thanks!! -S On Jan 1, 2013, at 6:42 AM, Colin Watson wrote: > (Part zero was a patch I already committed that dealt with some trivial > cases.) > > As I mentioned on #grub, and following up on > https://lists.gnu.org/archive/ht

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 01.01.2013 19:31, Seth Goldberg wrote: > Yay!! Does this change the minimum GCC version needed to build? We don't support anything older than 4.2. That's unlikely to change. -- Regards Vladimir 'φ-coder/phcoder' Serbinenko signature.asc Description: OpenPGP digital signature _

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Colin Watson
On Tue, Jan 01, 2013 at 10:31:52AM -0800, Seth Goldberg wrote: > Yay!! Does this change the minimum GCC version needed to build? As Vladimir said, it probably isn't worth the effort at this point for us to wind back the version we recommend, as I suspect not enough of us can conveniently test with

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 01.01.2013 22:14, Colin Watson wrote: > On Tue, Jan 01, 2013 at 10:31:52AM -0800, Seth Goldberg wrote: >> Yay!! Does this change the minimum GCC version needed to build? > > As Vladimir said, it probably isn't worth the effort at this point for > us to wind back the version we recommend, as I

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Andrey Borzenkov
В Tue, 1 Jan 2013 14:42:04 + Colin Watson пишет: > > * If a hook requires more than one local variable from its parent >function, declare "struct _ctx" with the necessary >variables, and convert both the hook and the parent to access the >variables in question via that structure

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread richardvo...@gmail.com
On Tue, Jan 1, 2013 at 4:37 PM, Andrey Borzenkov wrote: > В Tue, 1 Jan 2013 14:42:04 + > Colin Watson пишет: > > > > > * If a hook requires more than one local variable from its parent > >function, declare "struct _ctx" with the necessary > >variables, and convert both the hook and t

Boot windows on external disk : "not a valid root device"

2013-01-01 Thread Hervé Guillemet
Hi, I have a new Asus N56VZ laptop and replaced the Windows hard drive by a SSD with Linux installed. EFI Grub2 successfully launchs Linux. Now I'd like to boot Windows from the original hard drive mounted by USB. No matter what I try, either using grub.cfg or command line, Grub's chainloader fa

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Colin Watson
On Wed, Jan 02, 2013 at 01:37:38AM +0400, Andrey Borzenkov wrote: > В Tue, 1 Jan 2013 14:42:04 + > Colin Watson пишет: > > * If a hook requires more than one local variable from its parent > >function, declare "struct _ctx" with the necessary > >variables, and convert both the hook an

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Colin Watson
On Tue, Jan 01, 2013 at 05:24:00PM -0500, richardvo...@gmail.com wrote: > On Tue, Jan 1, 2013 at 4:37 PM, Andrey Borzenkov wrote: > > В Tue, 1 Jan 2013 14:42:04 + > > Colin Watson пишет: > > > * If a hook requires more than one local variable from its parent > > >function, declare "struct

Re: Boot windows on external disk : "not a valid root device"

2013-01-01 Thread Colin Watson
On Wed, Jan 02, 2013 at 12:31:14AM +0100, Hervé Guillemet wrote: > I have a new Asus N56VZ laptop and replaced the Windows hard drive by a > SSD with Linux installed. EFI Grub2 successfully launchs Linux. > Now I'd like to boot Windows from the original hard drive mounted by USB. > > No matter wha

Re: Boot windows on external disk : "not a valid root device"

2013-01-01 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 02.01.2013 01:14, Colin Watson wrote: > On Wed, Jan 02, 2013 at 12:31:14AM +0100, Hervé Guillemet wrote: >> I have a new Asus N56VZ laptop and replaced the Windows hard drive by a >> SSD with Linux installed. EFI Grub2 successfully launchs Linux. >> Now I'd like to boot Windows from the origina

Re: Boot windows on external disk : "not a valid root device"

2013-01-01 Thread Hervé Guillemet
Le 02/01/2013 01:14, Colin Watson a écrit : > On Wed, Jan 02, 2013 at 12:31:14AM +0100, Hervé Guillemet wrote: >> I have a new Asus N56VZ laptop and replaced the Windows hard drive by a >> SSD with Linux installed. EFI Grub2 successfully launchs Linux. >> Now I'd like to boot Windows from the origi

Re: Boot windows on external disk : "not a valid root device"

2013-01-01 Thread Hervé Guillemet
Le 02/01/2013 01:33, Vladimir 'φ-coder/phcoder' Serbinenko a écrit : > On 02.01.2013 01:14, Colin Watson wrote: > >> On Wed, Jan 02, 2013 at 12:31:14AM +0100, Hervé Guillemet wrote: >>> I have a new Asus N56VZ laptop and replaced the Windows hard drive by a >>> SSD with Linux installed. EFI Grub2

Re: Boot windows on external disk : "not a valid root device"

2013-01-01 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 02.01.2013 02:05, Hervé Guillemet wrote: > Le 02/01/2013 01:33, Vladimir 'φ-coder/phcoder' Serbinenko a écrit : >> On 02.01.2013 01:14, Colin Watson wrote: >> >>> On Wed, Jan 02, 2013 at 12:31:14AM +0100, Hervé Guillemet wrote: I have a new Asus N56VZ laptop and replaced the Windows hard d

Re: [PATCH] Removing nested functions, part one of lots

2013-01-01 Thread Colin Watson
On Wed, Jan 02, 2013 at 12:05:04AM +, Colin Watson wrote: > On Wed, Jan 02, 2013 at 01:37:38AM +0400, Andrey Borzenkov wrote: > > В Tue, 1 Jan 2013 14:42:04 + > > Colin Watson пишет: > > > * If a hook requires more than one local variable from its parent > > >function, declare "struct