Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <35279bb7-e986-9c90-3751-4758fab2f2e9@intel.com>
Date:   Wed, 9 Mar 2022 14:44:25 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.6.2
Subject: Re: [PATCH 4/4] intel_idle: Add SPR support with AMX INIT-state
Content-Language: en-CA
To:     Thomas Gleixner <tglx@linutronix.de>
CC:     Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        "x86@kernel.org" <x86@kernel.org>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        Peter Zijlstra <peterz@infradead.org>,
        "bp@alien8.de" <bp@alien8.de>,
        "mingo@redhat.com" <mingo@redhat.com>,
        "linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>
References: <20211104225226.5031-1-chang.seok.bae@intel.com>
 <20211104225226.5031-5-chang.seok.bae@intel.com> <878ry24qpb.ffs@tglx>
 <0B7F49C0-B850-45BC-BEC9-60DF3E2D88C5@intel.com> <8735oa4ipo.ffs@tglx>
From:   "Chang S. Bae" <chang.seok.bae@intel.com>
In-Reply-To: <8735oa4ipo.ffs@tglx>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SVlBd1lIQWpsUjhvUVcwbHFnd3FjZEZ0bjFPdTcrdzhjaDdNRjBhS0pqYnhz?=
 =?utf-8?B?OFNhS0RHQVN1T3JoVk1xTVo3VDJueXVGdXU0akYzbjA1Q2hNS0g0QW01WThu?=
 =?utf-8?B?MDQ4ODI0TjFxc1VIVWYwc2pnL3dETHN5N2dwMHNNMXlMMkZ0all4dzNOczB5?=
 =?utf-8?B?MnBwNXpOTTdEVnhYWTc3TXA3bU04OG9JUlVQa3Z5bEtLdzZtdUYySkVNdXNX?=
 =?utf-8?B?ejJpdCsrenphTlB3ai8zcjYxMVV1UFBXbjgwQTJMRk4zQ0ZlaUFQczB3WUlG?=
 =?utf-8?B?MXZsV09jenBxd2twRlBiTEpMVUoyMG5JTFJyR2pFWGJ5YWxFZjgzTXREWEVu?=
 =?utf-8?B?N3BURzBIWHpNTlVTaW5QSVNaOUx5TkJRMnNQMVRRektiNTE3THVxWFFGT05y?=
 =?utf-8?B?MFR6TDdWMkVkRnBaYlF6bnozK1R0aFZPMVp6d1c5VDU4ZmZhbTRoY2ZQM3Bu?=
 =?utf-8?B?MTBrUE9yWHdjNU1teUhIVnYxMSs5VWtnRTZUUkV3RVZ4TEZITlEzY3dic0lF?=
 =?utf-8?B?QWVOa2gwbWNCcTI3RHNuREt0aXhrZlRYeHNHMjNzM0JQUTU0WDQ3eHdKU0pr?=
 =?utf-8?B?V08yN0pDaWhsWnMrUXJyeUNKbldWc3JVcE9lWTRBTm1IQmtPVVhPUGdZVHhV?=
 =?utf-8?B?aVcxbjNlWE9HT2Q2T2M1cTlheHAwU0dBdTJzaDVFUXYxdVVpVlBBU1VqUmJ3?=
 =?utf-8?B?SWVQVG1xaTJVU1BZQXJZSkczc2Z0TzIyN0k5KzBrVTU5M0NoN0hXOHRzbGp5?=
 =?utf-8?B?UXpMRDRTb3NlcVB2QW1xRzFnMS8xaDBkR09zNEMrUlFLM3NmWXQ2bVRTYTVO?=
 =?utf-8?B?ZEk4RHNBT3BCZWRtOWJrTDJlZmF2Z2FqSDZuckRuRXVqNDNDS0xJNFlOSFQx?=
 =?utf-8?B?U0tpYW5aTm5ORW93VWlaZkMxaXNlUm1qWlk4dWgrRjg0cXB6bVB3REh3dThx?=
 =?utf-8?B?cGI2QkFENUNNTUljRUJHMWVLcTh6MUtiQ1BrWTFZMW9SWHVLcmxwdFdWSEFr?=
 =?utf-8?B?eHJCaVhqMFdYa0dMaVdyVld5Q0gwNUJKQ0N3Q2ZZcGh4eitBRUdTM1B2aWMr?=
 =?utf-8?B?SGE3S1M3L2Y4OUU5OFhvVEZSRjRVY1VSWjhKLzdkOVkxNDh6ZUQ4NGp5R09o?=
 =?utf-8?B?NGkyZjF0MXl1QTZrU1hvRGtXMThuZi9rbkxrZS91cXVLOGpPd1FYQ2Y3bHNy?=
 =?utf-8?B?a2lqT2Q1UWFub0wyQUtTUmtTbHc5V2JyTHNkaTBCamZRSEJiaEp6Rm80eHZS?=
 =?utf-8?B?di9LQ2p4ODM5ckJUVHVKdFZIU3BXbk55RmFHR0RQZTVLTDVpbXVCV3JraC92?=
 =?utf-8?B?TERmaVNGOFY1NzY3cUU1TnJqTi95WHlNTHFKeVk3QnQ5V0FHVGJGY0c3S1BC?=
 =?utf-8?B?cjVFUWxhc3ZxNWkxNTNYSzRRdmF6ZW9Bd2cwaVV3WHhZZGRQbjBMU2thS0g5?=
 =?utf-8?B?TWRlRkwxQTRIUDB3dW5RNHhYdWorM3k0NEl6cDV4Ry9UL2ZLMHlrMGc4MU50?=
 =?utf-8?B?eTl4WlF4N0pPRTJxTUwzQ2ZYRi9nUnVma3pzbHdMYlFTdE9uR2FFa0VvMUlQ?=
 =?utf-8?B?a05HNUJmMlpHdkRLS28rMDN1WmRJUzVhTnZmMUIyYkRheHJMWDRxVDlaTEdG?=
 =?utf-8?B?UjlIWHdTaEN0b2xldWRuUGRNTTIzN3oyL0QwaHdDbEovVFoyd1BHVXh4SGNK?=
 =?utf-8?B?RXdUdXdzdjk4U1I5NDNXeGtxY3o2TGwxUkhmaUhWRUJvTTYxZEs4aWpLV2ds?=
 =?utf-8?B?K0NTSEY0VUU4dUd5YmJHcXFiWTZKUUpzTXg4ZVcvT3NSMlRhT0E4WmdlZnhz?=
 =?utf-8?B?VCtaZWNweVkrck1HUVlJd0xzTGtUL005MEZGaC9XcXBBMGgrWnJRdVlVQmpi?=
 =?utf-8?B?V2VuRXdUWXl1MUo1RnZCN3hWbUdvcEo3ZXk2Z1hPdnJMamowRGpQaTdYVnlZ?=
 =?utf-8?B?Yk85c0FnL1ZFRm5yUEhXOS8wRDZHMG8zZ3RMaUs2WXBhU2FIWDJoSisyZDVO?=
 =?utf-8?B?b2VjYXk3eHM5Z2E0dklmb0JqcDQ3Tk1MTWNyZGZ0V1FuYTc0ci9rbE54UFVY?=
 =?utf-8?B?OHcwMnFvYmZmTzdMSStwb0ZURmo2cXNyODZPK0hIbjRzYlo1TnczWXFIdnNZ?=
 =?utf-8?B?U0pYWXNveDlvSWFtL2xyVmtzVjBSNjdqeUNlY0Y1RlM5dWNkV0w1TDNzSmJH?=
 =?utf-8?B?OUNSQVQwYlpUNWZPUDN1VDBsWURJZlg2UE80RFNMNkhKZmlaczI1MnB5d25G?=
 =?utf-8?Q?TSeUs9trXjYMBThojQCzGCSHyppDAW8NMYd6pkFZKY=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 47a300a0-3e51-4753-33a9-08da021e5de8
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB4855.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Mar 2022 22:44:27.5369
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 22+P3B6gZQwV/kvX/Im5h6Qx1q/Iy8Ql1PHmpJDDcAZ3ZECSebBBf7kzYw5Cu3K/XzY0ApUcd4tcAErj1UGm/oOks1IgxpoFWxXk3zu6hP8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB0046
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 11/5/2021 10:26 AM, Thomas Gleixner wrote:
> On Fri, Nov 05 2021 at 16:03, Bae, Chang Seok wrote:
> 
> My bad. I should have read the cover letter before complaining.
> 
>> I think PATCH3 is better. Maybe PATCH4 should not be sent together to avoid
>> such confusion.
> 
> Yes. patch 3 is way better than patch 4.

At the time of this post, it was clueless whether this TILERELEASE need 
is architectural or implementation-specific. But the latter is the case 
as it turns out. So PATCH4 only appears in V2 here -- 
https://lore.kernel.org/lkml/20220309223431.26560-1-chang.seok.bae@intel.com/

Thanks,
Chang