Re: [PATCH 0/2] kern/efi/mm.c: add range check on risc-v

I think the premise behind this allocator patch was that mcmodel=large does not work on risc-v, i think we are better of forcing mcmodel=large everywhere if that works. On Tue, Apr 22, 2025 at 9:01 AM Vladimir 'phcoder' Serbinenko wrote: > > > > Le mar. 22 avr. 2025, 07:26, Adriano Cordova Fedeli

Re: [PATCH 0/2] kern/efi/mm.c: add range check on risc-v

Le mar. 22 avr. 2025, 07:26, Adriano Cordova Fedeli < adriano.cord...@canonical.com> a écrit : > The series '[PATCH 0/3] Use -shared compilation instead of -Wl,-r' seems > to fix the relocation error and grub loads fine when mcmodel=large. But > still not sure what happens if mcmodel=medany and we

Re: [PATCH 0/2] kern/efi/mm.c: add range check on risc-v

The series '[PATCH 0/3] Use -shared compilation instead of -Wl,-r' seems to fix the relocation error and grub loads fine when mcmodel=large. But still not sure what happens if mcmodel=medany and we allocate something more than 2GiB away, probably relocation overflow. On Mon, 21 Apr 2025 at 20:03,

Re: [PATCH 0/2] kern/efi/mm.c: add range check on risc-v

Le mar. 22 avr. 2025, 01:13, Adriano Cordova Fedeli < adriano.cord...@canonical.com> a écrit : > Hi Vladimir, > > I just subscribed to the mailing list. Copying your 1/3 patch by hand says > it is corrupted. Do you have a git branch with that series for me to test, > or the .patch files? > Pushed

Re: [PATCH 0/2] kern/efi/mm.c: add range check on risc-v

Hi Vladimir, I just subscribed to the mailing list. Copying your 1/3 patch by hand says it is corrupted. Do you have a git branch with that series for me to test, or the .patch files? Best, Adriano On Mon, 21 Apr 2025 at 12:40, Vladimir 'phcoder' Serbinenko < phco...@gmail.com> wrote: > I belie

Re: [PATCH 0/2] kern/efi/mm.c: add range check on risc-v

I believe the problem you describe is fixed by series of patches '[PATCH 0/3] Use -shared compilation instead of -Wl,-r'. Can you try them? Regards Vladimir 'phcoder' Serbinenko Le lun. 21 avr. 2025, 18:56, Adriano Cordova a écrit : > The current grub risc-v ELF loader does not support some of

[PATCH 0/2] kern/efi/mm.c: add range check on risc-v

The current grub risc-v ELF loader does not support some of the relocations produced by -mcmodel=large (e.g. when compiled with -mcmodel=large grub-mkimage returns 'error: relocation 0x2b is not implemented yet.'), so it has to be compiled with -mcmodel=medany, which only works if the grub code fit