Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754654Ab2FSSsH (ORCPT ); Tue, 19 Jun 2012 14:48:07 -0400 Received: from smtp.snhosting.dk ([87.238.248.203]:29445 "EHLO smtp.domainteam.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753101Ab2FSSsF (ORCPT ); Tue, 19 Jun 2012 14:48:05 -0400 Date: Tue, 19 Jun 2012 20:48:03 +0200 From: Sam Ravnborg To: Geert Uytterhoeven Cc: linux-kbuild , Avi Kivity , Paul Gortmaker , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Linux/m68k , kvm@vger.kernel.org, Paul Mundt Subject: Re: asm/kvm_para.h from asm-generic (was: Re: linux-next: triage for April 19, 2012) Message-ID: <20120619184803.GA31656@merkur.ravnborg.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: 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: 1139 Lines: 31 > >> > >> I just noticed include/asm-generic/Kbuild.asm already had > >> > >> ifneq ($(wildcard $(srctree)/arch/$(SRCARCH)/include/asm/kvm_para.h \ > >> ? ? ? ? ? ? ? ? ?$(srctree)/include/asm-$(SRCARCH)/kvm_para.h),) > >> header-y ?+= kvm_para.h > >> endif > >> > >> but this doesn't seem to work. > >> > >> Kbuild people: which one is correct? Any use of wildcards in include/asm-generic/Kbuild.asm is bogus. We should: 1) add explicit header-y for kvm.h + a.out.h to the relevant architectures. 2) add explicit header-y for kvm_para.h for the archictectures that needs this file 3) and drop header-y / dummy file fro the others. Anything else is just papering over the initial wrong doings. I actually coded up "header-n" support to explicitly say: do not include this file for this arch - but looking at the actual problem being solved this is just bogus so I killed it off again. 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/