On 12/23/2025 2:32 PM, Mimi Zohar wrote:
On Tue, 2025-12-16 at 11:59 -0800, steven chen wrote:
+{
+ struct ima_queue_entry *qe, *qe_tmp;
+ LIST_HEAD(ima_measurements_staged);
+ unsigned int i;
+ long cur = number_logs;
The variable name "number_logs" is confusing. As I mentioned in the patch
description, there is one measurement list with multiple records. There aren't
multiple logs in the kernel (other than the staged list).
Will update it to "req_value". Thanks!
Please refer to the section titled "Naming" in Documentation/process/coding-
style.rst. Since this is the number of records being deleted, perhaps a better
variable name would be "num_records".
Will update. Thanks
Steven
