Re: [PATCH] cmdline: pass kernel command line as verbatim

2020-04-06 Thread Olaf Hering
Am Sat, 4 Apr 2020 13:31:40 +0800 schrieb Michael Chang : > 7 files changed, 229 insertions(+), 2 deletions(-) A lot of churn. It is called 'verbatim', yet it inserts extra characters in the output buffer. I'm still stunned why there is an argument about the obvious bugfix. Good luck. Olaf

Re: [PATCH v1] http: return error on unhandled HTTP error responses

2020-04-06 Thread Olaf Hering
Am Thu, 26 Mar 2020 20:36:47 +0100 schrieb Daniel Kiper : > May I ask you to do the change for 404 and test it too? If it works > please post a new patch. I think "http" needs more surgery. It happens to work despite the (apparently) bogus error handling. But I do not have time to work on this n

Re: does grub2 support proxy dhcp?

2020-04-06 Thread Olaf Hering
Am Fri, 3 Apr 2020 17:55:41 +0200 schrieb Daniel Kiper : > Could you send me your config file with proxy DHCP? I do not have it anymore, the proxy DHCP host is gone, the main DHCP host does now also TFTP. Olaf pgpCAoMXRwXUk.pgp Description: Digitale Signatur von OpenPGP __

Re: [PATCH] cmdline: pass kernel command line as verbatim

2020-04-06 Thread Michael Chang
On Mon, Apr 06, 2020 at 09:18:26AM +0200, Olaf Hering wrote: > Am Sat, 4 Apr 2020 13:31:40 +0800 > schrieb Michael Chang : > > > 7 files changed, 229 insertions(+), 2 deletions(-) > > A lot of churn. It is called 'verbatim', yet it inserts extra characters in > the output buffer. What verbatim

Re: [PATCH 01/10] i386: Make pmtimer tsc calibration not take 51 seconds to fail

2020-04-06 Thread Javier Martinez Canillas
Hello Daniel, On 4/3/20 6:40 PM, Daniel Kiper wrote: [snip] >> >> I've also tested this outcome on a real Intel Xeon E3-1275v3 on an Intel >> Server Board S1200V3RPS using the SDV.RP.B8 "Release" build here: >> https://firmware.intel.com/sites/default/files/UEFIDevKit_S1200RP_vB8.zip >> >> Signe

Re: [PATCH 10/10] module-verifier: make it possible to run checkers on grub-module-verifierxx.c

2020-04-06 Thread Javier Martinez Canillas
Hello Daniel, On 4/3/20 9:29 PM, Daniel Kiper wrote: > On Fri, Mar 13, 2020 at 08:17:17PM +0100, Javier Martinez Canillas wrote: >> From: Peter Jones >> >> This makes it so you can treat grub-module-verifierxx.c as a file you can >> build directly, so syntax checkers like vim's "syntastic" plugin

Re: [PATCH 08/10] templates: Output a menu entry for firmware setup on UEFI FastBoot systems

2020-04-06 Thread Javier Martinez Canillas
On 4/3/20 9:05 PM, Dimitri John Ledkov wrote: > There should be a slightly more recent version of the patch which changes > the label to "UEFI Firmware Settings" which more consistent with vendor > Bios documentation, GUI tools, systemctl, and Windows 10. We are currently > trying to have that labe

Re: [PATCH 00/10] Another set of patches from Fedora

2020-04-06 Thread Javier Martinez Canillas
On 4/3/20 9:34 PM, Daniel Kiper wrote: > On Fri, Mar 13, 2020 at 08:06:32PM +0100, Javier Martinez Canillas wrote: >> Hello, >> >> This is another set of somewhat small patches that we have been carrying in >> the >> Fedora package. I found that some of them have already been posted to the >> lis

Re: [PATCHv2] multiboot2: enable quirk-modules-after-kernel

2020-04-06 Thread Paul Menzel
Dear Zide, Thank you for posting version 2 of your patch. (Please add v2 to the patch tag next time.) Maybe use the commit message summary below. multiboot2: Implement quirk-modules-after-kernel Am 03.04.20 um 09:35 schrieb Zide Chen: In multiboot2, currently there is no way to control w

RE: [PATCHv2] multiboot2: enable quirk-modules-after-kernel

2020-04-06 Thread Chen, Zide
Hi Paul, Than k you very much for your comments! Will revise the commit messages base on your comments. I have tested on VirtualBox but will test on VMWare before sending out V3. Best Regards, Zide > -Original Message- > From: Paul Menzel > Sent: Monday, April 6, 2020 1:57 PM > To: Che

[PATCH] mkimage: powerpc: fix CHRP note descsz

2020-04-06 Thread Daniel Axtens
Currently, an image generated with 'grub-mkimage -n' causes an error when read with 'readelf -a': Displaying notes found at file offset 0x000106f0 with length 0x002c: OwnerData sizeDescription readelf: Warning: note with invalid namesz and/or descsz found at offset 0x

[PATCH V3] multiboot2: Implement quirk-modules-after-kernel

2020-04-06 Thread Zide Chen
In contrast to Mulitboot, in Mulitboot2, there is currently no way to control where to load the modules to. This could be a problem, e.g., the OS loaded by Multiboot2 needs to reserve low address for some particular purposes and not for loading modules. This patch implements the quirk quirk-module