[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [MULTIBOOT2 DOC PATCH v3 01/13] multiboot2: Replace u_phys with u32
From: |
Daniel Kiper |
Subject: |
Re: [MULTIBOOT2 DOC PATCH v3 01/13] multiboot2: Replace u_phys with u32 |
Date: |
Mon, 12 Dec 2016 14:48:49 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sat, Dec 10, 2016 at 08:23:15PM +0300, Andrei Borzenkov wrote:
> 07.12.2016 01:52, Daniel Kiper пишет:
> > u_phys is used just in two places and sometimes it may confuse reader.
> > Additionally, GRUB multiboot2 implementation does not use u_phys anywhere.
> > So, replace it with basic well defined and used in implementation u32 type.
> >
> > Signed-off-by: Daniel Kiper <address@hidden>
> > ---
> > doc/multiboot.texi | 11 ++++-------
> > 1 file changed, 4 insertions(+), 7 deletions(-)
> >
> > diff --git a/doc/multiboot.texi b/doc/multiboot.texi
> > index 4b92918..2bda9b7 100644
> > --- a/doc/multiboot.texi
> > +++ b/doc/multiboot.texi
> > @@ -299,9 +299,6 @@ little-endian, u32 is coded in little-endian.
> > The type of unsigned 64-bit data. Because the target architecture is
> > little-endian, u64 is coded in little-endian.
> >
> > address@hidden u_phys
> > -The type of unsigned data of the same size as target architecture physical
> > address size.
> > -
> > @item u_virt
> > The type of unsigned data of the same size as target architecture virtual
> > address size.
> >
>
> So if I understand it correctly, any address used in multiboot2 is
> limited to 32 bit, so anything that is relevant to boot protocol must
More or less. There are some exceptions when EFI x64 platforms come on the
stage.
It is described in spec.
> reside below 4G. Is my assumption correct?
Yep. More info you can find in patch #07.
Daniel
- [MULTIBOOT2 DOC PATCH v3 00/13] multiboot2: Update documentation, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 02/13] multiboot2: Replace u_virt with u32, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 01/13] multiboot2: Replace u_phys with u32, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 04/13] multiboot2: Replace redundant if with the, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 05/13] multiboot2: Clarify meaning of information request header tag, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 03/13] multiboot2: Rename Multiboot to Multiboot2, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 06/13] multiboot2: Fix description of EFI boot services tag, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 07/13] multiboot2: Add description of support for EFI boot services, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 10/13] multiboot2: Say that memory maps may not be available on EFI platforms, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 08/13] multiboot2: Add description of EFI image handle tags, Daniel Kiper, 2016/12/06
- [MULTIBOOT2 DOC PATCH v3 09/13] multiboot2: Add description of support for relocatable images, Daniel Kiper, 2016/12/06