grub-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]