[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] improve grub_mm_init_region() debug message
From: |
Pavel Roskin |
Subject: |
Re: [PATCH] improve grub_mm_init_region() debug message |
Date: |
Sat, 19 Jan 2008 21:26:15 -0500 |
On Sat, 2008-01-19 at 22:50 +0100, Robert Millan wrote:
> On Tue, Jan 15, 2008 at 07:00:55PM +0100, Robert Millan wrote:
> > On Tue, Jan 15, 2008 at 12:42:39PM -0500, Pavel Roskin wrote:
> > >
> > > On Tue, 2008-01-15 at 18:17 +0100, Robert Millan wrote:
> > > > - grub_dprintf ("mem", "Using memory for heap: addr=%p, size=%u\n",
> > > > addr, (unsigned int) size);
> > > > + grub_dprintf ("mem", "Using memory for heap: start=%p, end=%p\n",
> > > > addr, addr + size);
> > >
> > > Maybe addr+size-1 would be better? Inclusive boundaries are more
> > > intuitive. Just run "cat /proc/iomem" to see what I mean.
> >
> > Sounds better to me.
> >
> > Does this change (with addr + size -1) sound fine to everyone else?
>
> Uhm actually, since we already substract one on ieee1275 to workaround
> firmware bugs, this might get confusing.
Oh, I didn't know that.
> How about just "addr + size" ?
Fine with me.
--
Regards,
Pavel Roskin
- [PATCH] improve grub_mm_init_region() debug message, Robert Millan, 2008/01/15
- Re: [PATCH] improve grub_mm_init_region() debug message, Pavel Roskin, 2008/01/15
- Re: [PATCH] improve grub_mm_init_region() debug message, Robert Millan, 2008/01/15
- Re: [PATCH] improve grub_mm_init_region() debug message, Marco Gerards, 2008/01/21
- Re: [PATCH] improve grub_mm_init_region() debug message, Robert Millan, 2008/01/21
- Re: [PATCH] improve grub_mm_init_region() debug message, Marco Gerards, 2008/01/21
- Re: [PATCH] improve grub_mm_init_region() debug message, Robert Millan, 2008/01/21