Date: Thu, 24 May 2007 17:37:42 -0400
From: Dave Jones <davej@redhat.com>
To: Venki Pallipadi <venkatesh.pallipadi@intel.com>
Cc: Andi Kleen <ak@suse.de>, Andrew Morton <akpm@linux-foundation.org>,
       Len Brown <len.brown@intel.com>,
       linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] Display Intel Dynamic Acceleration feature in /proc/cpuinfo
Message-ID: <20070524213742.GB5380@redhat.com>
Mail-Followup-To: Dave Jones <davej@redhat.com>,
	Venki Pallipadi <venkatesh.pallipadi@intel.com>,
	Andi Kleen <ak@suse.de>, Andrew Morton <akpm@linux-foundation.org>,
	Len Brown <len.brown@intel.com>,
	linux-kernel <linux-kernel@vger.kernel.org>
References: <20070523224637.GB15887@linux-os.sc.intel.com> <200705242308.38360.ak@suse.de> <20070524211337.GB3692@linux-os.sc.intel.com> <200705242325.28011.ak@suse.de> <20070524212806.GA5249@linux-os.sc.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20070524212806.GA5249@linux-os.sc.intel.com>
User-Agent: Mutt/1.4.2.2i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 802
Lines: 21

On Thu, May 24, 2007 at 02:28:06PM -0700, Venki Pallipadi wrote:

 > > But actually reading the patch: it seems weird to detect the flag 
 > > in acpi-cpufreq and essentially change /proc/cpuinfo when a
 > > module is loaded. Why not in the intel setup function? And why is it 
 > > not in the standard CPUID 1 features mask anyways?
 > 
 > I can do it in intel setup function. But, the feature may not be activated
 > unless the driver is loaded.

The same could be said about est, vmx, and probably some others ;-)

	Dave

-- 
http://www.codemonkey.org.uk
-
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/