Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753789AbYJDOSd (ORCPT ); Sat, 4 Oct 2008 10:18:33 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752969AbYJDOSZ (ORCPT ); Sat, 4 Oct 2008 10:18:25 -0400 Received: from mga11.intel.com ([192.55.52.93]:36697 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752960AbYJDOSY convert rfc822-to-8bit (ORCPT ); Sat, 4 Oct 2008 10:18:24 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.33,360,1220252400"; d="scan'208";a="387530249" From: "Yu, Fenghua" To: David Woodhouse CC: Bjorn Helgaas , "Luck, Tony" , Jesse Barnes , Ingo Molnar , Avi Kivity , Stephen Rothwell , Andrew Morton , LKML , "linux-ia64@vger.kernel.org" Date: Sat, 4 Oct 2008 07:17:53 -0700 Subject: RE: [PATCH 2/2]Add Variable Page Size and IA64 Support in Intel IOMMU: IA64 Specific Part Thread-Topic: [PATCH 2/2]Add Variable Page Size and IA64 Support in Intel IOMMU: IA64 Specific Part Thread-Index: Ackl5+HUt8L9tt7nRaGn76xCd5YlnQAQFDPQ Message-ID: References: <20081001165750.GA21272@linux-os.sc.intel.com> <200810020951.08408.bjorn.helgaas@hp.com> <200810030941.42800.bjorn.helgaas@hp.com> <1223100597.30832.37.camel@macbook.infradead.org> In-Reply-To: <1223100597.30832.37.camel@macbook.infradead.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1427 Lines: 24 >> VT-d hardware supports both non cache coherency and cache coherency by >> bit Coherency in Extended Capabilities Register. > >But is the version without the cache coherency actually going to be >_seen_ on IA64? > Currently there is only one IA64 platform CraterLake supporting VT-d. Its BIOS sets cache coherency bit. But since there is no architecture spec saying cache coherency is required for all ia64 platforms, non cache coherency could be seen in future platforms having VT-d. And one VT-d architect explicitly tell me that ia64 Linux IOMMU needs to deal with non cache coherency case and flush cache line for non cache coherency case (just like x86-64 is doing). >> Could you please point me to the doc that explicitly says that >> architecturally ia64 doesn't need cache flush? > >For safety, we can always make the driver just refuse to initialise on >IA64 if the cache coherency bit isn't set. The current patch set works just fine for both cache coherency and non cache coherency. We don't need to abandon non cache coherency support on ia64 unless there is explicit spec claiming that non cache coherency is a requirement on all ia64 platforms. Thanks. -Fenghua -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/