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".
--
thanks,
Mimi
