Re: [PATCH 6/7] setup.c: clean up initrd related code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 19 Oct 2006 12:51:27 +0200, Franck Bui-Huu <vagabon.xyz@xxxxxxxxx> wrote:
> > If we passed a XKPHYS address to "rd_start=" option.  Bad usage :-)
> 
> ok so testing initrd_start against PAGE_OFFSET (instead of XKPHYS)
> is good check since we catch such bad usages. Do you agree ?

Yes.

> > It's wrong indeed.  But I can not see good way to handle such terrible
> > usage.  So ... let's ignore it.  I'm OK, are you ?
> 
> why do we need to handle them anyway ?
> 
> PAGE_OFFSET in XKPHYS means that the kernel runs in XKPHYS address
> space. We allow at boot time kernel address to be in CKSEG0 because
> we have a good reason to handle that. It allows to get a kernel
> smaller and faster at compile time.
> 
> PAGE_OFFSET in CKSEG0 means that the kernel runs in CKSEG0 address
> space. This means also that kernel can't handle XKPHYS address. But
> how would the kernel get addresses in XKPHYS (except user bad usages) ?

Sure.  No reason.  Excuse me for such a bad example ;)

---
Atsushi Nemoto


[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux