linux-kernel - Re: [PATCH v3] ACPI / tables: Add IORT to injectable table list

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, 2 Feb 2018 19:54:24 +0800
From:   Hanjun Guo <guohanjun@...wei.com>
To:     Yang Shunyong <shunyong.yang@...-semitech.com>, <rjw@...ysocki.net>
CC:     <lenb@...nel.org>, <linux-acpi@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>,
        Joey Zheng <yu.zheng@...-semitech.com>,
        "Wang Dongsheng" <dongsheng.wang@...-semitech.com>,
        Jiang Yutang <yutang2.jiang@...-semitech.com>
Subject: Re: [PATCH v3] ACPI / tables: Add IORT to injectable table list

On 2018/2/2 18:25, Yang Shunyong wrote:
> Loading IORT table from initrd is used to fix firmware IORT defects.

I don't think this fix "firmware defects", it just for debug purpose,
we will not use that for production purpose, right? I think above line
can be removed.

> Moreover, it is very useful to debug SMMU node/device probe, MSI
> allocation, stream id translation and IORT table from firmware. It
> is also very useful to enable SMMU and devices behind SMMU before
> firmware is ready.
> 
> This patch adds ACPI_SIG_IORT to the table, which enables IORT
> from initrd to override which from firmware.
> 
> Signed-off-by: Yang Shunyong <shunyong.yang@...-semitech.com>
> Cc: Joey Zheng <yu.zheng@...-semitech.com>
> Cc: Wang Dongsheng <dongsheng.wang@...-semitech.com>
> Cc: Jiang Yutang <yutang2.jiang@...-semitech.com>
> Cc: Hanjun Guo <guohanjun@...wei.com>

With that updated,

Acked-by: Hanjun Guo <hanjun.guo@...aro.org>

Thanks
Hanjun