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 <[email protected]> wrote: > > > > Le mar. 22 avr. 2025, 07:26, Adriano Cordova Fedeli > <[email protected]> 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 allocate something more than >> 2GiB away, probably relocation overflow. > > Is there any problem with making mcmodel large mandatory on riscv? >> >> >> On Mon, 21 Apr 2025 at 20:03, Vladimir 'phcoder' Serbinenko >> <[email protected]> wrote: >>> >>> >>> >>> Le mar. 22 avr. 2025, 01:13, Adriano Cordova Fedeli >>> <[email protected]> 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 as https://github.com/phcoder/GRUB/tree/mshared >>> _______________________________________________ >>> Grub-devel mailing list >>> [email protected] >>> https://lists.gnu.org/mailman/listinfo/grub-devel >> >> _______________________________________________ >> Grub-devel mailing list >> [email protected] >> https://lists.gnu.org/mailman/listinfo/grub-devel > > _______________________________________________ > Grub-devel mailing list > [email protected] > https://lists.gnu.org/mailman/listinfo/grub-devel _______________________________________________ Grub-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/grub-devel
