linux-kernel - Re: [tip:core/iommu] x86/gart: Do not select AGP for GART

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>] [day] [month] [year] [list]

Message-Id: <200909081227.34725.pavel@pavlinux.ru>
Date:	Tue, 8 Sep 2009 12:27:25 +0400
From:	Pavel Vasilyev <pavel@...linux.ru>
To:	linux-kernel@...r.kernel.org, mingo@...hat.com, hpa@...or.com,
	joerg.roedel@....com, tglx@...utronix.de
Subject: Re: [tip:core/iommu] x86/gart: Do not select AGP for GART_IOMMU

On Friday 04 September 2009 17:17:19 tip-bot for Pavel Vasilyev wrote:
> Commit-ID:  6ac162d6c01ac7626f46c68c0770556cf682ce34
> Gitweb:    
> http://git.kernel.org/tip/6ac162d6c01ac7626f46c68c0770556cf682ce34 Author: 
>    Pavel Vasilyev <pavel@...linux.ru>
> AuthorDate: Thu, 3 Sep 2009 16:20:55 +0200
> Committer:  Joerg Roedel <joerg.roedel@....com>
> CommitDate: Thu, 3 Sep 2009 16:20:55 +0200
>
> x86/gart: Do not select AGP for GART_IOMMU
>
> There is no dependency from the gart code to the agp code.
> And since a lot of systems today do not have agp anymore
> remove this dependency from the kernel configuration.
>
> Signed-off-by: Pavel Vasilyev <pavel@...linux.ru>
> Signed-off-by: Joerg Roedel <joerg.roedel@....com>
>
>
> ---
>  arch/x86/Kconfig |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index 13ffa5d..1d9c18a 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -586,7 +586,6 @@ config GART_IOMMU
>  	bool "GART IOMMU support" if EMBEDDED
>  	default y
>  	select SWIOTLB
> -	select AGP
>  	depends on X86_64 && PCI
>  	---help---
>  	  Support for full DMA access of devices with 32bit memory access only


But what about the second part?


diff --git a/drivers/char/agp/Kconfig b/drivers/char/agp/Kconfig
index ccb1fa8..2fb3a48 100644
--- a/drivers/char/agp/Kconfig
+++ b/drivers/char/agp/Kconfig
@@ -56,9 +56,8 @@ config AGP_AMD
          X on AMD Irongate, 761, and 762 chipsets.

 config AGP_AMD64
-       tristate "AMD Opteron/Athlon64 on-CPU GART support" if !GART_IOMMU
+      tristate "AMD Opteron/Athlon64 on-CPU GART support"
        depends on AGP && X86
-       default y if GART_IOMMU
        help
          This option gives you AGP support for the GLX component of
          X using the on-CPU northbridge of the AMD Athlon64/Opteron CPUs.

Download attachment "signature.asc " of type "application/pgp-signature" (195 bytes)