From: Mimi Zohar <zohar@linux.ibm.com>
To: Julian Braha <julianbraha@gmail.com>,
roberto.sassu@huawei.com, dmitry.kasatkin@gmail.com,
paul@paul-moore.com, jmorris@namei.org, serge@hallyn.com
Cc: eric.snowberg@oracle.com, linux-integrity@vger.kernel.org,
linux-security-module@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ima: allow users to specify the pcr index with IMA_MEASURE_PCR_IDX
Date: Tue, 01 Sep 2026 20:54:45 -0400 [thread overview]
Message-ID: <35ae5c9fd60874127d15f5fedb84c0a90e56eca4.camel@linux.ibm.com> (raw)
In-Reply-To: <20260824162215.1572367-1-julianbraha@gmail.com>
On Mon, 2026-08-24 at 17:22 +0100, Julian Braha wrote:
> The IMA_MEASURE_PCR_IDX option is currently not visible in the kconfig
> frontend, so it always uses its default, 10. This means that the
> 'range 8 14' is dead code, and users are unable to specify the pcr index
> value.
>
> In a previous discussion, Mimi explained that users should be able to use
> this config option to specify the pcr index. [1]
>
> Let's add a prompt for users to specify the pcr index, when EXPERT is
> enabled.
>
> This dead range was found by kconfirm, a static analysis tool for Kconfig.
>
> Signed-off-by: Julian Braha <julianbraha@gmail.com>
> ---
> Link: https://lore.kernel.org/all/1feff118-4afa-4b9c-86f1-271a7a88208f@gmail.com/T/#mc4efa2491b4937eb7c9e532c29ffba516a70e662 [1]
> ---
> security/integrity/ima/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/security/integrity/ima/Kconfig b/security/integrity/ima/Kconfig
> index b3a9f86809b0..2d5bb19ea6ac 100644
> --- a/security/integrity/ima/Kconfig
> +++ b/security/integrity/ima/Kconfig
> @@ -46,6 +46,7 @@ config IMA_KEXEC
>
> config IMA_MEASURE_PCR_IDX
> int
> + prompt "PCR Index for Aggregate" if EXPERT
> range 8 14
> default 10
> help
Thanks, Julian. The existing Help says, "If unsure, use the default 10." Could
we make this recommendation stronger? AI suggested:
IMA_MEASURE_PCR_IDX determines the TPM PCR register index
that IMA uses to maintain the integrity aggregate of the
measurement list. Most attestation tooling expects PCR 10.
The default is almost always what you want. Only change
this if you know what you are doing.
Thanks,
Mimi
next prev parent reply other threads:[~2026-09-02 0:55 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-24 16:22 [PATCH] ima: allow users to specify the pcr index with IMA_MEASURE_PCR_IDX Julian Braha
2026-09-02 0:54 ` Mimi Zohar [this message]
2026-09-02 11:38 ` Julian Braha
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=35ae5c9fd60874127d15f5fedb84c0a90e56eca4.camel@linux.ibm.com \
--to=zohar@linux.ibm.com \
--cc=dmitry.kasatkin@gmail.com \
--cc=eric.snowberg@oracle.com \
--cc=jmorris@namei.org \
--cc=julianbraha@gmail.com \
--cc=linux-integrity@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-security-module@vger.kernel.org \
--cc=paul@paul-moore.com \
--cc=roberto.sassu@huawei.com \
--cc=serge@hallyn.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).