linux-kernel - Re: [PATCH 0/2] Make cppc acpi driver aware of pcc subspace ids

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 10 Jul 2017 16:07:47 -0600
From:   "Prakash, Prashanth" <pprakash@...eaurora.org>
To:     "Rafael J. Wysocki" <rjw@...ysocki.net>,
        George Cherian <george.cherian@...ium.com>
Cc:     linux-acpi@...r.kernel.org, linux-kernel@...r.kernel.org,
        devel@...ica.org, lenb@...nel.org, jassisinghbrar@...il.com,
        robert.moore@...el.com, lv.zheng@...el.com,
        Ashwin Chaugule <ashwin.chaugule@...aro.org>,
        Ashwin Chaugule <ashwinch@...gle.com>
Subject: Re: [PATCH 0/2] Make cppc acpi driver aware of pcc subspace ids


On 6/28/2017 3:20 PM, Rafael J. Wysocki wrote:
> On Tuesday, June 13, 2017 02:17:09 PM George Cherian wrote:
>> The current cppc acpi driver works with only one pcc subspace id.
>> It maintains and registers only one pcc channel even if the acpi table has 
>> different pcc subspace ids. 
>>
>> As per ACPI 6.2 spec  all PCC registers, for all processors in the same 
>> performance domain (as defined by _PSD), must be defined to be in the same
>> subspace. The series tries to address the same by making cppc acpi driver 
>> aware of multiple possible pcc subspace ids.
>>
>> Patch 1 : In preparation to share the MAX_PCC_SUBSPACE definition with cppc acpi
>>           driver
>> Patch 2 : Make the cppc acpi driver aware of multiple pcc subspace ids.
>>
>>
>> George Cherian (2):
>>   mailbox: PCC: Move the MAX_PCC_SUBSPACES definition to header file
>>   ACPI / CPPC: Make cppc acpi driver aware of pcc subspace ids
>>
>>  drivers/acpi/cppc_acpi.c | 179 +++++++++++++++++++++++++----------------------
>>  drivers/mailbox/pcc.c    |   1 -
>>  include/acpi/pcc.h       |   1 +
>>  3 files changed, 97 insertions(+), 84 deletions(-)
> I need Ashwing and Prashanth to look at this series and let me know what they
> think.
Rafael/George,

I was on vacation last few weeks, I will go through this series this week and provide
feedback. Sorry about the delay.

--
Thanks,
Prashanth