From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753628AbZEGGNn (ORCPT ); Thu, 7 May 2009 02:13:43 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752029AbZEGGN3 (ORCPT ); Thu, 7 May 2009 02:13:29 -0400 Received: from mga01.intel.com ([192.55.52.88]:13966 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751950AbZEGGN3 (ORCPT ); Thu, 7 May 2009 02:13:29 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.40,307,1239001200"; d="scan'208";a="688212919" From: Weidong Han To: dwmw2@infradead.org, suresh.b.siddha@intel.com, mingo@elte.hu Cc: linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org, kvm@vger.kernel.org, Weidong Han Subject: [PATCH 1/2] Intel-IOMMU, intr-remap: set the whole 128bits of irte when modify/free it Date: Thu, 7 May 2009 14:16:35 +0800 Message-Id: <1241676996-27406-1-git-send-email-weidong.han@intel.com> X-Mailer: git-send-email 1.6.0.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Interrupt remapping table entry is 128bits. Currently, it only sets low 64bits of irte in modify_irte and free_irte. This ignores high 64bits setting of irte, that means source-id setting will be ignored. This patch sets the whole 128bits of irte when modify/free it. Following source-id checking patch depends on this. Signed-off-by: Weidong Han --- drivers/pci/intr_remapping.c | 10 +++++++--- 1 files changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/pci/intr_remapping.c b/drivers/pci/intr_remapping.c index f5e0ea7..946e170 100644 --- a/drivers/pci/intr_remapping.c +++ b/drivers/pci/intr_remapping.c @@ -309,7 +309,8 @@ int modify_irte(int irq, struct irte *irte_modified) index = irq_iommu->irte_index + irq_iommu->sub_handle; irte = &iommu->ir_table->base[index]; - set_64bit((unsigned long *)irte, irte_modified->low); + set_64bit((unsigned long *)&irte->low, irte_modified->low); + set_64bit((unsigned long *)&irte->high, irte_modified->high); __iommu_flush_cache(iommu, irte, sizeof(*irte)); rc = qi_flush_iec(iommu, index, 0); @@ -386,8 +387,11 @@ int free_irte(int irq) irte = &iommu->ir_table->base[index]; if (!irq_iommu->sub_handle) { - for (i = 0; i < (1 << irq_iommu->irte_mask); i++) - set_64bit((unsigned long *)(irte + i), 0); + for (i = 0; i < (1 << irq_iommu->irte_mask); i++) { + set_64bit((unsigned long *)&irte->low, 0); + set_64bit((unsigned long *)&irte->high, 0); + irte++; + } rc = qi_flush_iec(iommu, index, irq_iommu->irte_mask); } -- 1.6.0.4