Re: multiboot2.h & multiboot_header_tag_module_align

[Andrei Borzenkov <arvidjaar@gmail.com>]

В Thu, 04 Jun 2015 19:04:46 +0300
Toomas Soome <tsoome@me.com> пишет:

> 
> > On 04.06.2015, at 18:31, Vladimir 'φ-coder/phcoder' Serbinenko <phcoder@gmail.com> wrote:
> > 
> > On 04.06.2015 14:18, Toomas Soome wrote:
> >> hi!
> >> 
> >> i know this tag (multiboot_header_tag_module_align) is not really used, but it still would be nice if the copy/paste error of the actual structure would be fixed;)
> >> 
> > Can you propose a patch?
> >> rgds,
> 
> as its header tag, i think following should be it; header tags have 16bit ints for type and flags, 32bit size and based on name, this tag is to request bootloader to use specified alignment, so the data field should be 32bit uint as well (default MULTIBOOT_MOD_ALIGN is 0x00001000).
> 
> diff --git a/include/multiboot2.h b/include/multiboot2.h
> index 3ccff15..e3064da 100644
> --- a/include/multiboot2.h
> +++ b/include/multiboot2.h
> @@ -156,9 +156,7 @@ struct multiboot_header_tag_module_align
>    multiboot_uint16_t type;
>    multiboot_uint16_t flags;
>    multiboot_uint32_t size;
> -  multiboot_uint32_t width;
> -  multiboot_uint32_t height;
> -  multiboot_uint32_t depth;
> +  multiboot_uint32_t align;
What about consulting multiboot2 specification? :)

>  };
>  
>  struct multiboot_color
> 
> 
> 
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel