From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: Re: [PATCH] kvm: external module: support building against Windriver 2.0 (kernel 2.6.21) Date: Thu, 12 Mar 2009 19:06:01 +0100 Message-ID: <49B94F09.90302@siemens.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Bernhard Kohl Return-path: Received: from gecko.sbs.de ([194.138.37.40]:18557 "EHLO gecko.sbs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754523AbZCLSGK (ORCPT ); Thu, 12 Mar 2009 14:06:10 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: Bernhard Kohl wrote: > This is needed to compile kvm in a Windriver 2.0 distribution > (kernel 2.6.21). This kernel has an include file marker.h, but > trace_mark is not defined there. So the compat code in > kernel/include-compat/linux/marker.h is not included. I bet this is because Wind River patched some variant of LTTng into their kernel. However, I'm unsure if supporting significantly modified distribution kernels is in the scope of this compat layer. If it is ok for the maintainers, you should try to make the test more Wind River specific (did you check that there is no side-effect for normal <= 2.6.21 kernels?) and maybe add a comment. Jan > > Signed-off-by: Bernhard Kohl > --- > kernel/external-module-compat-comm.h | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/kernel/external-module-compat-comm.h > b/kernel/external-module-compat-comm.h > index a14cea2..e40501e 100644 > --- a/kernel/external-module-compat-comm.h > +++ b/kernel/external-module-compat-comm.h > @@ -25,6 +25,10 @@ > # undef CONFIG_KVM_TRACE > #endif > > +#if LINUX_VERSION_CODE <= KERNEL_VERSION(2,6,21) > +#define trace_mark(args...) ((void)0) > +#endif > + > /* > * 2.6.16 does not have GFP_NOWAIT > */ -- Siemens AG, Corporate Technology, CT SE 2 Corporate Competence Center Embedded Linux