linux-kernel - Re: Alpha: Compilation error 2.6.34-rc1 git a3d3203e4b

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <b4198de61003151450q67a24686pcc335b2e62923794@mail.gmail.com>
Date:	Mon, 15 Mar 2010 17:50:24 -0400
From:	Matt Turner <mattst88@...il.com>
To:	FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
Cc:	mcree@...on.net.nz, linux-kernel@...r.kernel.org,
	linux-alpha@...r.kernel.org, akpm@...ux-foundation.org
Subject: Re: Alpha: Compilation error 2.6.34-rc1 git a3d3203e4b

On Mon, Mar 15, 2010 at 6:23 AM, FUJITA Tomonori
<fujita.tomonori@....ntt.co.jp> wrote:
> On Mon, 15 Mar 2010 20:08:08 +1300
> Michael Cree <mcree@...on.net.nz> wrote:
>
>> I note that for_each_sg is defined in include/linux/scatterlist.h, which
>> arch/alpha/include/asm/dma-mapping.h attempts to load, however
>> include/linux/scatterlist.h was the one that included (indirectly)
>> arch/alpha/include/asm/dma-mapping.h.  There is therefore a complete
>> loop of includes from include/linux/scatterlist.h back to itself.
>
> Very sorry about that. Seems that I broke CONFIG_ALPHA_{TSUNAMI,
> TITAN, RAWHIDE, MARVEL}.
>
> =
> From: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
> Subject: [PATCH] alpha: fix compile errors in dma-mapping-common.h
>
> The commit c186caca3dbe7f44da624cb4f9d78e1b1dfb13b8 leads to the
> following errors with CONFIG_ALPHA_{TSUNAMI, TITAN, RAWHIDE, MARVEL}:
>
> /home/fujita/git/linux-2.6/include/asm-generic/dma-mapping-common.h: In function `dma_map_sg_attrs':
> /home/fujita/git/linux-2.6/include/asm-generic/dma-mapping-common.h:49: error: implicit declaration of function `for_each_sg'
> /home/fujita/git/linux-2.6/include/asm-generic/dma-mapping-common.h:50: error: syntax error before "kmemcheck_mark_initialized"
>
> Signed-off-by: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
> Reported-by: Michael Cree <mcree@...on.net.nz>
> ---
>  arch/alpha/include/asm/core_marvel.h  |    1 -
>  arch/alpha/include/asm/core_mcpcia.h  |    1 -
>  arch/alpha/include/asm/core_titan.h   |    1 -
>  arch/alpha/include/asm/core_tsunami.h |    1 -
>  4 files changed, 0 insertions(+), 4 deletions(-)
>
> diff --git a/arch/alpha/include/asm/core_marvel.h b/arch/alpha/include/asm/core_marvel.h
> index 30d55fe..dad300f 100644
> --- a/arch/alpha/include/asm/core_marvel.h
> +++ b/arch/alpha/include/asm/core_marvel.h
> @@ -12,7 +12,6 @@
>  #define __ALPHA_MARVEL__H__
>
>  #include <linux/types.h>
> -#include <linux/pci.h>
>  #include <linux/spinlock.h>
>
>  #include <asm/compiler.h>
> diff --git a/arch/alpha/include/asm/core_mcpcia.h b/arch/alpha/include/asm/core_mcpcia.h
> index acf55b4..21ac533 100644
> --- a/arch/alpha/include/asm/core_mcpcia.h
> +++ b/arch/alpha/include/asm/core_mcpcia.h
> @@ -6,7 +6,6 @@
>  #define MCPCIA_ONE_HAE_WINDOW 1
>
>  #include <linux/types.h>
> -#include <linux/pci.h>
>  #include <asm/compiler.h>
>
>  /*
> diff --git a/arch/alpha/include/asm/core_titan.h b/arch/alpha/include/asm/core_titan.h
> index a17f6f3..8cf79d1 100644
> --- a/arch/alpha/include/asm/core_titan.h
> +++ b/arch/alpha/include/asm/core_titan.h
> @@ -2,7 +2,6 @@
>  #define __ALPHA_TITAN__H__
>
>  #include <linux/types.h>
> -#include <linux/pci.h>
>  #include <asm/compiler.h>
>
>  /*
> diff --git a/arch/alpha/include/asm/core_tsunami.h b/arch/alpha/include/asm/core_tsunami.h
> index 58d4fe4..8e39ecf 100644
> --- a/arch/alpha/include/asm/core_tsunami.h
> +++ b/arch/alpha/include/asm/core_tsunami.h
> @@ -2,7 +2,6 @@
>  #define __ALPHA_TSUNAMI__H__
>
>  #include <linux/types.h>
> -#include <linux/pci.h>
>  #include <asm/compiler.h>
>
>  /*
> --
> 1.7.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Thanks for the fix. Will get this into my tree and to Linus ASAP.

Matt
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/