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  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]
Date:	Sun, 8 Nov 2009 13:12:32 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
Cc:	mingo@...hat.com, linux-kernel@...r.kernel.org,
	joerg.roedel@....com
Subject: Re: [PATCH] x86: use x86_platform for iommu_shutdown


* FUJITA Tomonori <fujita.tomonori@....ntt.co.jp> wrote:

> This patch cleans up pci_iommu_shutdown() a bit to use x86_platform
> (similar to how IA64 initializes an IOMMU driver).
> 
> This adds iommu_shutdown() to x86_platform to avoid calling every
> IOMMUs' shutdown functions in pci_iommu_shutdown() in order. The IOMMU
> shutdown functions are platform specific (we don't have multiple
> different IOMMU hardware) so the current way is pointless.
> 
> An IOMMU driver sets x86_platform.iommu_shutdown to the shutdown
> function if necessary.
> 
> Signed-off-by: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
> ---
>  arch/x86/include/asm/amd_iommu.h |    2 --
>  arch/x86/include/asm/gart.h      |    4 ----
>  arch/x86/include/asm/iommu.h     |    2 +-
>  arch/x86/include/asm/x86_init.h  |    1 +
>  arch/x86/kernel/amd_iommu_init.c |    6 +-----
>  arch/x86/kernel/crash.c          |    5 ++---
>  arch/x86/kernel/pci-dma.c        |    7 -------
>  arch/x86/kernel/pci-gart_64.c    |    6 ++++--
>  arch/x86/kernel/reboot.c         |    4 ++--
>  arch/x86/kernel/x86_init.c       |    2 ++
>  10 files changed, 13 insertions(+), 26 deletions(-)

Applied, thanks - this is a really nice cleanup!

	Ingo
--
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/

Powered by blists - more mailing lists