From mboxrd@z Thu Jan 1 00:00:00 1970 From: Izik Eidus Subject: Re: [Disscussion] Moving x86 code out of kvm.h Date: Thu, 15 Nov 2007 10:30:54 +0200 Message-ID: <1195115455.3165.6.camel@localhost.localdomain> References: <1195115055.21627.17.camel@thinkpad> <473C0494.2080404@qumranet.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel , kvm-ppc-devel , jyoung5-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org To: Avi Kivity Return-path: In-Reply-To: <473C0494.2080404-atKUWr5tajBWk0Htik3J/w@public.gmane.org> 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 Thu, 2007-11-15 at 10:34 +0200, Avi Kivity wrote: > Jerone Young wrote: > > This is a continuation of a conversation I have been having with Izik > > Eidus on IRC today. > > > > I plan on moving x86 code out of kvm.h and into kvm-x86.h. kvm-x86.h > > would then include kvm.h and would be included by files like svm.c > > instead of kvm.h. > > > > > > Which kvm.h are we talking about? include/linux/kvm.h or drivers/kvm/kvm.h? > > > Izik pointed out a big issue with this approach and that is it breaks > > compatibility with older userspace. We believe that everyone still cares > > that older userspace should be able to compile against a newer kvm > > kernels. Is this true or should we just dump it? > > > > Of course we can't break compatibility without an excellent reason. > > > Assuming this is true, we have come up with an interesting solution to > > the problem. > > > > - creation of new header name "portablekvm.h" (please give a better name > > if you know one). This would be the new place for common code that would > > normally be kvm.h. > > > > - still create header "kvm-x86.h" for x86 code and would include this > > new "portablekvm.h" header. > > > > - keep compatibility with older userspace by taking header "kvm.h" and > > jsut have it include kvm-x86.h & portablekvm.h and have no other code > > (unless it's for compatibility). > > > > What does everyone thing about this ? > > > > > > Have kvm.h include what you mean by that? ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/