From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:6115 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753215Ab0JAGYh (ORCPT ); Fri, 1 Oct 2010 02:24:37 -0400 Message-ID: <4CA57F93.8010101@redhat.com> Date: Fri, 01 Oct 2010 14:28:35 +0800 From: Cong Wang MIME-Version: 1.0 Subject: Re: [PATCH 1/4] Fix wrong source path in scripts/namespace.pl References: <1285828138-5873-1-git-send-email-amwang@redhat.com> <20101001145618.613e3e1f@s6510> In-Reply-To: <20101001145618.613e3e1f@s6510> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Stephen Hemminger Cc: linux-kbuild@vger.kernel.org, Andrew Morton , Michal Marek , linux-kernel@vger.kernel.org On 10/01/10 13:56, Stephen Hemminger wrote: > On Thu, 30 Sep 2010 14:28:55 +0800 > Amerigo Wang wrote: > >> File::Find will do chdir automatically, so we need to get >> the absolute patch with $File::Find::dir. >> >> Reported-by: Stephen Hemminger >> Signed-off-by: Amerigo Wang >> --- >> scripts/namespace.pl | 8 ++++---- >> 1 files changed, 4 insertions(+), 4 deletions(-) > > How did you test this? I tried your patches and when I run > it on a compiled kernel in a way that most developers would > do it: > $ make allmodconfig > $ make > $ ./scripts/namespace.pl I use 'make namespacecheck'. > > With your patch it still doesn't find the kernel source files. > I find it frustrating because my patch worked, yours didn't. > > -------- > No source file found for arch/x86/crypto/aes-i586-asm_32.o > No source file found for arch/x86/crypto/aes_glue.o ... Actually, this shows it is working. :) There are two reasons why you got so many of these: 1) You didn't do a 'make mrproper', so some source file may be deleted or renamed in git repo; 2) There are no path/to/foo.c for path/to/foo.o which is normal, we should add those files to exception list. I got much fewer of such warning when I tested it.