2.02~beta3 release

Hello, all. I've just released 2.02~beta3. The goal of this release is to chase bugs so that we can move to 2.02 release. So from now on if you think that your patch should be included in 2.02 rather than in 2.03, please add [2.02] to the beginning of your email subject The tarball is available at

Re: Grub translation

On 28.02.2016 14:32, Baadur Jobava wrote: > Hello, is there some resource out there on how to get into Grub > localization? > > I'd like to look at translating Grub to Romanian. > It's managed by Translation project. https://translationproject.org/html/welcome.html. > Thanks! > > Jobava > > >

Re: [PATCH] efidisk: Respect block_io_protocol buffer alignment

On 18.02.2016 19:05, Leif Lindholm wrote: > Returned from the OpenProtocol operation, the grub_efi_block_io_media > structure contains the io_align field, specifying the minimum alignment > required for buffers used in any data transfers with the device. > > Make grub_efidisk_readwrite() allocate

Re: How to chainload grub on different platforms?

On 24.02.2016 18:14, Andrei Borzenkov wrote: > On x86 we can use multiboot (on in exceptional case chainload boot block). > > On EFI we can use chainloader to launch EFI binary. > > What can we do on PowerPC? SPARC? ARM? MIPS? > > I think we really need some generic platform-independent interfac

