Re: [PATCH] Drop gnulib fix-base64.patch

2021-11-23 Thread Daniel Axtens
Robbie Harwood writes: > Originally added in 9fbdec2f6b4fa8b549daa4d49134d1fe89d95ef9 and > subsequently modified in 552c9fd08122a3036c724ce96dfe68aa2f75705f, > fix-base64.patch handled two problems we have using gnulib, which are > exerciesd by the base64 module but not directly caused by it. >

Re: [PATCH 0/2] powerpc-ieee1275: support larger core.elf images

2021-11-23 Thread John Paul Adrian Glaubitz
Hello Daniel! On 11/16/21 04:42, Daniel Axtens wrote: > Daniel Axtens (2): > powerpc-ieee1275: load grub at 8MB, not 2MB > ieee1275: set real-base in the PowerPC IEEE1275 Note to 32MB I have applied these now and tested them on my iBook G4 but that introduced the following regression: root@i

Re: [PATCH] Drop gnulib fix-base64.patch

2021-11-23 Thread Robbie Harwood
Daniel Axtens writes: > Robbie Harwood writes: > >> +/* For gnulib's base64 code. */ >> +#define _GL_ATTRIBUTE_CONST /* empty */ > > Do we support any compiler so old or configuration so weird that we > can't simply use 'const' here? Unfortunately it's not quite that simple. _GL_ATTRIBUTE_CONS

Re: [RESEND v3 0/3] use confidential computing provisioned secrets for disk decryption

2021-11-23 Thread Daniel Kiper
On Thu, Nov 18, 2021 at 12:15:55PM -0500, James Bottomley wrote: > On Thu, 2021-11-18 at 15:49 +0100, Daniel Kiper wrote: > > Hey, > > > > Adding Denis, Patrick and Glenn... > > > > James, please add them to the loop next time. > > Sure ... is there some way of telling who should be cc'd (I'm not a

Re: [PATCH] Drop gnulib fix-base64.patch

2021-11-23 Thread Daniel Kiper
CC-ing Daren, Patrick and Vladimir... On Tue, Nov 23, 2021 at 11:08:55AM -0500, Robbie Harwood wrote: > Daniel Axtens writes: > > > Robbie Harwood writes: > > > >> +/* For gnulib's base64 code. */ > >> +#define _GL_ATTRIBUTE_CONST /* empty */ > > > > Do we support any compiler so old or configur

Re: [PATCH 1/2] efinet: correct closing of SNP protocol

2021-11-23 Thread Daniel Kiper
First of all, sorry for late reply but I am busy... On Wed, Oct 06, 2021 at 10:30:42AM +0200, Heinrich Schuchardt wrote: > In the context of the implementation of the EFI_LOAD_FILE2_PROTOCOL for > the initial ramdisk it was observed that opening the SNP protocol failed. > https://lists.gnu.org/arc

Re: [PATCH 2/2] efi: library function grub_efi_close_protocol()

2021-11-23 Thread Daniel Kiper
On Wed, Oct 06, 2021 at 10:30:43AM +0200, Heinrich Schuchardt wrote: > Create a library function for CloseProtocol() and use it for the SNP > driver. > > Signed-off-by: Heinrich Schuchardt > --- > grub-core/kern/efi/efi.c | 18 ++ > grub-core/net/drivers/efi/efinet.c |

Re: [PATCH 0/4] Documentation improvements and fixes

2021-11-23 Thread Daniel Kiper
On Fri, Nov 05, 2021 at 02:12:54PM -0500, Glenn Washburn wrote: > These patches are pretty self explanatory. The first patch was requested by > Daniel and Daniel suggested the link to the INSTALL file from the git web > repo. > > Glenn > > Glenn Washburn (4): > docs: Add sentence on where Debian

Re: [PATCH 0/2] powerpc-ieee1275: support larger core.elf images

2021-11-23 Thread Daniel Axtens
Hi Adrian, Thanks for testing! > root@ibook-g4-14:/home/glaubitz/grub# grub-install > --macppc-directory=/boot/grub --no-nvram > Installing for powerpc-ieee1275 platform. > grub-install: warning: cannot open directory `/usr/local/share/locale': No > such file or directory. > grub-install: error

Re: [PATCH 03/19] mm: document grub internal memory management structures

2021-11-23 Thread Daniel Axtens
Glenn Washburn writes: > On Tue, 12 Oct 2021 18:29:52 +1100 > Daniel Axtens wrote: > >> I spent more than a trivial quantity of time figuring out pre_size and >> whether a memory region's size contains the header cell or not. >> >> Document the meanings of all the properties. Hopefully now no-o