linux-kernel - Re: [PATCH] mailbox: pcc: Support HW-Reduced Communication Subspace Type 2

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:	Fri, 8 Apr 2016 13:58:00 +0100
From:	Sudeep Holla <sudeep.holla@....com>
To:	hotran <hotran@....com>
Cc:	Jassi Brar <jassisinghbrar@...il.com>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Len Brown <lenb@...nel.org>,
	Robert Moore <robert.moore@...el.com>,
	Lv Zheng <lv.zheng@...el.com>, linux-kernel@...r.kernel.org,
	linux-acpi@...r.kernel.org, devel@...ica.org,
	Sudeep Holla <sudeep.holla@....com>, lho@....com,
	dhdang@....com
Subject: Re: [PATCH] mailbox: pcc: Support HW-Reduced Communication Subspace
 Type 2

Hi,

On 05/04/16 23:14, hotran wrote:
> ACPI 6.1 has a HW-Reduced Communication Subspace Type 2 intended for
> use on HW-Reduce ACPI Platform, which requires read-modify-write sequence
> to acknowledge doorbell interrupt. This patch provides the implementation
> for the Communication Subspace Type 2.
>
> Signed-off-by: Hoan Tran <hotran@....com>
> ---
>   drivers/mailbox/pcc.c | 384 +++++++++++++++++++++++++++++++++++++-------------

>   include/acpi/actbl3.h |   8 +-

This(2nd) file is generally imported from ACPICA directly.

So you need to work with acpica-devel list(you can cc Robert Moore) to
get that alone integrated in acpica projected so that iasl tool and
other related components also gain support for this first.

If the ACPI tables are generated by UEFI, it would be good to add the
support in tianocore project. IIRC tianocore has some PCC reference in
Acpi60.h header.

It would help if the above two are done before we start looking into
Linux changes(pcc.c). To summarize this patch needs to be split.

-- 
Regards,
Sudeep