Re: [tuellm...@kit.edu: Bug#866603: grub-common: enabling metadata_csum_seed feature on ext4 renders system unbootable]

2017-06-30 Thread Vladimir 'phcoder' Serbinenko
пт, 30 июн. 2017 г., 14:38 Colin Watson : > Hi, > > Could we possibly just ignore this feature for GRUB? For better or > worse we don't implement metadata checksumming, and we're only doing > read-only access, so I think it would be reasonably safe to ignore > metadata_csum_seed since we wouldn't

Re: Build failure on ppc64le

2017-07-05 Thread Vladimir 'phcoder' Serbinenko
On Tue, Jul 4, 2017, 19:48 Leif Lindholm wrote: > On Sat, Jul 01, 2017 at 02:41:05PM +0200, Paul Menzel wrote: > > Dear GRUB folks, > > > > Building GRUB at commit 7346682 (Allow GRUB to mount ext2/3/4 > > filesystems that have the encryption feature.) for the platform > > `powerpc-ieee1275` with

Re: Boot strap coded mostly in C

2017-07-13 Thread Vladimir 'phcoder' Serbinenko
On Thu, Jul 13, 2017, 09:32 Mr. Miyagi wrote: > Hello Everybody; > > I have just implemented a boot sector program and second stage program > written almost entirely in C using the -Os option of GCC. I am > wondering if this is a good idea, and if so, if it would be practical to > use for GRUB's

Re: [PATCH 1/2] Verify commands executed by grub

2017-07-21 Thread Vladimir 'phcoder' Serbinenko
On Wed, Jul 5, 2017, 23:26 Matthew Garrett wrote: > Pass commands to the verification code. We want to be able to log these > in the TPM verification case. > --- > grub-core/script/execute.c | 27 --- > include/grub/verify.h | 1 + > 2 files changed, 25 insertions(+

Re: [GRUB PARTUUID PATCH V4 0/3] Add PARTUUID detection support

2017-07-23 Thread Vladimir 'phcoder' Serbinenko
I plan to allocate entire day this week to grub. I plan on burning down the backlog On Sun, Jul 23, 2017, 20:02 Nick Vinson wrote: > Hello, > > I submitted this patch set back in May and have not heard anything. I > noticed it also has not been merged into the GRUB tree. Is there some > issue

Re: [PATCH 1/2] Verify commands executed by grub

2017-07-24 Thread Vladimir 'phcoder' Serbinenko
On Sat, Jul 22, 2017, 00:13 Matthew Garrett wrote: > On Fri, Jul 21, 2017 at 7:39 AM, Vladimir 'phcoder' Serbinenko > wrote: > > This omits all separators. So it considers e.g. ab and a b to be the > same. > > Can we have a better array serializati

Re: [PATCH] workaround for multi-device probing in grub-mkconfig

2017-07-27 Thread Vladimir 'phcoder' Serbinenko
As was discussed previously this approach is not correct one. On Sun, Jul 9, 2017, 20:08 Hannes Schweizer wrote: > > upstream bug: https://savannah.gnu.org/bugs/?48228 > Gentoo bug: https://bugs.gentoo.org/show_bug.cgi?id=581904 > --- > util/grub-mkconfig.in | 10 +- > 1 file changed, 5

Re: [PATCH 6/7] efi: restrict arm/arm64 linux loader initrd placement

2017-07-27 Thread Vladimir 'phcoder' Serbinenko
On Thu, Jul 27, 2017, 17:16 Daniel Kiper wrote: > On Mon, Jun 12, 2017 at 03:53:40PM +0100, Leif Lindholm wrote: > > The 32-bit arm Linux kernel is built as a zImage, which self-decompresses > > down to near start of RAM. In order for an initrd/initramfs to be > > accessible, it needs to be place

Re: [PATCH 7/7] efi: change heap allocation type to GRUB_EFI_LOADER_CODE

2017-07-27 Thread Vladimir 'phcoder' Serbinenko
On Thu, Jul 27, 2017, 17:30 Daniel Kiper wrote: > On Mon, Jun 12, 2017 at 03:53:41PM +0100, Leif Lindholm wrote: > > With upcoming changes to EDK2, allocations of type EFI_LOADER_DATA may > > not return regions with execute ability. Since modules are loaded onto > > the heap, change the heap allo

Re: [PATCH 6/7] efi: restrict arm/arm64 linux loader initrd placement

2017-07-27 Thread Vladimir 'phcoder' Serbinenko
On Thu, Jul 27, 2017, 17:40 Daniel Kiper wrote: > On Thu, Jul 27, 2017 at 03:18:16PM +0000, Vladimir 'phcoder' Serbinenko > wrote: > > On Thu, Jul 27, 2017, 17:16 Daniel Kiper wrote: > > > On Mon, Jun 12, 2017 at 03:53:40PM +0100, Leif Lindholm wrote: > &g

Re: [PATCH 7/7] efi: change heap allocation type to GRUB_EFI_LOADER_CODE

2017-07-27 Thread Vladimir 'phcoder' Serbinenko
On Thu, Jul 27, 2017, 17:44 Daniel Kiper wrote: > On Thu, Jul 27, 2017 at 03:33:06PM +0000, Vladimir 'phcoder' Serbinenko > wrote: > > On Thu, Jul 27, 2017, 17:30 Daniel Kiper wrote: > > > On Mon, Jun 12, 2017 at 03:53:41PM +0100, Leif Lindholm wrote: > &

Re: [PATCH v2 01/14] arm64/efi: move EFI_PAGE definitions to efi/memory.h

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
This assumes that all CPUs supporting EFI have 4K pages. What if some of CPUs have other page sizes? Le Thu, Aug 3, 2017 à 12:07 PM, Leif Lindholm a écrit : > The EFI page definitions and macros are generic and should not be confined > to atm64 headers - so move to efi/memory.h. > Also add EFI_P

Re: [PATCH v2 03/14] loader: drop local implementations of find_efi_mmap_size

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
LGTM Le Thu, Aug 3, 2017 à 12:07 PM, Leif Lindholm a écrit : > Now we have a grub_efi_find_mmap_size, replace near-identical > implementations > in i386/linux and mbi2 loaders with calls to that one. > > Signed-off-by: Leif Lindholm > --- > grub-core/loader/i386/linux.c | 51 > +---

Re: [PATCH v2 02/14] efi: add central copy of grub_efi_find_mmap_size

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
Looks good. But out of context the name is not descriptive enough. Can we add a comment near declaration to clarify what this functions is for? Le Thu, Aug 3, 2017 à 12:07 PM, Leif Lindholm a écrit : > There are several implementations of this function in the tree. > Add a central version in gru

Re: [PATCH v2 04/14] efi: add grub_efi_get_ram_base() function for arm*

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
Any reason to use plain error numbers without grub_error? Le Thu, Aug 3, 2017 à 12:09 PM, Leif Lindholm a écrit : > Since ARM platforms do not have a common memory map, add a helper > function that finds the lowest address region with the EFI_MEMORY_WB > attribute set in the UEFI memory map. > >

Re: [PATCH v2 05/14] efi: refactor grub_efi_allocate_pages

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
Le Thu, Aug 3, 2017 à 12:09 PM, Leif Lindholm a écrit : > Expose a new function, grub_efi_allocate_pages_real(), making it possible > to specify allocation type and memory type as supported by the UEFI > AllocatePages boot service. > > Make grub_efi_allocate_pages() a consumer of the new function

Re: [PATCH v2 06/14] efi: move fdt helper library

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
LGTM Le Thu, Aug 3, 2017 à 12:10 PM, Leif Lindholm a écrit : > There is nothing ARM64 (or even ARM) specific about the efi fdt helper > library, which is used for locating or overriding a firmware-provided > devicetree in a UEFI system - so move it to loader/efi for reuse. > > Move the fdtload.h

Re: [PATCH v2 07/14] efi: Add GRUB_PE32_MAGIC definition

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
LGTM On Thu, Aug 3, 2017, 12:09 Leif Lindholm wrote: > Add a generic GRUB_PE32_MAGIC definition for the PE 'MZ' tag and delete > the existing one in arm64/linux.h. > > Update arm64 Linux loader to use this new definition. > > Signed-off-by: Leif Lindholm > --- > grub-core/loader/arm64/linux.c

Re: [PATCH v2 08/14] arm64 linux loader: improve type portability

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
LGTM On Thu, Aug 3, 2017, 12:10 Leif Lindholm wrote: > In preparation for turning this into a common loader for 32-bit and 64-bit > platforms, ensure the code will compile cleanly for either. > > Signed-off-by: Leif Lindholm > --- > grub-core/loader/arm64/linux.c | 15 ++- > grub-c

Re: [PATCH v2 09/14] arm64 linux loader: rename functions and macros and move to common headers

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
On Thu, Aug 3, 2017, 12:14 Leif Lindholm wrote: > In preparation for using the linux loader for 32-bit and 64-bit platforms, > rename > grub_arm64*/GRUB_ARM64* to grub_efi*/GRUB_EFI*. > > Move prototypes for now-common functions to efi/efi.h. > > Signed-off-by: Leif Lindholm > --- > grub-core/l

Re: [PATCH v2 09/14] arm64 linux loader: rename functions and macros and move to common headers

2017-08-03 Thread Vladimir 'phcoder' Serbinenko
We can use armxx as an infix for names that are for both arm and aarch64 On Thu, Aug 3, 2017, 18:44 Vladimir 'phcoder' Serbinenko wrote: > > > On Thu, Aug 3, 2017, 12:14 Leif Lindholm wrote: > >> In preparation for using the linux loader for 32-bit and 64-bit >&g

Re: [PATCH v2 10/14] loader: switch arm/linux to grub_linux_kernel_header struct

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
Le Thu, Aug 3, 2017 à 12:15 PM, Leif Lindholm a écrit : > Use kernel header struct and generic magic definition to align with > arm64/linux loader. > > Signed-off-by: Leif Lindholm > --- > grub-core/loader/arm/linux.c | 11 +-- > include/grub/arm/linux.h | 15 --- > 2 fi

Re: [PATCH v2 11/14] arm/efi: switch to arm64 linux loader

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
Le Thu, Aug 3, 2017 à 12:10 PM, Leif Lindholm a écrit : > Switch over to the EFI-stub aware arm64 loader for 32-bit ARM platforms. > > Signed-off-by: Leif Lindholm > --- > grub-core/Makefile.core.def | 6 +++--- > grub-core/kern/efi/mm.c | 2 +- > include/grub/efi/efi.h | 2 -- > 3 fil

Re: [PATCH v2 12/14] arm: delete unused efi support from loader/arm

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
LGTM Le Thu, Aug 3, 2017 à 12:11 PM, Leif Lindholm a écrit : > The 32-bit arm efi port now shares the 64-bit linux loader, so delete > the now unused bits from the 32-bit linux loader. > > This in turn leaves the grub-core/kern/arm/efi/misc.c unused, so > delete that too. > > Signed-off-by: Leif

Re: [PATCH v2 13/14] efi: restrict arm/arm64 linux loader initrd placement

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
LGTM Le Thu, Aug 3, 2017 à 12:15 PM, Leif Lindholm a écrit : > The 32-bit arm Linux kernel is built as a zImage, which self-decompresses > down to near start of RAM. In order for an initrd/initramfs to be > accessible, it needs to be placed within the first ~768MB of RAM. > The initrd loader bui

Re: [PATCH v2 14/14] efi: change heap allocation type to GRUB_EFI_LOADER_CODE

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
LGTM Le Thu, Aug 3, 2017 à 12:09 PM, Leif Lindholm a écrit : > With upcoming changes to EDK2, allocations of type EFI_LOADER_DATA may > not return regions with execute ability. Since modules are loaded onto > the heap, change the heap allocation type to GRUB_EFI_LOADER_CODE in > order to permit

Re: [tuellm...@kit.edu: Bug#866603: grub-common: enabling metadata_csum_seed feature on ext4 renders system unbootable]

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
Makes sense. Go ahead with ignoring this flag Le Fri, Jun 30, 2017 à 2:50 PM, Colin Watson a écrit : > On Fri, Jun 30, 2017 at 12:43:02PM +0000, Vladimir 'phcoder' Serbinenko > wrote: > > пт, 30 июн. 2017 г., 14:38 Colin Watson : > > > Could we possibly just ign

Re: [PATCH 2/3] crypto: switch nonstandard escape sequence to its hex value

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
Please change this place to rather use GRUB_TERM_ESC and then change GRUB_TERM_ESC definition Le Thu, Aug 3, 2017 à 6:13 PM, Pete Batard a écrit : > Regards, > > /Pete > ___ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailman

Re: [PATCH 3/3] io: add a GZ prefix to gzio specific defines

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
If you're already changing it could you used GRUB_GZ_ rather than GZ_ prefix? Le Thu, Aug 3, 2017 à 6:13 PM, Pete Batard a écrit : > Regards, > > /Pete > ___ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailman/listinfo/grub-d

Re: [PATCH 1/3] zfs: remove size_t typedef and use grub_size_t instead

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
Committed, thanks Le Thu, Aug 3, 2017 à 6:12 PM, Pete Batard a écrit : > Regards, > > /Pete > ___ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailman/listinfo/grub-devel > ___ Grub-

Re: Decrypt AES(rijndael) file

2017-08-07 Thread Vladimir 'phcoder' Serbinenko
Le Mon, Jul 10, 2017 à 8:38 PM, Rafael Machado < rafaelrodrigues.mach...@gmail.com> a écrit : > Hi everyone > > I need to find a way to decrypt a file at grub, and this file is encrypted > as AES256 (rijndael). > > This is far from being a complete task description. What do you want to do with dec

Re: [PATCH] ehci: fix build error of print format specifier

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Fixed in 284afab081ef7ed7db2c59151853aa3c92d988f2 Le Tue, Jun 13, 2017 à 8:52 AM, Michael Chang a écrit : > It is introduced by commit e058df7b5a9cc7aaa9872eaa916b715544a8f9840, use > PRIxGRUB_ADDR as format specfier to fix the portability problem. > --- > grub-core/bus/usb/ehci.c | 4 ++-- > 1

Re: grub2-probe confused by double bind mount?

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Upstream doesn't use grub2 prefix. Please try with latest upstream and make sure it's compiled with libdevmapper On Mon, Aug 7, 2017, 08:57 Chris Murphy wrote: > It's not the double bind mount. I've removed it by eliminating the -o > subvol option and adjusted paths accordingly during assembly.

Re: [PATCH] * grub-core/fs/udf.c: Fix reading label, lvd.ident is dstring

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Committed with fixes and simplifications. The biggest problem was lack of check whether grub_malloc succeeded Le Thu, Jun 22, 2017 à 2:34 PM, Pali Rohár a écrit : > UDF dstring has stored length in the last byte of buffer. Therefore last > byte is not part of recorded characters. And empty strin

Re: [PATCH v2] * grub-core/fs/udf.c: Add support for UUID

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Committed with few fixes. Thanks Le Thu, Jun 22, 2017 à 2:42 PM, Pali Rohár a écrit : > Use same algorithm as in libblkid from util-linux v2.30. > > 1. Take first 16 bytes from UTF-8 encoded string of VolumeSetIdentifier > 2. If all bytes are hexadecimal digits, convert to lowercase and use as >

Re: grub-mkrescue reacts vaguely on xorriso problems

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Fixed both problems Le Sat, Jul 15, 2017 à 12:26 PM, Thomas Schmitt a écrit : > Hi, > > Ping. > > (Bug or feature ? If bug: what exactly would be the desirable behavior ?) > > -- > Date: Wed, 14 Jun 2017 11:37:11 +0200 > Fro

Re: Writing support for Yubikey in Grub?

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Le Mon, Jun 12, 2017 à 12:11 AM, Andrew Ross a écrit : > I'm contemplating an attempt at writing a module to enable use of the > challenge-response feature of the Yubikey to provide part of the > passphrase for a luks partition, using grub to do the initial > decryption. I'm after some advice on

Re: [PATCH v2 14/14] efi: change heap allocation type to GRUB_EFI_LOADER_CODE

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
It means that the patch is fine but something else stops it from being committed. In this case I wanted to go through the whole stack first Le Mon, Aug 7, 2017 à 6:19 PM, Konrad Rzeszutek Wilk a écrit : > On Mon, Aug 07, 2017 at 02:11:16PM +0000, Vladimir 'phcoder' Serbinenko >

Re: [PATCHv2 2/3] core: use GRUB_TERM_ definitions when handling term characters

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Le Mon, Aug 7, 2017 à 6:22 PM, Pete Batard a écrit : > This is the updated proposal from the original "[PATCH 2/3] crypto: > switch nonstandard escape sequence to its hex value". > > Following up on Vladimir's comments, I decided to look up all the GRUB > sources where key/term entries were being

Re: [PATCHv2 3/3] io: add a GRUB_GZ prefix to gzio specific defines

2017-08-07 Thread Vladimir &#x27;phcoder' Serbinenko
Committed Le Mon, Aug 7, 2017 à 6:23 PM, Pete Batard a écrit : > As requested, this new proposal uses GRUB_GZ_ instead of GZ_ as the new > prefix. > > Regards, > > /Pete > ___ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailm

Re: grub2-probe confused by double bind mount?

2017-08-08 Thread Vladimir &#x27;phcoder' Serbinenko
Did you mount /proc ? What is the contents of /proc/self/mountinfo ? On Tue, Aug 8, 2017, 06:35 Chris Murphy wrote: > On Mon, Aug 7, 2017 at 9:09 AM, Vladimir 'phcoder' Serbinenko > wrote: > > Upstream doesn't use grub2 prefix. Please try with latest upstream and &g

Re: [PATCH] unix exec: avoid atexit handlers when child exits

2017-08-30 Thread Vladimir &#x27;phcoder' Serbinenko
Comitted, thanks Le Mon, Aug 28, 2017 à 8:58 PM, Patrick Steinhardt a écrit : > The `grub_util_exec_redirect_all` helper function can be used to > spawn an executable and redirect its output to some files. After calling > `fork()`, the parent will wait for the child to terminate with > `waitpid(

Re: XFS in GRUB and sparse directories

2017-08-30 Thread Vladimir &#x27;phcoder' Serbinenko
Fixed, thanks Le Mon, Aug 21, 2017 à 3:37 PM, Tuomas Tynkkynen a écrit : > Hi, > > The XFS driver in GRUB doesn't seem to handle sparse blocks in directories > correctly. This can be reproduced by creating a simple XFS filesystem > with a couple of files removed from the 'middle: > > ---

Re: [PATCH] Implement checksum verification for gunzip

2017-08-30 Thread Vladimir &#x27;phcoder' Serbinenko
Committed, thanks Le Sun, May 1, 2016 à 2:33 PM, Stefan Fritsch a écrit : > This implements the crc32 check for the gzip format. Support for zlib's > adler checksum is not included, yet. > > diff --git a/contrib/grub2/grub-core/io/gzio.c > b/contrib/grub2/grub-core/io/gzio.c > index 0f2ea6b..432

Re: [PATCH v2 10/14] loader: switch arm/linux to grub_linux_kernel_header struct

2017-09-04 Thread Vladimir &#x27;phcoder' Serbinenko
Great, thanks! On Mon, Sep 4, 2017, 16:31 Leif Lindholm wrote: > On Mon, Aug 07, 2017 at 02:07:19PM +0000, Vladimir 'phcoder' Serbinenko > wrote: > > Le Thu, Aug 3, 2017 à 12:15 PM, Leif Lindholm > a > > écrit : > > > > > Use kernel header str

Re: master broken?

2017-09-05 Thread Vladimir &#x27;phcoder' Serbinenko
Repairing On Tue, Sep 5, 2017, 21:11 Leif Lindholm wrote: > Hi, > > Preparing a new prerequisite set for the ARM* EFI changes, I find > myself to test build any of the x86 targets after > > 21e4a6fa0 multiboot: disentangle multiboot and multiboot2 > > Build fails with > loader/multiboot_mbi2.c:2

Re: master broken?

2017-09-06 Thread Vladimir &#x27;phcoder' Serbinenko
_payload_eip; > > > ^~ > > / > Leif > > On Tue, Sep 05, 2017 at 07:11:32PM +, Vladimir 'phcoder' Serbinenko > wrote: > > Repairing > > > > On Tue, Sep 5, 2017, 21:11 Leif Lindholm > wrote: > > > >

Re: [PATCH] tsc: Change default tsc calibration method to EFI on EFI systems

2017-09-12 Thread Vladimir &#x27;phcoder' Serbinenko
On Wed, Sep 6, 2017, 22:21 David E. Box wrote: > On efi systems, make efi based tsc calibration the default, followed by > the pmtimer before using the pit. This prevents Grub boot failure on > newer x86 systems that power gate the pit. > Why does put code fail? I think it was changed to detect g

Re: ZFS boot environment patch

2017-09-15 Thread Vladimir &#x27;phcoder' Serbinenko
On Fri, Sep 15, 2017, 01:58 Colin Watson wrote: > On Thu, Sep 14, 2017 at 09:22:01PM +0200, Paul Lagerweij wrote: > > On Wed, 13 Sep 2017 at 09:17:42PM +0100, Colin Watson wrote: > > > > On Wed, Sep 13, 2017 at 11:02:48AM -0400, Konrad Rzeszutek Wilk > wrote: > > > > > @@ -62,9 +63,15 @@ case x"$

Re: [PATCH v2] sparc64: Add blocklist GPT support for SPARC

2017-10-12 Thread Vladimir &#x27;phcoder' Serbinenko
On Thu, Oct 12, 2017, 14:00 Daniel Kiper wrote: > On Mon, Oct 09, 2017 at 08:42:04AM -0600, Eric Snowberg wrote: > > > On Oct 9, 2017, at 5:57 AM, Daniel Kiper wrote: > > > On Fri, Oct 06, 2017 at 02:58:30PM -0600, Eric Snowberg wrote: > > >>> On Oct 6, 2017, at 8:04 AM, Daniel Kiper > wrote: >

Re: [PATCH] ls: prevent double open

2017-10-18 Thread Vladimir &#x27;phcoder' Serbinenko
On Mon, Oct 16, 2017, 22:11 Eric Snowberg wrote: > Prevent a double open. This can cause problems with some ieee1275 > devices, causing the system to hang. The double open can occur > as follows: > Why does it? Underlying firmware device should not be aware of how many times grub device is open

Re: [PATCH] ls: prevent double open

2017-10-18 Thread Vladimir &#x27;phcoder' Serbinenko
On Wed, Oct 18, 2017, 19:02 Eric Snowberg wrote: > > > On Oct 18, 2017, at 10:32 AM, Vladimir 'phcoder' Serbinenko < > phco...@gmail.com> wrote: > > > > > > > > On Mon, Oct 16, 2017, 22:11 Eric Snowberg > wrote: > > Prevent a double

Re: [PATCH] grub-core/commands/probe.c: Add bootable parameter

2017-12-12 Thread Vladimir &#x27;phcoder' Serbinenko
On Tue, Dec 12, 2017, 09:56 Carlo Caione wrote: > From: Carlo Caione > > Sometimes we are interested in checking whether an msdos partition has > the boot flag set. Introduce a new parameter to the probe command to > check this. > > Signed-off-by: Carlo Caione > --- > grub-core/commands/probe.

Re: [PATCH] grub-core/commands/probe.c: Add bootable parameter

2017-12-18 Thread Vladimir &#x27;phcoder' Serbinenko
On Wed, Dec 13, 2017, 12:30 Carlo Caione wrote: > On Tue, Dec 12, 2017 at 7:23 PM, Vladimir 'phcoder' Serbinenko > wrote: > > > > > > On Tue, Dec 12, 2017, 09:56 Carlo Caione wrote: > >> > >> From: Carlo Caione > >> > >>

Re: [PATCH] hexdump: Add parameter to save output to variable

2017-12-18 Thread Vladimir &#x27;phcoder' Serbinenko
On Thu, Dec 14, 2017, 07:00 Carlo Caione wrote: > From: Carlo Caione > > In endless we are using hexdump to read into the hiberfil.sys windows > file to detect if windows is hybernated or not. > > With this patch we introduce a new parameter to hexdump to enable the > possibility to save the out

Re: [HELP] cryptomount is slow, what is the proper way to [PATCH] libgcrypt-grub?

2018-01-26 Thread Vladimir &#x27;phcoder' Serbinenko
On Thu, 25 Jan 2018, 14:57 , wrote: > phcoder and everyone else on the list, hello. > > As many of you know, the builtin LUKS decryption in GRUB is a major feature > that enables many advanced setups, such as coreboot-based Full Disk > Encryption. > > However, it has been reported [1] the speed o

Re: [PATCH 1/2] mkimage: avoid copying relocations for sections that won't be copied.

2018-02-15 Thread Vladimir &#x27;phcoder' Serbinenko
Patch looks good. I'm sick right now. I'll commit in few days when I feel better unless someone commits it before Le mer. 31 janv. 2018 à 17:28, Peter Jones a écrit : > Some versions of gcc include a plugin called "annobin", and in some > build systems this is enabled by default. This plugin cr

Re: [PATCH 2/2] .mod files: Strip annobin annotations and .eh_frame, and their relocations

2018-02-15 Thread Vladimir &#x27;phcoder' Serbinenko
LGTM Le mer. 31 janv. 2018 à 17:28, Peter Jones a écrit : > This way debuginfo built from the .module will still include this > information, but the final result won't have the data we don't actually > need in the modules, either on-disk, loaded at runtime, or in prebuilt > images. > > Signed-of

Re: zfs preview

2009-04-26 Thread Vladimir &#x27;phcoder' Serbinenko
posting in the archives because his postings > show up under more than one name :-/ > I recently switched to another mailer and so the sender name changed from 'phcoder' to "Vladimir Serbinenko". Now I changed it to "Vladimir 'phcoder' Serbinenko" i

Re: [PATCH] Preboot support

2009-04-26 Thread Vladimir &#x27;phcoder' Serbinenko
-- >> >> ___ >> Grub-devel mailing list >> Grub-devel@gnu.org >> http://lists.gnu.org/mailman/listinfo/grub-devel >> > > > ___ > Grub-devel mailing list &g

Re: [PATCH] Split #2: parser/reader separation

2009-04-26 Thread Vladimir &#x27;phcoder' Serbinenko
standalone > source file normal/dyncmd.c and normal/autofs.c. > > -- > Bean > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > > -- Regards Vladimir 'phco

Re: [PATCH] Preboot support

2009-04-27 Thread Vladimir &#x27;phcoder' Serbinenko
Commited On Sun, Apr 26, 2009 at 4:38 PM, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > Warning fixed in the patch. If nobody objects I commit it tomorrow > > > On Wed, Apr 15, 2009 at 10:40 AM, John Stanley wrote: > >> The way it looks to me is

Re: idea: adjust conf file layout

2009-04-27 Thread Vladimir &#x27;phcoder' Serbinenko
le FS_SOURCES which will be added to tools sources in corresponding files. This is useful because different platforms provide different set of tools > > > -- > Bean > > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: [PATCH] Use symbol database to maintain module dependence

2009-04-27 Thread Vladimir &#x27;phcoder' Serbinenko
_____ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: [PATCH] Split #2: parser/reader separation

2009-04-27 Thread Vladimir &#x27;phcoder' Serbinenko
On Mon, Apr 27, 2009 at 7:51 AM, Bean wrote: > On Sun, Apr 26, 2009 at 11:50 PM, Vladimir 'phcoder' Serbinenko > wrote: > > Hello this patch breaks grub-emu. Also some files include grub/rescue.h > and > > this has to be fixed. > > Hi, > > For lar

Re: [PATCH] ACPI spoofing

2009-04-27 Thread Vladimir &#x27;phcoder' Serbinenko
ortunately now linux has an > interpreter which workarounds most known bugs and has an ability to put dsdt > on ramdisk. But it's not the case for all OS. Hence this patch > -- > > Regards > Vladimir 'phcoder' Serbinenko > -- Regards Vladimir 'phcoder'

Re: [PATCH] Split #2: parser/reader separation

2009-05-01 Thread Vladimir &#x27;phcoder' Serbinenko
ot >> > necessary for rescue prompt) >> >> Good point. >> >> -- >> Bean >> >> >> ___ >> Grub-devel mailing list >> Grub-devel@gnu.org >> http://lists.gnu.org/mailman/listinfo/grub-devel >> > &

[BUGFIX] lexer waits for space to terminate the varname

2009-05-02 Thread Vladimir &#x27;phcoder' Serbinenko
Hello. A varname may be terminated by any character which isn't in a set [A-Za-z0-9_] and not only space. Here is a fix -- Regards Vladimir 'phcoder' Serbinenko diff --git a/kern/parser.c b/kern/parser.c index e931853..feaee09 100644 --- a/kern/parser.c +++ b/kern/parser.c

Re: absence notice...

2009-05-02 Thread Vladimir &#x27;phcoder' Serbinenko
I might read this mailing list occasionally, but don't be suppriced if I > do not notice questions addressed to me. > > Thanks, > Vesa Jääskeläinen > > > _______ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: [PATCH] ACPI spoofing

2009-05-02 Thread Vladimir &#x27;phcoder' Serbinenko
commited On Mon, Apr 27, 2009 at 9:37 PM, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > Should work on EFI now > > On Sat, Apr 11, 2009 at 11:06 PM, phcoder wrote: > >> Hello, here is the patch to spoof ACPI tables. It's useful for >> de

Re: [PATCH] efiemu

2009-05-02 Thread Vladimir &#x27;phcoder' Serbinenko
commited On Mon, Apr 27, 2009 at 9:40 PM, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > Install scripts fixed. Additionally now skipped commands (efiemu_loadcore, > efiemu_pnvram) are executed automatically > > > On Wed, Apr 15, 2009 at 12:10 AM, phc

Re: [PATCH] xnu

2009-05-02 Thread Vladimir &#x27;phcoder' Serbinenko
commited On Mon, Apr 27, 2009 at 9:42 PM, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > Update. No need to initialize efiemu manually anymore > > On Wed, Apr 15, 2009 at 12:46 AM, phcoder wrote: > >> Hello. Here is my xnu patch. Tested on i386-

Re: [BUGFIX] lexer waits for space to terminate the varname

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
2) echo $hello or add the terminating > character explicitly in the transition table: > > > { GRUB_PARSER_STATE_VARNAME, GRUB_PARSER_STATE_TEXT, '/ ', 1}, > > > On Sat, May 2, 2009 at 8:02 PM, Vladimir 'phcoder' Serbinenko

Re: [PATCH] FreeBSD 64-bit kernel support

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
/boot/device.hints > set FreeBSD.vfs.root.mountfrom=ufs:/dev/ad0s1a > boot > > Test successfully on FreeBSD 7.1 amd64. > > -- > Bean > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinf

Re: status grub2 port of grub-legasy map command

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
(hd1) aren't swapped in grub2 > > -- > -- Lazy, Oblivious, Recurrent Disaster -- Habbit > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: [PATCH] xnu

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
On Sun, May 3, 2009 at 6:32 AM, Bean wrote: > Hi, > > xnu usage is quite complicated, perhaps you could write a wiki page on it. > On it > > On Sun, May 3, 2009 at 7:20 AM, Vladimir 'phcoder' Serbinenko > wrote: > > commited > > > > O

[PATCH] hfs+ uuid

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
This is a patch to support UUIDs on HFS+. MD5 code is copied from Michael Gorven's patch which is copied from libgcrypt nearly verbatim. Thanks for Cris for the info about how xnu expects UUID to be -- Regards Vladimir 'phcoder' Serbinenko diff --git a/commands/xnu_uuid.c b/comm

Re: [BUGFIX] lexer waits for space to terminate the varname

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
On Sun, May 3, 2009 at 2:37 PM, Bean wrote: > Hi, > > On Sun, May 3, 2009 at 5:12 PM, Vladimir 'phcoder' Serbinenko > wrote: > > Hello > > > > On Sat, May 2, 2009 at 4:08 PM, Bean wrote: > >> > >> Hi, > >> > &g

Re: [BUGFIX] lexer waits for space to terminate the varname

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
On Sun, May 3, 2009 at 2:52 PM, Bean wrote: > On Sun, May 3, 2009 at 8:39 PM, Vladimir 'phcoder' Serbinenko > wrote: > > > > > > On Sun, May 3, 2009 at 2:37 PM, Bean wrote: > >> > >> Hi, > >> > >> On Sun, May 3, 20

Re: [PATCH] Use symbol database to maintain module dependence

2009-05-03 Thread Vladimir &#x27;phcoder' Serbinenko
On Mon, Apr 27, 2009 at 9:58 PM, Bean wrote: > On Tue, Apr 28, 2009 at 2:13 AM, Vladimir 'phcoder' Serbinenko > wrote: > > symdb code seems to duplicate your list code. Perhaps you could reuse the > > file from kernel thus making maintaining easier > > Hi, >

Re: [PATCH] hfs+ uuid

2009-05-04 Thread Vladimir &#x27;phcoder' Serbinenko
commited On Mon, May 4, 2009 at 3:22 PM, Pavel Roskin wrote: > On Mon, 2009-05-04 at 15:08 +0200, Vladimir 'phcoder' Serbinenko wrote: > > > Setting two environment variables is undocumented. I think > > rd_string > > should not be

Re: [PATCH] hfs+ uuid

2009-05-04 Thread Vladimir &#x27;phcoder' Serbinenko
On Sun, May 3, 2009 at 6:55 PM, Pavel Roskin wrote: > On Sun, 2009-05-03 at 12:42 +0200, Vladimir 'phcoder' Serbinenko wrote: > > This is a patch to support UUIDs on HFS+. MD5 code is copied from > > Michael Gorven's patch which is copied from libgcrypt nearly ver

Re: truecrypt support in grub ?

2009-05-04 Thread Vladimir &#x27;phcoder' Serbinenko
cond best is the one containing detailed info with it Anyway this discussion goes away from grub so please don't continue it here. -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: grub-dumpbios

2009-05-05 Thread Vladimir &#x27;phcoder' Serbinenko
APL01,EDID" length: 128 bytes: 00 ff ff ff ff ff ff 00 06 10 73 9c 01 01 01 01 10 11 01 03 80 1d 12 78 0a 90 b5 99 58 52 8e 26 1e 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01 01 01 01 01 01 01 52 1c 00 a0 50 20 17 30 30 20 36 00 1e b3 10 00 00 18 00 00 00 01 0

Re: grub-dumpbios

2009-05-05 Thread Vladimir &#x27;phcoder' Serbinenko
On Tue, May 5, 2009 at 2:23 PM, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > > > > ... as Stefan points out (thanks Stefan) this may not be so >> > straightforwarded. I don't think this kind of tweaking is suitable >> &

Re: grub-dumpbios

2009-05-05 Thread Vladimir &#x27;phcoder' Serbinenko
On Tue, May 5, 2009 at 4:36 PM, Peter Cros wrote: > > > On Tue, May 5, 2009 at 10:25 PM, Vladimir 'phcoder' Serbinenko < > phco...@gmail.com> wrote: > >> >> >> Forgot the most important question: does it help in any way to generate a >> suit

Re: grub-install --root-directory=/mnt /dev/sda1 fails

2009-05-06 Thread Vladimir &#x27;phcoder' Serbinenko
> option relative to it. > > Does anyone have a better idea? Don't we already have a function which transforms host directory into grub directory? AFAIR we have. > > -- > Felix Zielcke > > > > ___________ > Grub-devel mailing li

Re: Grub Scripting

2009-05-08 Thread Vladimir &#x27;phcoder' Serbinenko
gt; > Thanks! > > -- > Chris Umphress <http://www.epicvoyage.org> > > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards Vladimir 'phcoder' Serbinenko __

Re: status grub2 port of grub-legasy map command

2009-05-09 Thread Vladimir &#x27;phcoder' Serbinenko
he patch. As you suggested, "grub_symbol_t" > was replaced with "void". Also, drivemap.h no longer exists, its little > content integrated into drivemap.c. Last but not least, I've mostly > adopted your version of the assembly file (indenting, labe

[PATCH] trampoline for linux on 64-bit systems

2009-05-09 Thread Vladimir &#x27;phcoder' Serbinenko
Hello. On 64-bit systems the code may be loaded above 4GiB. When grub tries to switch to 32-bit mode before launching linux it causes the reboot. Here is a patch to make this part of code into a trampoline -- Regards Vladimir 'phcoder' Serbinenko diff --git a/ChangeLog b/ChangeLog ind

Re: [PATCH] gptsync

2009-05-09 Thread Vladimir &#x27;phcoder' Serbinenko
ion type for hfs+ isn't >>> > autodetected, I will fix this, just I thought it ws unnecessary AFAIK >>> every >>> > OS which understands HFS+ understands GPT. >>> >>> At least Mac OS X 10.3 and earlier don't understand GPT, and it ac

Re: status grub2 port of grub-legasy map command

2009-05-09 Thread Vladimir &#x27;phcoder' Serbinenko
Hello 2009/5/9 Javier Martín > El sáb, 09-05-2009 a las 11:17 +0200, Vladimir 'phcoder' Serbinenko > escribió: > > +/* Realmode far ptr (2 * 16b) to the previous INT13h handler. */ > > +extern grub_uint32_t grub_drivemap_int13_oldhandler; > > I prefer it to

Re: status grub2 port of grub-legasy map command

2009-05-10 Thread Vladimir &#x27;phcoder' Serbinenko
isk and ata > should be fixed to avoid these kind of bugs. > It's more difficult then this. The bugs can be in BIOS. Again I think we need a third opinion > > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: [PATCH] Make grub2 compile with Apple's gcc

2009-05-11 Thread Vladimir &#x27;phcoder' Serbinenko
e: util/i386/efi/grub-mkimage.c: line 104: strcpy (kernel_image + sizeof (Elf_Ehdr) + GRUB_KERNEL_MACHINE_PREFIX, prefix); > > Attempt to compile macho2img.c gave error, but is this needed for x86_64? > > On Mon, May 11, 2009 at 2:46 AM, Vladimir 'phcoder' Serbinenko > wrote:

Re: [PATCH] Make grub2 compile with Apple's gcc

2009-05-11 Thread Vladimir &#x27;phcoder' Serbinenko
On Mon, May 11, 2009 at 6:30 PM, Vladimir 'phcoder' Serbinenko wrote: > On Mon, May 11, 2009 at 12:48 PM, Peter Cros wrote: >> >> Hi, >> patches were applied with grub2 rev 2202, compiles and builds grub.efi in >> Mac OSX 10.5.6 , i686-apple-darwin9-gc

Re: [PATCH] Make grub2 compile with Apple's gcc

2009-05-11 Thread Vladimir &#x27;phcoder' Serbinenko
improvement for 64-bit apple compiler > > On Mon, May 11, 2009 at 6:45 PM, Vladimir 'phcoder' Serbinenko > wrote: >> On Mon, May 11, 2009 at 6:30 PM, Vladimir 'phcoder' Serbinenko >> wrote: >>> On Mon, May 11, 2009 at 12:48 PM, Peter Cros wrote

Re: status grub2 port of grub-legasy map command

2009-05-13 Thread Vladimir &#x27;phcoder' Serbinenko
; > Removed INT13H_REBASE and INT13H_TONEWADDR, as they were used only once. > > Comments have been improved. > > -- > Regards, > Pavel Roskin > > > ___________ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel

Re: status grub2 port of grub-legasy map command

2009-05-13 Thread Vladimir &#x27;phcoder' Serbinenko
t; > -- > Regards, > Pavel Roskin > > > ___________ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

Re: status grub2 port of grub-legasy map command

2009-05-14 Thread Vladimir &#x27;phcoder' Serbinenko
On Thu, May 14, 2009 at 4:03 PM, Pavel Roskin wrote: > On Thu, 2009-05-14 at 08:49 +0200, Vladimir 'phcoder' Serbinenko wrote: >> Hello, I had two clear oppositions which weren't resolved. I don't >> believe that merge patches screwing up the pendin opp

Re: status grub2 port of grub-legasy map command

2009-05-14 Thread Vladimir &#x27;phcoder' Serbinenko
r isn't well defined. > -- > Regards, > Pavel Roskin > > > ___ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel > -- Regards Vladimir 'phcoder' Serbinenko ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel

<    1   2   3   4   5   6   7   8   9   10   >