Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754059AbYJDAVt (ORCPT ); Fri, 3 Oct 2008 20:21:49 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753382AbYJDAVl (ORCPT ); Fri, 3 Oct 2008 20:21:41 -0400 Received: from mga03.intel.com ([143.182.124.21]:29168 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753301AbYJDAVk convert rfc822-to-8bit (ORCPT ); Fri, 3 Oct 2008 20:21:40 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.33,359,1220252400"; d="scan'208";a="54837838" From: "Yu, Fenghua" To: Bjorn Helgaas CC: "Luck, Tony" , Jesse Barnes , David Woodhouse , Ingo Molnar , Avi Kivity , Stephen Rothwell , Andrew Morton , LKML , "linux-ia64@vger.kernel.org" Date: Fri, 3 Oct 2008 17:21:36 -0700 Subject: RE: [PATCH 1/2]Add Variable Page Size and IA64 Support in Intel IOMMU: Generic Part Thread-Topic: [PATCH 1/2]Add Variable Page Size and IA64 Support in Intel IOMMU: Generic Part Thread-Index: AcklbV5MAZpOn9dVQ12fpl9B/z8RpQASNXHw Message-ID: References: <200810020931.48952.bjorn.helgaas@hp.com> <200810030933.10627.bjorn.helgaas@hp.com> In-Reply-To: <200810030933.10627.bjorn.helgaas@hp.com> 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: 1355 Lines: 27 >> >> The current Intel IOMMU code assumes that both host page size and Intel IOMMU page size are 4K. The first patch supports variable page size. This provides support for IA64 which has multiple page sizes. >> >> > >> This patch also adds some other code hooks for IA64 platform including DMAR_OPERATION_TIMEOUT definition, . >> >> >Can you split this patch up? It contains several logically separate >> changes: >> > - casting things to unsigned long long >> > - adding stuff under #ifdef CONFIG_IA64 >> > - page size changes >> > - whitespace changes >> >> Depends on who is picking up the generic patch. If it's needed, I can >split it into multiple patches. >Regardless of who's picking up the patch, splitting it makes it >easier to review and easier to spot bugs, and makes bisection yield >better information. Since more than 95% of code in the patch is directly related to page size changes, splitting the patch will generate one big patch (95% of current patch size) and a few small patches. I would like to still send one single updated patch based on collected comments. 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/