Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933207AbZGPUAQ (ORCPT ); Thu, 16 Jul 2009 16:00:16 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933148AbZGPUAP (ORCPT ); Thu, 16 Jul 2009 16:00:15 -0400 Received: from pfepa.post.tele.dk ([195.41.46.235]:55436 "EHLO pfepa.post.tele.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932765AbZGPUAN (ORCPT ); Thu, 16 Jul 2009 16:00:13 -0400 Date: Thu, 16 Jul 2009 22:00:12 +0200 From: Sam Ravnborg To: Masami Hiramatsu Cc: Ingo Molnar , Steven Rostedt , lkml , systemtap , kvm , DLE , Jim Keniston , "H. Peter Anvin" , Ananth N Mavinakayanahalli , Srikar Dronamraju , Frederic Weisbecker , Andi Kleen , Vegard Nossum , Avi Kivity , Przemys??awPawe??czyk Subject: Re: [PATCH -tip -v12 01/11] x86: instruction decoder API Message-ID: <20090716200012.GB5994@merkur.ravnborg.org> Mail-Followup-To: Masami Hiramatsu , Ingo Molnar , Steven Rostedt , lkml , systemtap , kvm , DLE , Jim Keniston , "H. Peter Anvin" , Ananth N Mavinakayanahalli , Srikar Dronamraju , Frederic Weisbecker , Andi Kleen , Vegard Nossum , Avi Kivity , Przemys??awPawe??czyk References: <20090716155652.6266.39970.stgit@localhost.localdomain> <20090716155700.6266.71011.stgit@localhost.localdomain> <20090716163708.GB5804@merkur.ravnborg.org> <4A5F6356.9080505@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4A5F6356.9080505@redhat.com> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1774 Lines: 44 On Thu, Jul 16, 2009 at 01:28:54PM -0400, Masami Hiramatsu wrote: > Sam Ravnborg wrote: > >> diff --git a/arch/x86/include/asm/inat.h b/arch/x86/include/asm/inat.h > >> new file mode 100644 > >> index 0000000..01e079a > >> --- /dev/null > >> +++ b/arch/x86/include/asm/inat.h > >> @@ -0,0 +1,125 @@ > >> +#ifndef _ASM_INAT_INAT_H > >> +#define _ASM_INAT_INAT_H > > > > [With reference to comment on patch 2/12...] > > You create inat.h here. > > Could you investigave what is needed to factor out the stuff > > needed from userspace so we can avoid the ugly havk where > > you redefine types.h? > > Sorry, I'm a bit confusing. > Would you mean that I should break down user_include.h and > add those redefined types in inat.h? No - try to factor out what is needed for your program so you can avoid user_include.h entirely. > > > Maybe create a inat_types.h + inat.h as we do in other cases? > > And inat_types.h has two parts, one for kernel, and one for > userspace(which is moved from user_include.h), is that right? More like inat_types.h include pure definitions and inat.h define all the macros (that would be much nicer if expressed as static inlines). The real thing to consider is what is needed from your userspace program and is also required by the kernel. I did not event remotely try to find out - as I guess you know it. So try to isolate these bits somehow and you have then nicely dropped a lot of dependencies on the remainign headers and can thus hopefully get rid of the ugly usser_include.h hack. Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/