On 24.09.2024 12:28, Frediano Ziglio wrote: > No need to have it coded in assembly. > > Signed-off-by: Frediano Ziglio <frediano.zig...@cloud.com> > --- > Changes since v1: > - update some comments; > - explain why %ebx is saved before calling efi_parse_mbi2; > - move lea before test instruction; > - removed asmlinkage from efi_multiboot2 and add to efi_parse_mbi2; > - fix line length; > - update an error message specifying "Multiboot2" instead of "Multiboot"; > - use obj-bin-X instead of obj-X in Makefile; > - avoid restoring %eax (MBI magic).
Despite this long list of changes earlier comments were left unaddressed. The new function is still named as if it did only parsing, the stub change is still in here and (if already not separated out) not mentioned at all in the description, and (as Andrew has now also pointed out) the declaration of efi_multiboot2() didn't move to a header. Maybe I forgot some more. Please make sure you address earlier comments before sending a new version. Jan