From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: =?iso-8859-1?q?=5BRFC=5D_Proposed_new_directory_layou?= =?iso-8859-1?q?t_for_kvm_and=09virtualization?= Date: Wed, 12 Dec 2007 00:14:42 +0100 Message-ID: <200712120014.43314.arnd@arndb.de> References: <475E5CBB.9080608@qumranet.com> <20071211151143.GA7751@osiris.boeblingen.de.ibm.com> <475EB23E.1060002@qumranet.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Cc: Heiko Carstens , linux-kernel , Avi Kivity To: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Return-path: In-Reply-To: <475EB23E.1060002-atKUWr5tajBWk0Htik3J/w@public.gmane.org> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org On Tuesday 11 December 2007, Avi Kivity wrote: > Heiko Carstens wrote: > > On Tue, Dec 11, 2007 at 11:47:39AM +0200, Avi Kivity wrote: > > > >> =A0arch/*/kvm/ =A0 arch dependent kvm code > >> =A0 =A0 = > > > > Maybe arch/*/virt/ ? No need to add an own directory for each hyperviso= r. > > > There will be several kvm files in that directory for x86. If you have = > just one file, I guess arch/*/virt/kvm.c is okay. In that case, even arch/*/{mm,kernel}/kvm.c would work fine, as long as you only want to support one hypervisor, or a small set of small files, there is no need to have even have a virt directory. However, if you want to have symmetry across the architectures, I think arch/*/kvm/ works out best, along the lines of = arch/*/{perfmon,crypto,xen,power}. Neither of these is particularly large, but arch/* is typically not that crowded either. Arnd <>< ------------------------------------------------------------------------- SF.Net email is sponsored by: Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php