]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
amd-iommu: resume cleanup
authorChris Wright <chrisw@sous-sol.org>
Tue, 16 Jun 2009 07:01:37 +0000 (03:01 -0400)
committerJoerg Roedel <joerg.roedel@amd.com>
Tue, 16 Jun 2009 08:19:16 +0000 (10:19 +0200)
Now that enable_iommus() will call iommu_disable() for each iommu,
the call to disable_iommus() during resume is redundant.  Also, the order
for an invalidation is to invalidate device table entries first, then
domain translations.

Signed-off-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
arch/x86/kernel/amd_iommu_init.c

index 068a3569f8375d1fd8d863585040a23be425c132..10b2accd12ea5983d917b0646cf445f9c8fe8b93 100644 (file)
@@ -1079,12 +1079,6 @@ static void disable_iommus(void)
 
 static int amd_iommu_resume(struct sys_device *dev)
 {
-       /*
-        * Disable IOMMUs before reprogramming the hardware registers.
-        * IOMMU is still enabled from the resume kernel.
-        */
-       disable_iommus();
-
        /* re-load the hardware */
        enable_iommus();
 
@@ -1092,8 +1086,8 @@ static int amd_iommu_resume(struct sys_device *dev)
         * we have to flush after the IOMMUs are enabled because a
         * disabled IOMMU will never execute the commands we send
         */
-       amd_iommu_flush_all_domains();
        amd_iommu_flush_all_devices();
+       amd_iommu_flush_all_domains();
 
        return 0;
 }