From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Youquan,Song" Subject: Re: [PATCH]acpi c-states: Fix ACPI C3 is wrongly mapped to C2 Date: Mon, 14 Dec 2009 05:02:00 -0500 Message-ID: <20091214100200.GA17243@youquan-linux.bj.intel.com> References: <20091212181442.GA22832@youquan-linux.bj.intel.com> <20091212142727.GA6867@comet.dominikbrodowski.net> <20091212235549.GA7753@youquan-linux.bj.intel.com> <20091213084317.GA15989@isilmar.linta.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga09.intel.com ([134.134.136.24]:36725 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750720AbZLNC0b (ORCPT ); Sun, 13 Dec 2009 21:26:31 -0500 Content-Disposition: inline In-Reply-To: <20091213084317.GA15989@isilmar.linta.de> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Dominik Brodowski Cc: "Youquan,Song" , lenb@kernel.org, venkatesh.pallipadi@intel.com, kent.liu@intel.com, chaohong.guo@intel.com, youquan.song@intel.com, linux-acpi@vger.kernel.org > > That's true, if CPU has more 3 states such as C3, C6, C7, there are > > all mapped to ACPI C3. But the current situation is that if platform > > does NOT support ACPI C2, the user interface /sys show us ACPI C2 is > > supported which actual is ACPI C3. > > > > As you know, ACPI C3 and ACPI C2 have much different, such as: BUS SNOOP > > or not, ACPI C3 has better power saving etc. we should not mix them. > > Yes, but what happens if there are two states of type C2? The whole concept > of "type C" and "state C" was broken from the beginning... Sorry. In my mind, there is no two states of ACPI C2. If processor C3 above are mapped to ACPI C3, so processor C1 is mapped to ACPI C1, processor C2 is mapped to ACPI C2. Len Brown, Am I right? If I am wrong, I will file another patches for it. -Youquan