From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752447AbaGHEMo (ORCPT ); Tue, 8 Jul 2014 00:12:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:64670 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751060AbaGHEMm (ORCPT ); Tue, 8 Jul 2014 00:12:42 -0400 Message-ID: <1404792755.4256.110.camel@ul30vt.home> Subject: Re: linux-next: build failure after merge of the iommu tree From: Alex Williamson To: Stephen Rothwell Cc: Joerg Roedel , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org Date: Mon, 07 Jul 2014 22:12:35 -0600 In-Reply-To: <20140708140246.72b72682@canb.auug.org.au> References: <20140708140246.72b72682@canb.auug.org.au> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2014-07-08 at 14:02 +1000, Stephen Rothwell wrote: > Hi Joerg, > > After merging the iommu tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > arch/powerpc/platforms/built-in.o:(.opd+0x29e8): multiple definition of `iommu_device_destroy' > arch/powerpc/kernel/built-in.o:(.opd+0x5628): first defined here > arch/powerpc/platforms/built-in.o: In function `.iommu_device_unlink': > (.text+0x13240): multiple definition of `.iommu_device_unlink' > arch/powerpc/kernel/built-in.o:(.text+0x2b240): first defined here > arch/powerpc/platforms/built-in.o: In function `.iommu_device_create': > (.text+0x13210): multiple definition of `.iommu_device_create' > arch/powerpc/kernel/built-in.o:(.text+0x2b210): first defined here > arch/powerpc/platforms/built-in.o: In function `.iommu_device_destroy': > (.text+0x13220): multiple definition of `.iommu_device_destroy' > arch/powerpc/kernel/built-in.o:(.text+0x2b220): first defined here > arch/powerpc/platforms/built-in.o: In function `.iommu_device_link': > (.text+0x13230): multiple definition of `.iommu_device_link' > arch/powerpc/kernel/built-in.o:(.text+0x2b230): first defined here > arch/powerpc/platforms/built-in.o:(.opd+0x2a00): multiple definition of `iommu_device_link' > arch/powerpc/kernel/built-in.o:(.opd+0x5640): first defined here > arch/powerpc/platforms/built-in.o:(.opd+0x2a18): multiple definition of `iommu_device_unlink' > arch/powerpc/kernel/built-in.o:(.opd+0x5658): first defined here > arch/powerpc/platforms/built-in.o:(.opd+0x29d0): multiple definition of `iommu_device_create' > arch/powerpc/kernel/built-in.o:(.opd+0x5610): first defined here > > Caused by commit c61959ecbbc6 ("iommu: Add sysfs support for IOMMUs"). > CONFIG_IOMMU_API is not set for this build. Forgotten "static > inline"s :-( Yep, I already posted a patch for this: http://lists.linuxfoundation.org/pipermail/iommu/2014-July/009123.html Sorry for the trouble. Thanks, Alex