From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH] a new UniCore32 arch-dependent patch for linux-2.6.37-rc1 Date: Tue, 30 Nov 2010 16:26:42 +0100 Message-ID: <201011301626.42547.arnd@arndb.de> References: <00a701cb8883$649d39f0$2dd7add0$@mprc.pku.edu.cn> <201011291735.19556.arnd@arndb.de> <009b01cb9079$6605cd90$321168b0$@mprc.pku.edu.cn> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from moutng.kundenserver.de ([212.227.17.9]:60099 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753402Ab0K3P0n (ORCPT ); Tue, 30 Nov 2010 10:26:43 -0500 In-Reply-To: <009b01cb9079$6605cd90$321168b0$@mprc.pku.edu.cn> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Guan Xuetao Cc: 'Greg KH' , 'Andrew Morton' , 'Linus Torvalds' , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org 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. 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 ;-) Arnd