From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752356AbeCLIkj (ORCPT ); Mon, 12 Mar 2018 04:40:39 -0400 Received: from mail-wr0-f169.google.com ([209.85.128.169]:39478 "EHLO mail-wr0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751229AbeCLIki (ORCPT ); Mon, 12 Mar 2018 04:40:38 -0400 X-Google-Smtp-Source: AG47ELtsZQjiTz5gjx/1iXUsT5o8INB9WUlP75YE/UdJr34xhjLL6b811MZFoDcMFIo9nXxF4d8BXQ== Date: Mon, 12 Mar 2018 09:40:33 +0100 From: Ingo Molnar To: David Wang Cc: "rjw@rjwysocki.net" , "len.brown@intel.com" , "pavel@ucw.cz" , "tglx@linutronix.de" , "mingo@redhat.com" , "hpa@zytor.com" , "x86@kernel.org" , "linux-pm@kernel.org" , "linux-kernel@vger.kernel.org" , "Bruce Chang (VAS)" , "Cooper Yan(BJ-RD)" , "Qiyuan Wang(BJ-RD)" , Benjamin Pan , Luke Lin , "Tim Guo(BJ-RD)" , "Cobe Chen(BJ-RD)" , "Jiangbo Wang(BJ-RD)" Subject: Re: =?utf-8?B?562U5aSNOiBbUEFUQ0g=?= =?utf-8?Q?=5D?= Delete some unusefull operations for centaur CPU/platform Message-ID: <20180312084033.lignixrpg4ruoxbq@gmail.com> References: <1519963908-4359-1-git-send-email-davidwang@zhaoxin.com> <20180304093146.rujp4f62fv6xotb7@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * David Wang wrote: > [David] pr->flags.has_cst means BIOS define valid C state table. And at lease > define 2 entries. On all centaur platform which support C3, this condition is > always true. > [David] Just as the following comment said, we need not execute WBINVD and > ARB_DISABLE/ARB_ENABLE when entering C3 if bm_check=1 and bm_control=0. This > logic is valid for all platform not only for Centaur, I think. Ok, fair enough! Thanks, Ingo