From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH] iommu/amd: Add workaround to propery clearing IOMMU status register Date: Tue, 9 Apr 2013 11:49:29 +0200 Message-ID: <20130409094929.GE6858@8bytes.org> References: <1365031144-2543-1-git-send-email-suravee.suthikulpanit@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1365031144-2543-1-git-send-email-suravee.suthikulpanit-5C7GfCeVMHo@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: suravee.suthikulpanit-5C7GfCeVMHo@public.gmane.org Cc: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: iommu@lists.linux-foundation.org On Wed, Apr 03, 2013 at 06:19:04PM -0500, Suthikulpanit, Suravee wrote: > In the system with multiple IOMMU,this handling scheme complicates the > synchronization of the IOMMU data structures and status registers as > there could be multiple threads competing for the same IOMMU while > the other IOMMU could be left unhandled. I really like that change in the interrupt handling code to pull the different IOMMUs independently. But its problematic to change that in the same patch that adds an erratum workaround. Please move that erratum fix into a seperate small patch that we can backport to -stable and resend. Joerg