Re: [PATCH v2 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi

On 26.02.2016 12:15, Fu Wei wrote: > Hi Andrei, > > On 26 February 2016 at 18:50, Andrei Borzenkov wrote: >> On Fri, Feb 26, 2016 at 8:59 AM, Fu Wei wrote: >>> +@subsection xen_module >>> >>> -@deffn Command xen_linux file [arguments] >>> -Load a dom0 kernel image for xen hypervi

Re: OBP available region patch fails with clang

On 24.02.2016 18:35, Andrei Borzenkov wrote: > kern/ieee1275/mmap.c:52:22: error: comparison of integers of different > signs: 'grub_ssize_t' > (aka 'int') and 'unsigned int' [-Werror,-Wsign-compare] > if (available_size > sizeof (available)) > ~~ ^ ~~ > 1

Re: aarch64 clang build failure

On 24.02.2016 19:25, Andrei Borzenkov wrote: > TARGET_OBJ2ELF= sh genmod.sh moddep.lst disk.module > build-grub-module-verifier disk.mod > build-grub-module-verifier: error: unsupported relocation 0x10d. > https://llvm.org/bugs/show_bug.cgi?id=26030 signature.asc Description: OpenPGP digital s

[PATCH] U-Boot self-relocatable images

Currently to compile arm-uboot port for a different platform you need to relink it at new address with modifying source code. This set of patches changes to U-boot in-place loading and images that relocate themselves. I'd be interested in opinion of ARM guys From 80f8cfe29ace49167c1ad6439ddd15519b0

Re: [PATCH] efidisk: Respect block_io_protocol buffer alignment

On 18.02.2016 16:00, Leif Lindholm wrote: > Returned from the OpenProtocol operation, the grub_efi_block_io_media > structure contains the io_align field, specifying the minimum alignment > required for buffers used in any data transfers with the device. > > Make grub_efidisk_readwrite() allocate

Re: Respect EFI block-io buffer alignment

On 17.02.2016 18:00, Andrei Borzenkov wrote: > 17.02.2016 19:23, Vladimir 'φ-coder/phcoder' Serbinenko пишет: >> On 17.02.2016 16:48, Leif Lindholm wrote: >>> This resolves a complete failure to access devices connected to the >>> SATA port on the ARM ltd. Jun

Re: Respect EFI block-io buffer alignment

On 17.02.2016 16:48, Leif Lindholm wrote: > This resolves a complete failure to access devices connected to the > SATA port on the ARM ltd. Juno platform (apart from a violation of the > UEFI block io protocol). > > The below is a bit of a hack, but I'd like some feedback on preferred > solution b

Re: [PATCH] sparc64: OBP available region contains grub. Start at grub_phys_end

Committed, thanks On 12.02.2016 22:35, Eric Snowberg wrote: > OBP available region contains grub. Start at grub_phys_end. > > This prevents a problem where grub was being overwritten since > grub_phys_start does not start at a zero offset within the memory > map. > > Signed-off-by: Eric Snowberg

Re: [PATCH v3 2/3] i386: Add support for loading from android bootimg

me if I'm wrong but it should be >> clear from context of file is Android image or usual linux one? >> >> Le ven. 12 févr. 2016 20:19, Shea Levy a écrit : >> >>> On 2016-02-12 12:50, Vladimir 'φ-coder/phcoder' Serbinenko wrote: &g

Re: [RFC] grub-install: allow none or multiple install devices on PC BIOS

On 08.05.2015 20:53, Andrei Borzenkov wrote: > There are two main applications. > > 1. Omit install device to create generic image intended for chainloading > from other master loader. Such image can be put on any device (or file > system) and will still be able to find its $root. Currently even w

Re: [PATCH] core/partmap: Add El Torito boot catalog parsing

On 20.06.2015 12:16, Ross Lagerwall wrote: > GRUB calculates a boot device based on the device path given by the > firmware so: Please reread what Andrei said. GRUB combines stored prefix and information from firmware. Syntax ()/boot/grub will always use the full disk signature.asc Description:

Re: CMOS_ADDRESS of ThinkVantage

On 02.08.2015 17:28, Andreas Freimuth wrote: > Hi, > > i just wonted to contribute the correct CMOS address for the > ThinkVantage-Button for: > > LENOVO ThinkPad T410s (2912W1C) > > it is: > > GRUB_BUTTON_CMOS_ADDRESS=101:3 > Added to docs, thank you > Regards > > > > _

Re: [PATCH v3 1/3] Add helper functions to interact with android bootimg disks.

On 12.02.2016 18:42, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 08.02.2016 21:47, Shea Levy wrote: >> Currently, the kernel, command line, and ramdisk can be extracted. >> --- >> grub-core/Makefile.core.def| 1 + >> gru

Re: [PATCH v3 2/3] i386: Add support for loading from android bootimg

On 08.02.2016 21:47, Shea Levy wrote: > --- > grub-core/loader/i386/linux.c | 27 +-- > 1 file changed, 21 insertions(+), 6 deletions(-) > > diff --git a/grub-core/loader/i386/linux.c b/grub-core/loader/i386/linux.c > index fddcc46..6ab8d3c 100644 > --- a/grub-core/loader/

Re: [PATCH v3 1/3] Add helper functions to interact with android bootimg disks.

On 08.02.2016 21:47, Shea Levy wrote: > Currently, the kernel, command line, and ramdisk can be extracted. > --- > grub-core/Makefile.core.def| 1 + > grub-core/loader/android_bootimg.c | 253 > + > include/grub/android_bootimg.h | 12 ++ > 3 fil

Re: Dead link in TODO file

On 12.02.2016 09:28, Thomas Huth wrote: > > Hi, > > The TODO file in the grub repository refers to Grub Wiki at > http://grub.enbug.org/ ... however, that URL seems to be invalid > nowadays. Is there a new Wiki somewhere else? > Unfortunately not unless we find a volunteer to host it > Thomas

Re: [PATCH] grub-install: Include all decompressor modules in pvxen core image.

On 07.12.2014 18:26, Andrei Borzenkov wrote: > В Sun, 07 Dec 2014 18:18:47 +0100 > Vladimir 'φ-coder/phcoder' Serbinenko пишет: > >> On 30.11.2014 14:34, Ian Campbell wrote: >>> On Sun, 2014-11-30 at 11:51 +, Ian Campbell wrote: >>>> On Sun, 201

Re: [PATCH 4/5] Cryptomount support plain dm-crypt

On 29.06.2015 16:30, John Lane wrote: > From: John Lane > > --- > grub-core/disk/cryptodisk.c | 298 > +++- > grub-core/disk/luks.c | 195 + > include/grub/cryptodisk.h | 8 ++ > 3 files changed, 310 insertions(+), 19

Re: [PATCH v2 2/6] relocator: Do not use memory region if its starta is smaller than size

Applied, thanks On 20.07.2015 16:35, Daniel Kiper wrote: > malloc_in_range() should not use memory region if its starta is smaller > than size. Otherwise target wraps around and points to region which is > usually not a RAM, e.g.: > > loader/multiboot.c:93: segment 0: paddr=0x80, memsz=0x3f800

Re: Plain dm-crypt

On 27.10.2015 12:15, Andrei Borzenkov wrote: > On Tue, Oct 27, 2015 at 2:10 PM, Vladimir 'phcoder' Serbinenko > wrote: >> There are patches for it but they will not be integrated as plain dm-crypt >> has no advantages compared to LUKS and cannot be configured reliably when >> device names change a

Re: [PATCH] efidisk: prevent errors from diskfilter scan of removable drives

On 12.02.2016 15:38, Andrei Borzenkov wrote: > On Fri, Feb 12, 2016 at 5:29 PM, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> On 05.02.2016 17:56, Andrei Borzenkov wrote: >>> Map EFI_NO_MEDIA to GRUB_ERR_OUT_OF_RANGE that is ignored by diskfilter. >>>

Re: macosx (darwin) fail to boot from Grub2

Fixed xnu code. On 28.06.2015 11:45, dan...@msw.it wrote: > Hi friends, > I've just installed successifully Ubuntu 14.04 on iMac 9.1 (Darwin). > > The partitions are so suddivided: > > > |-EFI-FAT32-|---MAC-HFS+---|--/BOOT-EXT4--|/ROOT-EXT4|/HOME-REISERFS-|---SWAP---| > > > >

Re: [PATCH] efidisk: prevent errors from diskfilter scan of removable drives

On 05.02.2016 17:56, Andrei Borzenkov wrote: > Map EFI_NO_MEDIA to GRUB_ERR_OUT_OF_RANGE that is ignored by diskfilter. This > actually matches pretty close (we obviously attempt to read outside of media) > and avoids adding more error codes. > > This affects only internally initiated scans. If re

Re: [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping

On 11.02.2016 15:13, Juergen Gross wrote: > On 11/02/16 13:33, Daniel Kiper wrote: >> On Thu, Feb 11, 2016 at 08:53:24AM +0100, Juergen Gross wrote: >>> Modern pvops linux kernels support an initrd not covered by the initial >>> mapping. This capability is flagged by an elf-note. >>> >>> In case th

Re: [PATCH v2 3/6] xen: factor out allocation of page tables into separate function

On 11.02.2016 13:53, Juergen Gross wrote: > On 11/02/16 13:27, Daniel Kiper wrote: >> On Thu, Feb 11, 2016 at 08:53:23AM +0100, Juergen Gross wrote: >>> Do the allocation of page tables in a separate function. This will >>> allow to do the allocation at different times of the boot preparations >>>

Re: [PATCH] [RFC] Add exitcode support

On 18.08.2015 21:17, Ben Hildred wrote: > Let's assume for a minute that I have compiled grub as a multiboot image > and have called it from another bootloader, say iPXE.Now iPXE assumes > that any false return is an error. What happens when grub returns with > exit next, does iPXE get a true or fa

Re: [PATCH] arm64: build with -mcmodel=large

On 22.01.2016 03:14, Colin Watson wrote: > On Fri, Dec 25, 2015 at 06:18:55PM +, Leif Lindholm wrote: >> So, it seems this toolchain generates the HI21/LO12 relocation combo: >> - R_AARCH64_ADR_PREL_PG_HI21/R_AARCH64_ADR_PREL_PG_HI21_NC >> - R_AARCH64_LDST16_ABS_LO12_NC >> - R_AARCH64_LDST32_AB

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

On 22.01.2016 14:01, Andrew Cooper wrote: > On 22/01/16 12:56, Vladimir 'φ-coder/phcoder' Serbinenko wrote: >> On 22.09.2015 10:53, Ian Campbell wrote: >>> Hi Vladimir & grub-devel, >>> >>> Do you have any thoughts on this issue with i386 pv-gru

Re: [PATCH V2] net: fix ipv6 routing

Were Andrey's comments ever adressed? Other than his comments the patch looks good. On 20.09.2015 11:50, Andrei Borzenkov wrote: > 28.08.2015 18:24, Josef Bacik пишет: >> ipv6 routing in grub2 is broken, we cannot talk to anything outside >> our local >> network or anything that doesn't route in ou

Re: PV-Grub2 works here but doesnt work their...

On 10.09.2015 20:46, Shaun Reitan wrote: > I posted this to the Xen Mailing Lists too but figured i should post > here as well. > > We are experiencing a odd issue after we built grub2 support. The image > we built works fine on some hosts and then just hangs on others. > > I built grub2 as f

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

On 22.09.2015 10:53, Ian Campbell wrote: > Hi Vladimir & grub-devel, > > Do you have any thoughts on this issue with i386 pv-grub2? > Is it still an issue? If so I'll try to replicate it. From stack dump I see that it has jumped to NULL. GRUB has no threads so it's not a race condition with itsel

Re: test command (in-)compatibility

On 08.12.2015 17:36, Andrei Borzenkov wrote: > `test' in GRUB implicitly assumes `and' operation between consecutive > terms and does not enforce proper syntax like UNIX (bash) `test' does. Both > > test x y z > test x = y z = w > > result in error in Linux and are silently accepted by GRUB with

Re: [Xen-devel] Uniform commands for booting xen

On 18.01.2016 11:28, Ian Campbell wrote: > On Mon, 2016-01-11 at 15:06 +0100, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> On 13.11.2015 10:50, Ian Campbell wrote: >>> On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: >>>>> How do you ex

Re: [Xen-devel] Uniform commands for booting xen

On 11.01.2016 15:32, Jan Beulich wrote: On 11.01.16 at 15:06, wrote: >> On 13.11.2015 10:50, Ian Campbell wrote: >>> On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: > How do you express modules other than kernel+initrd in that > scheme, without grub needing to be aware of a

Re: [Xen-devel] Uniform commands for booting xen

On 13.11.2015 10:50, Ian Campbell wrote: > On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: >>> How do you express modules other than kernel+initrd in that >>> scheme, without grub needing to be aware of any new addition we >>> may find necessary going forward? >>> >> >> Are modules used

Re: ARM*-EFI timers

On 07.01.2016 19:59, Andrei Borzenkov wrote: > 07.01.2016 21:08, Vladimir 'φ-coder/phcoder' Serbinenko пишет: >> Hello, all. In my automated tests I found out that on ARM64-EFI sleep 10 >> actually sleeps for 100s. The culprit is that EFI doesn't call our timer

ARM*-EFI timers

Hello, all. In my automated tests I found out that on ARM64-EFI sleep 10 actually sleeps for 100s. The culprit is that EFI doesn't call our timer every 1ms but every 10ms. I propose time1.diff to correct: request EFI to call us every 10ms and increment timer variable by 10. For arm64 I propose to u

Re: enable_progress_indicator

On 06.01.2016 12:27, Andrei Borzenkov wrote: > Do we really need yet another magic variable? Tests already set > "grubshell", and in normal case why would one load progress but then > disable it? > > Why progress gets loaded in the first place? Can we somehow omit it? > emu loads all the modules.

Re: util/bin2h.c?

On 01.01.2016 09:01, Andrei Borzenkov wrote: > Do still need it? It is not used since 2 years. > Feel free to remove it > ___ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailman/listinfo/grub-devel > signature.asc Descript

Re: [PATCH] arm64: build with -mcmodel=large

On 25.12.2015 19:18, Leif Lindholm wrote: > On Thu, Dec 24, 2015 at 04:20:29AM +, Colin Watson wrote: >> On Thu, Dec 24, 2015 at 04:14:20AM +, Colin Watson wrote: >>> This fixes a build failure with very current GCC versions, such as the one >>> in Ubuntu xenial. Leif (or anyone with suita

Re: [PATCH] arm64: build with -mcmodel=large

On 25.12.2015 19:18, Leif Lindholm wrote: > On Thu, Dec 24, 2015 at 04:20:29AM +, Colin Watson wrote: >> On Thu, Dec 24, 2015 at 04:14:20AM +, Colin Watson wrote: >>> This fixes a build failure with very current GCC versions, such as the one >>> in Ubuntu xenial. Leif (or anyone with suita

Re: [PATCH] 30_os-prober: derive --class from os-prober generated label

Go ahead On 26.12.2015 08:09, Andrey Borzenkov wrote: > Currently only Windows gets distinguished icons, everything else is displayed > using the same generic one. Add additional --class based on os-prober returned > label, which usually is expected to match primary distribution name. > > Also use

Re: [PATCH] normal: fix get_logical_num_lines

On 28.12.2015 05:09, Michael Chang wrote: > On Thu, Dec 24, 2015 at 02:48:34PM +0300, Andrei Borzenkov wrote: >> On Wed, Dec 23, 2015 at 7:45 AM, Michael Chang wrote: >>> In menu editing mode, grub2 shows bogus line if the character being >>> edited is at last column of entry. This patch fixes the

Re: grub causing NVDIMMs to be treated as normal memory

On 28.11.2015 07:41, Elliott, Robert (Persistent Memory) wrote: > If you can define a standard meaning for 16 and 20, that'd be more > useful than marking them as OEM defined. There will always be a mix > of software that interprets it as unusable vs. follows this new > advice. 16 would be "RAM ho

Re: Each grub-mkrescue run leaves a file in /tmp/

On 28.12.2015 04:40, Andrei Borzenkov wrote: > 27.12.2015 23:48, Thomas Schmitt пишет: >> Hi, >> >> when running grub-mkrescue from Debian Sid, i get a >> growing collection of files in /tmp. One per run. >> >> Names are like: >> /tmp/grub.00529W >> >> Content is always the same: >> insmod part

Re: [PATCH v3 2/3] mkrescue: add argument --fixed-time to get reproducible uuids

On 04.12.2015 19:32, Alexander Couzens wrote: > The uuid generation is based on the time. > --- > util/grub-mkrescue.c | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) > This breaks uniqueness assumptions for UUID and we use UUID to find the right disk, as it's not possibl

Re: [PATCH v3 3/3] Makefile/coreboot use SOURCE_DATE_EPOCH as time source if set

Looks good On 04.12.2015 19:32, Alexander Couzens wrote: > mkstandalone sets timestamps for files which can be overriden by a > fixed_timestamp. > This makes it possible to build reproducible builds for coreboot. > > To build a reproducible build of grub for coreboot do: > export SOURCE_DATE_EPOC

Re: [PATCH v3 1/3] mkstandalone: add argument --fixed-time to override mtime of files

> + fixed_time = -1; -1 is actually perfectly valid. Can we have a second boolean to avoid special-casing -1? signature.asc Description: OpenPGP digital signature ___ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/g

Re: [PATCH 3/3] Makefile: use FIXED_TIMESTAMP for mkstandalone if set

On 04.12.2015 17:10, Alexander Couzens wrote: > mkstandalone sets timestamps for files which can be overriden by a > fixed_timestamp. > This makes it possible to build reproducible builds for coreboot. > > To build a reproducible build of grub for coreboot do: > make default_payload.elf FIXED_TIM

Re: [PATCH 1/3] mkstandalone: add argument --fixed-time to override mtime of files

On 04.12.2015 17:10, Alexander Couzens wrote: > mkstandalone adds several files to an archive. Doing this it uses the > mtime to give these files a timestamp. > --fixed-time overrides these timestamps with a given. > > Replacing all timestamps with a specific one is required > to get reproducible

Re: Grub get and set efi variables

On 14.11.2015 05:03, Andrei Borzenkov wrote: > 13.11.2015 22:42, Ignat Korchagin пишет: > +static enum efi_var_type > +parse_efi_var_type (const char *type) > +{ > + if (!grub_strncmp (type, "string", sizeof("string"))) > +return EFI_VAR_STRING; > + I thi

Re: grub causing NVDIMMs to be treated as normal memory

New version attached >> GRUB_MEMORY_COREBOOT_TABLES = 16, >> GRUB_MEMORY_CODE = 20, >> /* This one is special: it's used internally but is never reported > Note (b): The internal GRUB_MEMORY_CODE (20) value is > leaking through to the E820 table. > > That appears to

Re: flex version

Andrei, could you please commit this patch? On 15.11.2015 07:26, Andrei Borzenkov wrote: > 14.11.2015 19:22, Peter Cheung пишет: >> hi >> my flex in mac is installed using macports. >> >> $flex --version >> flex 2.5.35 Apple(flex-31) >> >> And the line in configure fail >> >> version=`$LEX --ve

Re: grub causing NVDIMMs to be treated as normal memory

What about this patch for the passing of pram? diff --git a/grub-core/mmap/efi/mmap.c b/grub-core/mmap/efi/mmap.c index 900a4d6..0c03c5d 100644 --- a/grub-core/mmap/efi/mmap.c +++ b/grub-core/mmap/efi/mmap.c @@ -118,6 +118,12 @@ grub_efi_mmap_iterate (grub_memory_hook_t hook, void *hook_data,

Re: [Xen-devel] [PATCH GRUB] Allow initrd concatenation on ARM64

On 12.11.2015 14:27, Ian Campbell wrote: > On Thu, 2015-11-12 at 14:08 +0100, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> While on it also change "xen_linux" to "xen_kernel" to be vendor-neutral >> Could someone test please? I only compile-test

Re: [PATCH v2 3/6] i386/relocator: Add grub_relocator64_efi relocator

On 10.11.2015 15:38, Daniel Kiper wrote: >>> - if (entry_specified) >>> > > + if (keep_bs && efi_entry_specified) >>> > > +grub_multiboot_payload_eip = efi_entry; >>> > > + else if (entry_specified) >>> > > grub_multiboot_payload_eip = entry; >>> > > >> > This seems redundant. > What is

Uniform commands for booting xen

Hello, all. I'd like to have set of commands that would boot xen on all platforms. I thought of following set: xen_hypervisor FILE XEN_OPTIONS xen_kernel FILE KERNEL_OPTIONS xen_initrd INITRD INITRD INITRD all initrds are concatenated. xen_xsm ??? On arm64 it would use the arm64 xen FDT protocol

Re: [PATCH v2 5/6] multiboot2: Add support for relocatable images

On 12.11.2015 14:15, Daniel Kiper wrote: > On Tue, Nov 10, 2015 at 04:23:46PM +0100, Vladimir 'phcoder' Serbinenko wrote: >> Le 10 nov. 2015 3:52 PM, "Daniel Kiper" a écrit : >>> On Mon, Nov 09, 2015 at 09:08:35PM +0100, Vladimir 'φ-coder/phcoder

[PATCH GRUB] Allow initrd concatenation on ARM64

While on it also change "xen_linux" to "xen_kernel" to be vendor-neutral Could someone test please? I only compile-tested it diff --git a/docs/grub.texi b/docs/grub.texi index 1df3db2..112b42b 100644 --- a/docs/grub.texi +++ b/docs/grub.texi @@ -3859,7 +3859,7 @@ you forget a command, you can run t

Re: [PATCH v3 3/4] * util/grub.d/20_linux_xen.in: Add support of the XEN boot on aarch64

On 10.11.2015 08:01, Andrei Borzenkov wrote: > Not really, although this is a good observation. Actually I think that > xen_initrd should indeed have behavior 1, because what it does is > provide initrd image to Linux kernel, and - although not often used - > initrd image may be constructed by conc

Re: [PATCH v4] ofdisk: add sas disks to the device list

On 11.11.2015 14:14, Paulo Flabiano Smorigo wrote: > + ptr = (grub_uint64_t *) (table + sizeof (grub_uint64_t) * i); > + grub_snprintf (bufptr, 32, "/disk@%" PRIxGRUB_UINT64_T, *ptr); Please declare table as being grub_uint64_t * and then just use table[i]. This code violates cast

Re: [PATCH] broken ESC navigation if authentication is used

On 11.06.2015 05:55, Andrei Borzenkov wrote: > В Wed, 10 Jun 2015 21:35:51 +0200 > "Vladimir 'phcoder' Serbinenko" пишет: > >> This patch may allow to escape to shell if menu was called from context >> without menu entries. This may happen inadvertently I.a. when using >> configfile. You need to

Re: [PATCH v2 1/6] gitignore: Ignore *.orig, *.rej and *.swp files

On 09.11.2015 16:39, Daniel Kiper wrote: > On Mon, Nov 09, 2015 at 04:34:20PM +0100, Vladimir 'φ-coder/phcoder' > Serbinenko wrote: >> On 09.11.2015 16:29, Daniel Kiper wrote: >>> On Wed, Nov 04, 2015 at 01:03:56PM +0100, Vladimir 'phcoder' Serbinenko

Re: [PATCH v2 5/6] multiboot2: Add support for relocatable images

On 20.07.2015 16:35, Daniel Kiper wrote: > Signed-off-by: Daniel Kiper What is handling the actual relocation? Why doesn't this patch include support for ELF relocations? > --- > grub-core/loader/i386/multiboot_mbi.c |6 ++-- > grub-core/loader/multiboot.c | 12 +-- > grub-core

Re: [PATCH v2 4/6] multiboot2: Add tags used to pass ImageHandle to loaded image

On 09.11.2015 20:15, Vladimir 'phcoder' Serbinenko wrote: > Lgtm > > Le 20 juil. 2015 4:36 PM, "Daniel Kiper" > a écrit : > > Add tags used to pass ImageHandle to loaded image. It is used > by at least ExitBootServices() function. > > Signed-off-by: D

Re: [PATCH v2 3/6] i386/relocator: Add grub_relocator64_efi relocator

On 20.07.2015 16:35, Daniel Kiper wrote: > Add grub_relocator64_efi relocator. It will be used on EFI 64-bit platforms > when multiboot2 compatible image requests MULTIBOOT_TAG_TYPE_EFI_BS. Relocator > will set lower parts of %rax and %rbx accordingly to multiboot2 specification. > On the other han

Fwd: Re: [PATCH v2 6/6] multiboot2: Do not pass memory maps to image if EFI boot services are enabled

Forwarded Message Subject:Re: [PATCH v2 6/6] multiboot2: Do not pass memory maps to image if EFI boot services are enabled Date: Mon, 9 Nov 2015 20:08:38 +0100 From: Vladimir 'phcoder' Serbinenko To: Daniel Kiper And you also need to adjust loader code to r

Fwd: Re: [PATCH v2 6/6] multiboot2: Do not pass memory maps to image if EFI boot services are enabled

Forwarded Message Subject:Re: [PATCH v2 6/6] multiboot2: Do not pass memory maps to image if EFI boot services are enabled Date: Mon, 9 Nov 2015 20:07:23 +0100 From: Vladimir 'phcoder' Serbinenko To: Daniel Kiper This one is good to go. But the main reason

Re: [PATCH v2 2/6] relocator: Do not use memory region if its starta is smaller than size

On 21.07.2015 08:42, Andrei Borzenkov wrote: > On Mon, Jul 20, 2015 at 5:35 PM, Daniel Kiper wrote: >> malloc_in_range() should not use memory region if its starta is smaller >> than size. Otherwise target wraps around and points to region which is >> usually not a RAM, e.g.: >> >> loader/multiboo

Re: [PATCH v2 1/6] gitignore: Ignore *.orig, *.rej and *.swp files

On 09.11.2015 16:29, Daniel Kiper wrote: > On Wed, Nov 04, 2015 at 01:03:56PM +0100, Vladimir 'phcoder' Serbinenko wrote: >> Le 12 août 2015 11:04 AM, "Ian Campbell" a écrit : >>> >>> >>> (Having written the below I see too late that this is a grub patch not a >>> Xen one, a tag in the subject for

Re: [PATCH] Use UEFI Time Service to calibrate TSC

On 09.11.2015 14:23, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 09.11.2015 09:03, Michael Chang wrote: >> On Mon, Nov 09, 2015 at 10:29:55AM +0300, Andrei Borzenkov wrote: >>> On Mon, Nov 9, 2015 at 10:07 AM, Michael Chang wrote: >>>> This patch tri

Re: [PATCH] Use UEFI Time Service to calibrate TSC

On 09.11.2015 09:03, Michael Chang wrote: > On Mon, Nov 09, 2015 at 10:29:55AM +0300, Andrei Borzenkov wrote: >> On Mon, Nov 9, 2015 at 10:07 AM, Michael Chang wrote: >>> This patch tries to detect PIT timer is broken and use UEFI Time Service >>> to calibrate TSC. >> >> Second try :) >> >> https:

Re: [PATCH] ofdisk: allocate space for vscsi table

On 09.11.2015 02:33, Paulo Flabiano Smorigo wrote: > + /* 64 entries should be enough */ > + table_size = sizeof (grub_uint64_t) * 64; Can we do something better than assuming a particular upper limit? You don't even pass this limit to the function in any way. You basically get a buffer o

Re: Anonymous commit (empty Author and Committer)

On 30.10.2015 21:09, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 30.10.2015 21:06, Vladimir 'φ-coder/phcoder' Serbinenko wrote: >> On 30.10.2015 15:26, Andrei Borzenkov wrote: >>> See >>> http://git.savannah.gnu.org/cgit/grub.git/comm

Re: [PATCH] mkimage: zero fill alignment space

On 30.10.2015 16:49, Andrei Borzenkov wrote: > This did not cause real problem but is good for reproducible builds. I hit > it with recent bootinfoscript that displays embedded config; I was puzzled > by random garbage at the end. > > Also remove redundant zeroing code where we fill in the whole m

Re: Anonymous commit (empty Author and Committer)

On 30.10.2015 21:06, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 30.10.2015 15:26, Andrei Borzenkov wrote: >> See >> http://git.savannah.gnu.org/cgit/grub.git/commit/?id=206676601eb853fc319df14cd3398fbdfde665ac >> >> >> I was not even aware that th

Re: Anonymous commit (empty Author and Committer)

On 30.10.2015 15:26, Andrei Borzenkov wrote: > See > http://git.savannah.gnu.org/cgit/grub.git/commit/?id=206676601eb853fc319df14cd3398fbdfde665ac > > > I was not even aware that this is possible. Is there anything on server > side that can prevent it? > > Would be good if commit were amended an

Re: [PATCH v3 3/4] * util/grub.d/20_linux_xen.in: Add support of the XEN boot on aarch64

On 30.10.2015 09:44, Fu Wei wrote: > Hi Vladimir, > > Great thanks for your suggestion! :-) > > On 29 October 2015 at 23:25, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >>> +if [ "x$machine" != xaarch64 ]; then >>> + mult

Re: [PATCH v3 3/4] * util/grub.d/20_linux_xen.in: Add support of the XEN boot on aarch64

> +if [ "x$machine" != xaarch64 ]; then > + multiboot_cmd="multiboot" > + module_linux_cmd="module" > + module_initrd_cmd="module --nounzip" > +else > + multiboot_cmd="xen_hypervisor" > + module_linux_cmd="xen_linux" > + module_initrd_cmd="xen_initrd" > +fi > + Please do not

Re: Be File System identified as bfs

> As you can see, it thinks /dev/sda6 a 'bfs' partition, causing script > 83haiku to fail. > 12 befs|befs_be) debug "$partition is a BeFS partition" ;; Looks like this is a simple one-line change. befs|befs_be is a name used by old driver which had to be removed signature.asc Description

Re: [edk2] [grub PATCH] efinet: disable MNP background polling

On 14.10.2015 17:39, Seth Goldberg wrote: > > >> On Oct 14, 2015, at 4:08 AM, Daniel Kiper wrote: >> >>> On Wed, Oct 14, 2015 at 05:19:32AM +, Ye, Ting wrote: >>> Hi all, >>> >>> If I understand the issue correctly, I don't quite agree that UEFI >>> spec is imprecise about SNP constraints de

Re: [PATCH v3 2/4] arm64: Add xen_boot module file

Committed without the xen_module command. Its argument parsing was non-trivial and I don't quite get what its intent is. Can you resubmit? On 23.07.2015 07:16, fu@linaro.org wrote: > From: Fu Wei > > grub-core/loader/arm64/xen_boot.c > > - This adds support for the Xen boot on ARM specific

Re: [PATCH v3 1/4] arm64: Add and export some accessor functions for xen boot

On 23.07.2015 07:16, fu@linaro.org wrote: > From: Fu Wei > > Add accessor functions of "loaded" flag in > grub-core/loader/arm64/linux.c. > > Export accessor functions of "loaded" flag and > grub_linux_get_fdt function in include/grub/arm64/linux.h. > > Purpose: Reuse the existing code of d

Re: [PATCH 1/3] ieee1275: ofdisk dangling pointer

On 26.10.2015 22:43, Eric Snowberg wrote: > Within commit: 87ec3b7fa9061f470616ed927fc140e995831c00 - > "Don't continue to query block-size if disk doesn't have it.” > a dangling pointer was introduced. > > Fix dangling pointer issue in grub_ofdisk_open where devpath is freed > and then used again

Re: [PATCH 2/3] ieee1275: ofdisk memory leak

On 26.10.2015 22:43, Eric Snowberg wrote: > Fix memory leak added within commit: > 87ec3b7fa9061f470616ed927fc140e995831c00 - "Don't continue to > query block-size if disk doesn't have it.” > Committed, thanks > Signed-off-by: Eric Snowberg > --- > grub-core/disk/ieee1275/ofdisk.c |6 +-

Re: [PATCH 3/3] ieee1275: ofdisk - don't continue to query block-size after we have it

On 26.10.2015 22:43, Eric Snowberg wrote: > Within commit: 87ec3b7fa9061f470616ed927fc140e995831c00 - "Don't continue > to query block-size if disk doesn't have it.” Disks that returned 0 to the > block-size query, still get queried every time. > > Fix logic in grub_ofdisk_get_block_size so the b

Re: [PATCH v2] Suport for bi-endianess in elf file

On 23.07.2015 16:10, Paulo Flabiano Smorigo wrote: > Updated version with the suggestions from Andrei Borzenkov. > > * grub-core/kern/elf.c: check and switch endianess with grub_{be,le}_to > cpu functions. > * grub-core/kern/elfXX.c: Likewise. Applied after fixing following problems: * Since you a

Re: [PATCH] Make CTRL and ALT keys work as expected on EFI systems (version 5).

On 25.02.2014 23:12, Peter Jones wrote: > This is version 4. > > Changes from version 1: > - handles SHIFT as a modifier > - handles F11 and F12 keys > - uses the handle provided by the system table to find our _EX protocol. > > Changes from version 2: > - eliminate duplicate keycode translation.

Re: [PATCH] sparc64: boot performance improvements

On 11.10.2015 18:49, Eric Snowberg wrote: > >> On Oct 10, 2015, at 1:35 AM, Vladimir 'phcoder' Serbinenko >> wrote: >> >> >> Le 10 oct. 2015 3:31 AM, "Eric Snowberg" a écrit : >>> >>> On Oct 9, 2015, at 12:50 AM, Andrei Borzenkov wrote: On Thu, Oct 8, 2015 at 2:20 AM, Eric Snowb

Re: Keyboard not working with QEMU and coreboot and GRUB payload

On 18.08.2015 14:11, Andrei Borzenkov wrote: > On Tue, Aug 18, 2015 at 12:41 AM, Paul Menzel > wrote: >> Dear GRUB folks, >> >> >> just a note that using QEMU with coreboot and GRUB payload, I am unable >> to enter anything. >> >> Using libpayload based payloads, the keyboard works, so I think it’

Re: Add check when store disk cache

On 19.09.2015 09:00, Andrei Borzenkov wrote: > 18.09.2015 12:07, Arch Stack пишет: >> I want to use the part of the filesystem codes in GRUB to read different >> filesystems on Windows. I have almost completed it and I will release >> it in >> a few days. >> But it crash sometimes because of the wr

Re: GNU GRUB maintenance

On 08.10.2015 16:52, Andrei Borzenkov wrote: > On Thu, Oct 8, 2015 at 12:14 AM, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> Hello, all. I'm sorry for not being available to do enough maintenance >> for GRUB in last time but I was overbooked. Yet there is a g

Re: GNU GRUB maintenance

On 08.10.2015 21:34, Konrad Rzeszutek Wilk wrote: > On October 8, 2015 10:52:25 AM EDT, Andrei Borzenkov > wrote: >> On Thu, Oct 8, 2015 at 12:14 AM, Vladimir 'φ-coder/phcoder' Serbinenko >> wrote: >>> Hello, all. I'm sorry for not being available t

Re: [PATCH v2] Ensure that MIPS target code is compiled for the O32 ABI.

On 09.10.2015 23:14, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 13.09.2015 08:32, Andrei Borzenkov wrote: >> 08.09.2015 20:11, Vladimir 'φ-coder/phcoder' Serbinenko пишет: >>> On 23.08.2015 23:50, Mark H Weaver wrote: >>>> Include -ma

Re: [PATCH v2] Ensure that MIPS target code is compiled for the O32 ABI.

On 13.09.2015 08:32, Andrei Borzenkov wrote: > 08.09.2015 20:11, Vladimir 'φ-coder/phcoder' Serbinenko пишет: >> On 23.08.2015 23:50, Mark H Weaver wrote: >>> Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile >>> code for the O32 ABI

Re: [PATCH] sparc64: boot performance improvements

On 06.10.2015 19:39, Eric Snowberg wrote: > Keep of devices open. This can save 6 - 7 seconds per open call and > can decrease boot times from over 10 minutes to 29 seconds on > larger SPARC systems. The open/close calls with some vendors' > SAS controllers cause the entire card to be reinitializ

  1   2   3   4   5   6   7   8   9   10   >