From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754078AbcHVKuo (ORCPT ); Mon, 22 Aug 2016 06:50:44 -0400 Received: from mx2.suse.de ([195.135.220.15]:38208 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752594AbcHVKun (ORCPT ); Mon, 22 Aug 2016 06:50:43 -0400 Date: Mon, 22 Aug 2016 12:50:32 +0200 From: Joerg Roedel To: Fabian Frederick Cc: Ingo Molnar , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/1 linux-next] iommu/amd: remove AMD_IOMMU_STATS Message-ID: <20160822105032.GG2547@suse.de> References: <1471038887-13199-1-git-send-email-fabf@skynet.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1471038887-13199-1-git-send-email-fabf@skynet.be> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 12, 2016 at 11:54:47PM +0200, Fabian Frederick wrote: > Commit e85e8f69cedb > ("iommu/amd: Remove statistics code") > removed that config. > > Signed-off-by: Fabian Frederick > --- > arch/x86/configs/x86_64_defconfig | 1 - > drivers/iommu/amd_iommu_proto.h | 4 ---- > 2 files changed, 5 deletions(-) > > diff --git a/arch/x86/configs/x86_64_defconfig b/arch/x86/configs/x86_64_defconfig > index d28bdab..7ef4a09 100644 > --- a/arch/x86/configs/x86_64_defconfig > +++ b/arch/x86/configs/x86_64_defconfig > @@ -255,7 +255,6 @@ CONFIG_RTC_CLASS=y > CONFIG_DMADEVICES=y > CONFIG_EEEPC_LAPTOP=y > CONFIG_AMD_IOMMU=y > -CONFIG_AMD_IOMMU_STATS=y > CONFIG_INTEL_IOMMU=y > # CONFIG_INTEL_IOMMU_DEFAULT_ON is not set > CONFIG_EFI_VARS=y > diff --git a/drivers/iommu/amd_iommu_proto.h b/drivers/iommu/amd_iommu_proto.h > index 0bd9eb3..697bccc 100644 > --- a/drivers/iommu/amd_iommu_proto.h > +++ b/drivers/iommu/amd_iommu_proto.h > @@ -78,12 +78,8 @@ static inline int amd_iommu_create_irq_domain(struct amd_iommu *iommu) > extern int amd_iommu_complete_ppr(struct pci_dev *pdev, int pasid, > int status, int tag); > > -#ifndef CONFIG_AMD_IOMMU_STATS > - > static inline void amd_iommu_stats_init(void) { } That function can be removed too, while you are at it. Thanks for cleaning that up. Joerg