[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <00ef01cb9156$fd21d920$f7658b60$@mprc.pku.edu.cn>
Date: Wed, 1 Dec 2010 20:54:59 +0800
From: "Guan Xuetao" <guanxuetao@...c.pku.edu.cn>
To: "'Arnd Bergmann'" <arnd@...db.de>
Cc: <linux-kernel@...r.kernel.org>, <linux-arch@...r.kernel.org>
Subject: RE: [PATCH] a new UniCore32 arch-dependent patch for linux-2.6.37-rc1
> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd@...db.de]
> Sent: Tuesday, November 30, 2010 11:27 PM
> To: Guan Xuetao
> Cc: 'Greg KH'; 'Andrew Morton'; 'Linus Torvalds'; linux-
> kernel@...r.kernel.org; linux-arch@...r.kernel.org
> Subject: Re: [PATCH] a new UniCore32 arch-dependent patch for linux-
> 2.6.37-rc1
>
> On Tuesday 30 November 2010, Guan Xuetao wrote:
> > > On Saturday 27 November 2010, Guan Xuetao wrote:
> > > > >
> > > > > In what way does the architecture enforce this? What are the
> > > > > contents of this page? Can you make it an actual VDSO rather
> > > > > than a magic page that sits in the user address space?
> > > > Page table is created for vector page and exceptions entry stub.
> > > > However, vector page is not in the user address space.
> > >
> > > Interesting. So if it's not mapped into user space, why do you even
> > > need to have the vectors at a specific page? I think ARM only maps
> > > it to the high page because that page is shared to user space, while
> > > most architectures just have their interrupt vectors in the linar
> > > mapping, since the hardware typically uses the physical address to
find it.
> >
> > In UniCore-32, when MMU enabled, vector page address is virtual address.
>
> Ok. I see.
>
> The arch_exit_mmap() and arch_setup_additional_pages() functions seems
> to refer to how ARM maps this page into user space as well, you should
> probably change that.
Vector page is also used as kuser page, so arch_setup_additional_pages need
to
Map it into user space.
>
> I've finished the second half of the code review now, I'll follow up with
an
> email but drop the personal Cc to everyone besides us, in case people are
> getting bored from the discussion by now ;-)
Thanks again.
Guan Xuetao
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists